[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d01460fe-527e-49bd-a420-289ecbc17999@imgtec.com>
Date: Wed, 15 Oct 2025 13:26:16 +0000
From: Matt Coster <Matt.Coster@...tec.com>
To: Ankan Biswas <spyjetfayed@...il.com>
CC: "bbrezillon@...nel.org" <bbrezillon@...nel.org>,
"himal.prasad.ghimiray@...el.com" <himal.prasad.ghimiray@...el.com>,
"dakr@...nel.org" <dakr@...nel.org>,
"robin.clark@....qualcomm.com"
<robin.clark@....qualcomm.com>,
"matthew.brost@...el.com"
<matthew.brost@...el.com>,
"aliceryhl@...gle.com" <aliceryhl@...gle.com>,
"thomas.hellstrom@...ux.intel.com" <thomas.hellstrom@...ux.intel.com>,
"maarten.lankhorst@...ux.intel.com" <maarten.lankhorst@...ux.intel.com>,
"mripard@...nel.org" <mripard@...nel.org>,
"tzimmermann@...e.de"
<tzimmermann@...e.de>,
"simona@...ll.ch" <simona@...ll.ch>,
"skhan@...uxfoundation.org" <skhan@...uxfoundation.org>,
"khalid@...nel.org"
<khalid@...nel.org>,
"david.hunter.linux@...il.com"
<david.hunter.linux@...il.com>,
"linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>,
"linux-kernel-mentees@...ts.linux.dev"
<linux-kernel-mentees@...ts.linux.dev>,
"dri-devel@...ts.freedesktop.org"
<dri-devel@...ts.freedesktop.org>,
Stephen Rothwell <sfr@...b.auug.org.au>,
Alessio Belle <Alessio.Belle@...tec.com>,
Alexandru Dadu
<Alexandru.Dadu@...tec.com>,
Frank Binns <Frank.Binns@...tec.com>
Subject: Re: [PATCH v2] drm/gpuvm: Fix kernel-doc warning for
drm_gpuvm_map_req.map
On 15/10/2025 12:31, Ankan Biswas wrote:
> The kernel-doc for struct drm_gpuvm_map_req.map was added as '@...map'
> instead of '@...', leading to this warning during htmldocs build:
>
> WARNING: include/drm/drm_gpuvm.h:1083 struct member 'map' not described in 'drm_gpuvm_map_req'
>
> Fixes: 000a45dce7ad ("drm/gpuvm: Pass map arguments through a struct")
> Reported-by: Stephen Rothwell <sfr@...b.auug.org.au>
> Signed-off-by: Ankan Biswas <spyjetfayed@...il.com>
Hi Ankan,
Thanks for the fix! Is there a public report from Stephen that you can
link to with a Closes: tag after the Reported-by: tag (see [1])?
Otherwise, this looks good to me and I'll be happy to R-b and take it
via drm-misc-fixes.
Cheers,
Matt
[1]: https://www.kernel.org/doc/html/latest/process/5.Posting.html#patch-formatting-and-changelogs
> ---
> Changes in v2:
> - typos fixed 'map_ops' -> '@...map', 'map' -> '@...'
>
> include/drm/drm_gpuvm.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_gpuvm.h b/include/drm/drm_gpuvm.h
> index 8890ded1d907..476990e761f8 100644
> --- a/include/drm/drm_gpuvm.h
> +++ b/include/drm/drm_gpuvm.h
> @@ -1078,7 +1078,7 @@ struct drm_gpuva_ops {
> */
> struct drm_gpuvm_map_req {
> /**
> - * @op_map: struct drm_gpuva_op_map
> + * @map: struct drm_gpuva_op_map
> */
> struct drm_gpuva_op_map map;
> };
--
Matt Coster
E: matt.coster@...tec.com
Download attachment "OpenPGP_signature.asc" of type "application/pgp-signature" (237 bytes)
Powered by blists - more mailing lists