[PATCH] drm/bridge: Separate of_drm_find_bridge() preconditions list

Bagas Sanjaya posted 1 patch 3 weeks, 4 days ago
drivers/gpu/drm/drm_bridge.c | 1 +
1 file changed, 1 insertion(+)
[PATCH] drm/bridge: Separate of_drm_find_bridge() preconditions list
Posted by Bagas Sanjaya 3 weeks, 4 days ago
Sphinx reports htmldocs warnings:

Documentation/gpu/drm-kms-helpers:197: drivers/gpu/drm/drm_bridge.c:1521: ERROR: Unexpected indentation. [docutils]
Documentation/gpu/drm-kms-helpers:197: drivers/gpu/drm/drm_bridge.c:1523: WARNING: Block quote ends without a blank line; unexpected unindent. [docutils]

Fix them by separating preconditions list in of_drm_find_bridge()
kernel-doc comment from previous paragraph.

Fixes: 9da0e06abda87b ("drm/bridge: deprecate of_drm_find_bridge()")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Closes: https://lore.kernel.org/linux-next/20260105155735.3b4012b6@canb.auug.org.au/
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 drivers/gpu/drm/drm_bridge.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c
index 6dcf8f6d3ecfa6..b788c4279add5b 100644
--- a/drivers/gpu/drm/drm_bridge.c
+++ b/drivers/gpu/drm/drm_bridge.c
@@ -1518,6 +1518,7 @@ EXPORT_SYMBOL(of_drm_find_and_get_bridge);
  * The bridge returned by this function is not refcounted. This is
  * dangerous because the bridge might be deallocated even before the caller
  * has a chance to use it. To use this function you have to do one of:
+ *
  * - get a reference with drm_bridge_get() as soon as possible to
  *   minimize the race window, and then drm_bridge_put() when no longer
  *   using the pointer

base-commit: c40b50c3cfbe274f054c6a2d2fa62cd7c4650460
-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH] drm/bridge: Separate of_drm_find_bridge() preconditions list
Posted by Luca Ceresoli 3 weeks, 3 days ago
Hello Bagas,

On Wed Jan 14, 2026 at 12:56 AM CET, Bagas Sanjaya wrote:
> Sphinx reports htmldocs warnings:
>
> Documentation/gpu/drm-kms-helpers:197: drivers/gpu/drm/drm_bridge.c:1521: ERROR: Unexpected indentation. [docutils]
> Documentation/gpu/drm-kms-helpers:197: drivers/gpu/drm/drm_bridge.c:1523: WARNING: Block quote ends without a blank line; unexpected unindent. [docutils]
>
> Fix them by separating preconditions list in of_drm_find_bridge()
> kernel-doc comment from previous paragraph.
>
> Fixes: 9da0e06abda87b ("drm/bridge: deprecate of_drm_find_bridge()")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Closes: https://lore.kernel.org/linux-next/20260105155735.3b4012b6@canb.auug.org.au/
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>

Thanks for your patch.

However the issue had been previously reported on Dec 30 [0] and I had sent
a fix the next day [1], but without any feedback so I couldn't apply it to
drm-misc-next.

My patch is slightly different from yours. I'm OK with either version.

[0] https://lore.kernel.org/oe-kbuild-all/202512302319.1PGGt3CN-lkp@intel.com/
[1] https://lore.kernel.org/lkml/20251231-drm-bridge-alloc-getput-drm_of_find_bridge-kdoc-fix-v1-1-193a03f0609c@bootlin.com/

Best regards,
Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com