[AMD Official Use Only - General]
Looks good to me. Approved.
Reviewed-by: Alvin Lee <alvin.l...@amd.com>
Thanks,
Alvin
-----Original Message-----
From: Srinivasan Shanmugam <srinivasan.shanmu...@amd.com>
Sent: Monday, June 12, 2023 10:00 AM
To: Pillai, Aurabindo <aurabindo.pil...@amd.com>; Siqueira, Rodrigo
<rodrigo.sique...@amd.com>; Wentland, Harry <harry.wentl...@amd.com>; Mahfooz,
Hamza <hamza.mahf...@amd.com>; Wang, Chao-kai (Stylon) <stylon.w...@amd.com>;
Lee, Alvin <alvin.l...@amd.com>
Cc: amd-gfx@lists.freedesktop.org; SHANMUGAM, SRINIVASAN
<srinivasan.shanmu...@amd.com>
Subject: [PATCH] drm/amd/display: Correct kdoc formats in
dcn32_resource_helpers.c
Fixes the following gcc with W=1:
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:285:
warning: Function parameter or member 'dc' not described in
'dcn32_determine_det_override'
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:285:
warning: Function parameter or member 'context' not described in
'dcn32_determine_det_override'
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:285:
warning: Function parameter or member 'pipes' not described in
'dcn32_determine_det_override'
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:624:
warning: Cannot understand *
*****************************************************************
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:676:
warning: Cannot understand *
*****************************************************************
Cc: Alvin Lee <alvin.l...@amd.com>
Cc: Stylon Wang <stylon.w...@amd.com>
Cc: Hamza Mahfooz <hamza.mahf...@amd.com>
Cc: Rodrigo Siqueira <rodrigo.sique...@amd.com>
Cc: Harry Wentland <harry.wentl...@amd.com>
Cc: Aurabindo Pillai <aurabindo.pil...@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmu...@amd.com>
Reviewed-by: Rodrigo Siqueira <rodrigo.sique...@amd.com>
---
.../display/dc/dcn32/dcn32_resource_helpers.c | 29 ++++++++-----------
1 file changed, 12 insertions(+), 17 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c
b/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c
index 4882c3684b82..a9c41ef0751f 100644
--- a/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c
+++ b/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c
@@ -273,9 +273,9 @@ bool dcn32_is_psr_capable(struct pipe_ctx *pipe)
* among those pipes.
* 4. Assign the DET override to the DML pipes.
*
- * @param [in]: dc: Current DC state
- * @param [in]: context: New DC state to be programmed
- * @param [in]: pipes: Array of DML pipes
+ * @dc: Current DC state
+ * @context: New DC state to be programmed
+ * @pipes: Array of DML pipes
*
* Return: void
*/
@@ -621,11 +621,10 @@ bool dcn32_check_native_scaling_for_res(struct pipe_ctx
*pipe, unsigned int widt }
/**
- *
************************************************************************************************
- * dcn32_subvp_drr_admissable: Determine if SubVP + DRR config is admissible
+ * dcn32_subvp_drr_admissable() - Determine if SubVP + DRR config is
+ admissible
*
- * @param [in]: dc: Current DC state
- * @param [in]: context: New DC state to be programmed
+ * @dc: Current DC state
+ * @context: New DC state to be programmed
*
* SubVP + DRR is admissible under the following conditions:
* - Config must have 2 displays (i.e., 2 non-phantom master pipes) @@ -633,9
+632,7 @@ bool dcn32_check_native_scaling_for_res(struct pipe_ctx *pipe,
unsigned int widt
* - Other display must have Freesync enabled
* - The potential DRR display must not be PSR capable
*
- * @return: True if admissible, false otherwise
- *
- *
************************************************************************************************
+ * Return: True if admissible, false otherwise
*/
bool dcn32_subvp_drr_admissable(struct dc *dc, struct dc_state *context) { @@
-673,11 +670,11 @@ bool dcn32_subvp_drr_admissable(struct dc *dc, struct
dc_state *context) }
/**
- *
************************************************************************************************
- * dcn32_subvp_vblank_admissable: Determine if SubVP + Vblank config is
admissible
+ * dcn32_subvp_vblank_admissable() - Determine if SubVP + Vblank config
+ is admissible
*
- * @param [in]: dc: Current DC state
- * @param [in]: context: New DC state to be programmed
+ * @dc: Current DC state
+ * @context: New DC state to be programmed
+ * @vlevel: Voltage level calculated by DML
*
* SubVP + Vblank is admissible under the following conditions:
* - Config must have 2 displays (i.e., 2 non-phantom master pipes) @@ -686,9
+683,7 @@ bool dcn32_subvp_drr_admissable(struct dc *dc, struct dc_state
*context)
* - DML must have output DRAM clock change support as SubVP + Vblank
* - The potential vblank display must not be PSR capable
*
- * @return: True if admissible, false otherwise
- *
- *
************************************************************************************************
+ * Return: True if admissible, false otherwise
*/
bool dcn32_subvp_vblank_admissable(struct dc *dc, struct dc_state *context,
int vlevel) {
--
2.25.1
