[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <qyoun6sfdpdxxzwystaimsiw3o33jwvoqxm3kpawczju2b3dl5@ytjpgyabbzet>
Date: Wed, 24 Jul 2024 16:30:39 -0300
From: "Ricardo B. Marliere" <rbmarliere@...il.com>
To: Abhishek Tamboli <abhishektamboli9@...il.com>
Cc: airlied@...il.com, daniel@...ll.ch, harry.wentland@....com,
sunpeng.li@....com, Rodrigo.Siqueira@....com, alexander.deucher@....com,
christian.koenig@....com, Xinhui.Pan@....com, amd-gfx@...ts.freedesktop.org,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org, skhan@...uxfoundation.org,
marcelomspessoto@...il.com, aurabindo.pillai@....com, adnelson@....com,
linux-kernel-mentees@...ts.linuxfoundation.org
Subject: Re: [PATCH] drm/amd/display: Fix documentation warning for mpc.h
On 24 Jul 24 23:24, Abhishek Tamboli wrote:
> Fix documentation compile warning by adding description
> for program_3dlut_size function.
>
> Remove the following warning:
> ./drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h:1043: warning: Function parameter or struct member 'program_3dlut_size' not described in 'mpc_funcs'
>
> Signed-off-by: Abhishek Tamboli <abhishektamboli9@...il.com>
> ---
> drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h | 16 +++++++++++++++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h b/drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h
> index 40a9b3471208..615c69d966e7 100644
> --- a/drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h
> +++ b/drivers/gpu/drm/amd/display/dc/inc/hw/mpc.h
> @@ -1039,7 +1039,21 @@ struct mpc_funcs {
> */
> void (*program_lut_mode)(struct mpc *mpc, const enum MCM_LUT_ID id, const enum MCM_LUT_XABLE xable,
> bool lut_bank_a, int mpcc_id);
> - void (*program_3dlut_size)(struct mpc *mpc, bool is_17x17x17, int mpcc_id);
> + /**
> + * @program_3dlut_size:
> + *
> + * Program 3D LUT size.
> + *
> + * Parameters:
> + * - [in/out] mpc - MPC context.
> + * - [in] is_17x17x17 - Boolean Flag.
> + * - [in] mpcc_id - MPCC physical instance.
> + *
> + * Return:
> + *
> + * void
> + */
Reviewed-by: Ricardo B. Marliere <ricardo@...liere.net>
> + void (*program_3dlut_size)(struct mpc *mpc, bool is_17x17x17, int mpcc_id);
> };
>
> #endif
> --
> 2.34.1
>
Powered by blists - more mailing lists