Re: [PATCH v3 19/19] drm/bridge: analogix_dp: don't create a panel_bridge

From: Heiko Stuebner <heiko@sntech.de>
To: Damon Ding <damon.ding@rock-chips.com>,
	Luca Ceresoli <luca.ceresoli@bootlin.com>,
	Andrzej Hajda <andrzej.hajda@intel.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Robert Foss <rfoss@kernel.org>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Jonas Karlman <jonas@kwiboo.se>,
	Jernej Skrabec <jernej.skrabec@gmail.com>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Jessica Zhang <jesszhan0024@gmail.com>,
	Laurentiu Palcu <laurentiu.palcu@oss.nxp.com>,
	Lucas Stach <l.stach@pengutronix.de>, Frank Li <Frank.Li@nxp.com>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Paul Cercueil <paul@crapouillou.net>,
	Paul Kocialkowski <paulk@sys-base.io>,
	Linus Walleij <linusw@kernel.org>,
	Inki Dae <inki.dae@samsung.com>,
	Jagan Teki <jagan@amarulasolutions.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Sean Paul <sean@poorly.run>, Jonathan Corbet <corbet@lwn.net>,
	Shuah Khan <skhan@linuxfoundation.org>,
	Randy Dunlap <rdunlap@infradead.org>,
	Luca Ceresoli <luca.ceresoli@bootlin.com>
Cc: Albert Esteve <aesteve@redhat.com>,
	Anusha Srivatsa <asrivats@redhat.com>,
	Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
	Hui Pu <Hui.Pu@gehealthcare.com>,
	Ian Ray <ian.ray@gehealthcare.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
	linux-mips@vger.kernel.org, linux-rockchip@lists.infradead.org,
	linux-stm32@st-md-mailman.stormreply.com, asahi@lists.linux.dev,
	linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org,
	linux-tegra@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH v3 19/19] drm/bridge: analogix_dp: don't create a panel_bridge
Date: Tue, 22 Sep 2026 10:22:37 +0200	[thread overview]
Message-ID: <1889588.TLkxdtWsSY@phil> (raw)
In-Reply-To: <DLLOZYM6YWJ3.6Q0LRD1WABGW@bootlin.com>

Hi Luca,

Am Dienstag, 22. September 2026, 10:12:41 Mitteleuropäische Sommerzeit schrieb Luca Ceresoli:
> On Mon Sep 21, 2026 at 5:07 AM CEST, Damon Ding wrote:
> > On 9/16/2026 9:45 PM, Luca Ceresoli wrote:

[...]

> >> @@ -163,18 +161,6 @@ static int exynos_dp_probe(struct platform_device *pdev)
> >>   	 */
> >>   	platform_set_drvdata(pdev, dp);
> >>
> >> -	/* This is for the backward compatibility. */
> >> -	np = of_parse_phandle(dev->of_node, "panel", 0);
> >> -	if (np) {
> >> -		dp->plat_data.panel = of_drm_find_panel(np);
> >> -
> >> -		of_node_put(np);
> >> -		if (IS_ERR(dp->plat_data.panel))
> >> -			return PTR_ERR(dp->plat_data.panel);
> >> -
> >> -		goto out;
> >> -	}
> >> -
> >
> > TIP: In my WIP v3 patch series, I use of_drm_find_panel() together with
> > devm_drm_panel_bridge_add() to maintain this backward compatibility. But
> > I also think this old approach should be deprecated now. Shall we add
> > some notes in the commit message to mention the deprecation of this method?
> 
> What is the "WIP v3" series exactly? Do you have a link (to v2 in case v3
> is not yet public)?

https://lore.kernel.org/linux-rockchip/20260804081717.741404-1-damon.ding@rock-chips.com/
is the series in question.

Heiko