[PATCH 1/2] drm/amd/display: fix comment format
André Almeida
andrealmeid at igalia.com
Tue Jul 12 14:40:59 UTC 2022
Hi Alex,
Às 11:31 de 12/07/22, Alex Deucher escreveu:
> This is not a compliant kerneldoc comment, so make it just
> a regular comment.
>
> Reported-by: kernel test robot <lkp at intel.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> drivers/gpu/drm/amd/display/dc/dcn314/dcn314_optc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_optc.c b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_optc.c
> index c4a5f8198209..436c3545a983 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_optc.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_optc.c
> @@ -42,7 +42,7 @@
> #define FN(reg_name, field_name) \
> optc1->tg_shift->field_name, optc1->tg_mask->field_name
>
> -/**
> +/*
> * Enable CRTC
> * Enable CRTC - call ASIC Control Object to enable Timing generator.
> */
This comment is placed in a random part of the file, the proper fix it
would be to place it above the function that it's describing:
static bool optc314_enable_crtc(struct timing_generator *optc)
There it could even be a correct kerneldoc comment, fixed the function
name and adding the argument description.
More information about the amd-gfx
mailing list