linux-kernel - Re: [PATCH v8 05/18] drm/bridge: Move legacy bridge driver out of imx directory for multi-platform use

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <DG2S064XPGXL.1MQDAKWZEK50K@bootlin.com>
Date: Sat, 31 Jan 2026 13:33:29 +0100
From: "Luca Ceresoli" <luca.ceresoli@...tlin.com>
To: "Damon Ding" <damon.ding@...k-chips.com>, <andrzej.hajda@...el.com>,
 <neil.armstrong@...aro.org>, <rfoss@...nel.org>
Cc: <Laurent.pinchart@...asonboard.com>, <jonas@...boo.se>,
 <jernej.skrabec@...il.com>, <maarten.lankhorst@...ux.intel.com>,
 <mripard@...nel.org>, <tzimmermann@...e.de>, <airlied@...il.com>,
 <simona@...ll.ch>, <shawnguo@...nel.org>, <s.hauer@...gutronix.de>,
 <kernel@...gutronix.de>, <festevam@...il.com>, <inki.dae@...sung.com>,
 <sw0312.kim@...sung.com>, <kyungmin.park@...sung.com>, <krzk@...nel.org>,
 <alim.akhtar@...sung.com>, <jingoohan1@...il.com>,
 <p.zabel@...gutronix.de>, <hjc@...k-chips.com>, <heiko@...ech.de>,
 <andy.yan@...k-chips.com>, <dmitry.baryshkov@....qualcomm.com>,
 <dianders@...omium.org>, <m.szyprowski@...sung.com>,
 <jani.nikula@...el.com>, <linux-kernel@...r.kernel.org>,
 <dri-devel@...ts.freedesktop.org>, <imx@...ts.linux.dev>,
 <linux-arm-kernel@...ts.infradead.org>,
 <linux-samsung-soc@...r.kernel.org>, <linux-rockchip@...ts.infradead.org>
Subject: Re: [PATCH v8 05/18] drm/bridge: Move legacy bridge driver out of
 imx directory for multi-platform use

Hello Damon, Dmitry,

On Wed Dec 17, 2025 at 10:33 AM CET, Damon Ding wrote:
> As suggested by Dmitry, the DRM legacy bridge driver can be pulled
> out of imx/ subdir for multi-platform use. The driver is also renamed
> to make it more generic and suitable for platforms other than i.MX.
>
> Signed-off-by: Damon Ding <damon.ding@...k-chips.com>
> Suggested-by: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
> Tested-by: Marek Szyprowski <m.szyprowski@...sung.com>
>
> ---
>
> Changes in v7:
> - Rename legacy-bridge to of-display-mode-bridge.
> - Remove unnecessary API drm_bridge_is_legacy().

...

> --- a/drivers/gpu/drm/bridge/Kconfig
> +++ b/drivers/gpu/drm/bridge/Kconfig
> @@ -244,6 +244,16 @@ config DRM_NXP_PTN3460
>  	help
>  	  NXP PTN3460 eDP-LVDS bridge chip driver.
>
> +config DRM_OF_DISPLAY_MODE_BRIDGE
> +	tristate
> +	depends on DRM_BRIDGE && OF
> +	help
> +	  This is a DRM bridge implementation that uses of_get_drm_display_mode
> +	  to acquire display mode.
> +
> +	  Newer designs should not use this bridge and should use proper panel
> +	  driver instead.

"Newer designs should not use this bridge", but in patch 8 you are
introducing a new use ot devm_drm_of_display_mode_bridge(). Shouldn't you
instead "use proper panel driver instead" in patch 8?

Can you point to the conversation where the converstaion where Dmitry
suggested this change? Maybe it already contains the answer to my
qustion. Also adding a link to that conversation in the commit message
woule be useful.

Luca

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