[PATCH] [drm-misc-next] drm/nouveau: fix kernel-doc comments

Ben Skeggs bskeggs at nvidia.com
Thu Jan 9 03:41:28 UTC 2025


On 9/1/25 09:43, Timur Tabi wrote:

> Fix some malformed kernel-doc comments that were added in a recent commit.
>
> Also, kernel-doc does not support global variables, so change those
> kernel-doc comments into regular comments.
>
> Fixes: 214c9539cf2f ("drm/nouveau: expose GSP-RM logging buffers via debugfs")
> Reported-by: kernel test robot <lkp at intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202412310834.jtCJj4oz-lkp@intel.com/
>
> Signed-off-by: Timur Tabi <ttabi at nvidia.com>

Reviewed-by: Ben Skeggs <bskeggs at nvidia.com>

> ---
>   drivers/gpu/drm/nouveau/nouveau_drm.c          |  2 +-
>   drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c | 12 ++++++++----
>   2 files changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c
> index 21d2d9ca5e85..8c970f018c00 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_drm.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
> @@ -118,7 +118,7 @@ static struct drm_driver driver_platform;
>   #ifdef CONFIG_DEBUG_FS
>   struct dentry *nouveau_debugfs_root;
>   
> -/**
> +/*
>    * gsp_logs - list of nvif_log GSP-RM logging buffers
>    *
>    * Head pointer to a a list of nvif_log buffers that is created for each GPU
> diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c b/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> index 58502102926b..10fe2d15b5ce 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> @@ -1111,7 +1111,7 @@ enum registry_type {
>   #define REGISTRY_MAX_KEY_LENGTH		64
>   
>   /**
> - * registry_list_entry - linked list member for a registry key/value
> + * struct registry_list_entry - linked list member for a registry key/value
>    * @head: list_head struct
>    * @type: dword, binary, or string
>    * @klen: the length of name of the key
> @@ -1327,7 +1327,7 @@ struct nv_gsp_registry_entries {
>   	u32 value;
>   };
>   
> -/**
> +/*
>    * r535_registry_entries - required registry entries for GSP-RM
>    *
>    * This array lists registry entries that are required for GSP-RM to
> @@ -2101,7 +2101,7 @@ MODULE_PARM_DESC(keep_gsp_logging,
>   #define NV_GSP_MSG_EVENT_UCODE_LIBOS_CLASS_PMU		0xf3d722
>   
>   /**
> - * rpc_ucode_libos_print_v1E_08 - RPC payload for libos print buffers
> + * struct rpc_ucode_libos_print_v1e_08 - RPC payload for libos print buffers
>    * @ucode_eng_desc: the engine descriptor
>    * @libos_print_buf_size: the size of the libos_print_buf[]
>    * @libos_print_buf: the actual buffer
> @@ -2162,7 +2162,7 @@ r535_gsp_msg_libos_print(void *priv, u32 fn, void *repv, u32 repc)
>   }
>   
>   /**
> - * create_debufgs - create a blob debugfs entry
> + * create_debugfs - create a blob debugfs entry
>    * @gsp: gsp pointer
>    * @name: name of this dentry
>    * @blob: blob wrapper
> @@ -2788,6 +2788,10 @@ static bool is_empty(const struct debugfs_blob_wrapper *b)
>   
>   /**
>    * r535_gsp_copy_log - preserve the logging buffers in a blob
> + * @parent: the top-level dentry for this GPU
> + * @name: name of debugfs entry to create
> + * @s: original wrapper object to copy from
> + * @t: new wrapper object to copy to
>    *
>    * When GSP shuts down, the nvkm_gsp object and all its memory is deleted.
>    * To preserve the logging buffers, the buffers need to be copied, but only
>
> base-commit: 5d40d4fae6f2fb789f48207a9d4772bbee970b5c
> prerequisite-patch-id: dbfbf26b276d26be3dbeec57723c6ac0aec4a102


More information about the Nouveau mailing list