linux-next: build warnings after merge of the drm-misc-fixes tree

linux-next: build warnings after merge of the drm-misc-fixes tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 1568 bytes --]

Hi all,

After merging the drm-misc-fixes tree, today's linux-next build (htmldocs)
produced these warnings:

Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2446: ERROR: Unexpected indentation. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2448: WARNING: Block quote ends without a blank line; unexpected unindent. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2452: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2453: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2457: ERROR: Unexpected indentation. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2458: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2459: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2460: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]

Introduced by commit

  6cc44e9618f0 ("drm: Add directive to format code in comment")

interacting with commit

  bb324f85f722 ("drm/gpuvm: Wrap drm_gpuvm_sm_map_exec_lock() expected usage in literal code block")

from the drm-msm-fixes tree.  Both get applied.

The drm-misc-fixes commit is unneeded.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the drm-misc-fixes tree

[Bagas Sanjaya]

[-- Attachment #1: Type: text/plain, Size: 1725 bytes --]

On Mon, Aug 18, 2025 at 12:38:53PM +1000, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the drm-misc-fixes tree, today's linux-next build (htmldocs)
> produced these warnings:
> 
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2446: ERROR: Unexpected indentation. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2448: WARNING: Block quote ends without a blank line; unexpected unindent. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2452: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2453: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2457: ERROR: Unexpected indentation. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2458: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2459: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
> Documentation/gpu/drm-mm:506: drivers/gpu/drm/drm_gpuvm.c:2460: WARNING: Definition list ends without a blank line; unexpected unindent. [docutils]
> 
> Introduced by commit
> 
>   6cc44e9618f0 ("drm: Add directive to format code in comment")
> 
> interacting with commit
> 
>   bb324f85f722 ("drm/gpuvm: Wrap drm_gpuvm_sm_map_exec_lock() expected usage in literal code block")

Duh! drm_gpuvm.c gets tangled...

Danilo, can you please drop my commit (and kept Javier's one)?

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: linux-next: build warnings after merge of the drm-misc-fixes tree

[Danilo Krummrich]

On 8/18/25 7:56 AM, Bagas Sanjaya wrote:
> Danilo, can you please drop my commit (and kept Javier's one)?

I think yours is in drm-msm-fixes, which I don't maintain.

@Rob: Can you drop it please?

Re: linux-next: build warnings after merge of the drm-misc-fixes tree

[Bala-Vignesh-Reddy]

Hi all,

Those htmldocs warnings come from missing Sphinx markup in 
`drm_gpuvm_sm_map_exec_lock()`.

This is already addressed by my patch:

[PATCH] drm/gpuvm: Fix kernel-doc formatting and typo in
drm_gpuvm_sm_map_exec_lock

Link: https://lore.kernel.org/lkml/20250803092622.27532-1-reddybalavignesh9979@gmail.com/

The patch inserts the required directive and corrects the return-value typo.
I have verified this.

Could anyone please pick this up or keep it queued, so 
warnings disappear from -next.

Thanks!
Bala Vignesh

Re: linux-next: build warnings after merge of the drm-misc-fixes tree

[Bagas Sanjaya]

[-- Attachment #1: Type: text/plain, Size: 834 bytes --]

On Thu, Aug 21, 2025 at 12:05:07AM +0530, Bala-Vignesh-Reddy wrote:
> Hi all,
> 
> Those htmldocs warnings come from missing Sphinx markup in 
> `drm_gpuvm_sm_map_exec_lock()`.
> 
> This is already addressed by my patch:
> 
> [PATCH] drm/gpuvm: Fix kernel-doc formatting and typo in
> drm_gpuvm_sm_map_exec_lock
> 
> Link: https://lore.kernel.org/lkml/20250803092622.27532-1-reddybalavignesh9979@gmail.com/
> 
> The patch inserts the required directive and corrects the return-value typo.
> I have verified this.
> 
> Could anyone please pick this up or keep it queued, so 
> warnings disappear from -next.

I think this has been already addressed by Javier's patch and mine (see the
full thread). However, one of them must be in mainline.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]