[PATCH v14 2/3] drm: bridge: panel: Add drmm_panel_bridge_add_nodrm

Maxime Ripard maxime at cerno.tech
Mon Feb 27 17:47:27 UTC 2023


On Mon, Feb 27, 2023 at 11:02:30PM +0530, Jagan Teki wrote:
> drmm_panel_bridge_add_nodrm is an another type of DRM-managed action
> helper with nodrm pointer.
> 
> DRM pointer is required to perform DRM-managed action,
> - The conventional component-based drm bridges, the DRM pointer can
>   access in component ops bind API.
> - The non-component-based bridges (like host DSI bridges), the DRM
>   pointer can access only when a specific bridge has been found
>   via bridge->dev.
> 
> This drmm_panel_bridge_add_nodrm helper exclusively for the
> non-component-based bridges.
> 
> Cc: Maxime Ripard <mripard at kernel.org>
> Cc: Laurent Pinchart <Laurent.pinchart at ideasonboard.com>
> Cc: Linus Walleij <linus.walleij at linaro.org>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>

It's the third or fourth time I'm telling you this, and second time
today.

You keep respinning that patch, but the issue lies in the samsung
driver. Fix it and you won't need that helper.

Until then, you'll only get an automatic NAK from me, I'm done reviewing
those patches over and over again, making the same comments that you'll
just ignore.

Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20230227/646b297e/attachment.sig>


More information about the dri-devel mailing list