Re: [PATCH v2] Fix duplicate C declaration warnings

From: Matthew Wilcox
Date: Sat Mar 23 2024 - 16:44:04 EST


On Sat, Mar 23, 2024 at 10:01:47PM +0530, Amogh Cheluvaraj wrote:
> Fix duplicate C declaration warnings at
> Documentation/gpu/drm-kms.rst that was found by
> compiling htmldocs

I'm sure this removes the warning, but it removes all kernel-doc
which exists in drivers/gpu/drm/drm_fourcc.c. Isn't there a more
granular fix than this?

> /home/amogh/Linux_Kernel_Workspace/linux-next/Documentation/gpu/drm-
> kms:360: ./drivers/gpu/drm/drm_fourcc.c:344: WARNING: Duplicate C
> declaration, also defined at gpu/drm-kms:39.
> Declaration is '.. c:function:: const struct drm_format_info *
> drm_format_info (u32 format)'.
> /home/amogh/Linux_Kernel_Workspace/linux-next/Documentation/gpu/drm-
> kms:461: ./drivers/gpu/drm/drm_modeset_lock.c:392: WARNING: Duplicate C
> declaration, also defined at gpu/drm-kms:49.
> Declaration is '.. c:function:: int drm_modeset_lock (struct
> drm_modeset_lock *lock, struct drm_modeset_acquire_ctx *ctx)'.
>
> Signed-off-by: Amogh Cheluvaraj <amogh.linux.kernel.dev@xxxxxxxxx>
> ---
>
> changes in v2
> - add warnings found after compilation
> - fix grammar in commit description
>
> ---
> Documentation/gpu/drm-kms.rst | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/Documentation/gpu/drm-kms.rst b/Documentation/gpu/drm-kms.rst
> index 13d3627d8bc0..a4145f391e43 100644
> --- a/Documentation/gpu/drm-kms.rst
> +++ b/Documentation/gpu/drm-kms.rst
> @@ -357,9 +357,6 @@ Format Functions Reference
> .. kernel-doc:: include/drm/drm_fourcc.h
> :internal:
>
> -.. kernel-doc:: drivers/gpu/drm/drm_fourcc.c
> - :export:
> -
> .. _kms_dumb_buffer_objects:
>
> Dumb Buffer Objects
> @@ -458,9 +455,6 @@ KMS Locking
> .. kernel-doc:: include/drm/drm_modeset_lock.h
> :internal:
>
> -.. kernel-doc:: drivers/gpu/drm/drm_modeset_lock.c
> - :export:
> -
> KMS Properties
> ==============
>
> --
> 2.44.0
>
>