From: "Luca Ceresoli" <luca.ceresoli@bootlin.com>
To: "Doug Anderson" <dianders@chromium.org>,
"Luca Ceresoli" <luca.ceresoli@bootlin.com>
Cc: "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>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.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>,
<linux-arm-kernel@lists.infradead.org>,
<linux-mediatek@lists.infradead.org>,
"Jitao Shi" <jitao.shi@mediatek.com>,
"CK Hu" <ck.hu@mediatek.com>,
"Pin-yen Lin" <treapking@chromium.org>,
"Enric Balletbo i Serra" <eballetbo@kernel.org>
Subject: Re: [PATCH v2 08/12] drm/bridge: parade-ps8640: select DRM_PANEL_BRIDGE
Date: Mon, 31 Aug 2026 11:01:00 +0200 [thread overview]
Message-ID: <DL308Z0JXJ31.2V3H29DTG4MSM@bootlin.com> (raw)
In-Reply-To: <CAD=FV=Xdj_foBqt9A9EVVoCA2BxAV3UZNz3SuVPu2ZoNXgnS=w@mail.gmail.com>
Hi,
On Thu Aug 27, 2026 at 6:23 PM CEST, Doug Anderson wrote:
> Hi,
>
> On Tue, Aug 25, 2026 at 3:11 AM Luca Ceresoli <luca.ceresoli@bootlin.com> wrote:
>>
>> This driver uses the drm_panel_bridge API and not the drm_panel API so it
>> should select DRM_PANEL_BRIDGE and not DRM_PANEL.
>
> Which exact "drm_panel_bridge" API does it use? I don't see any...
It calls devm_drm_of_get_bridge() [0], which is in bridge/panel.c [1],
which is enabled by CONFIG_DRM_PANEL_BRIDGE [2].
[0] https://elixir.bootlin.com/linux/v7.2.2/source/drivers/gpu/drm/bridge/parade-ps8640.c#L636
[1] https://elixir.bootlin.com/linux/v7.2.2/source/drivers/gpu/drm/bridge/panel.c#L483-L515
[2] https://elixir.bootlin.com/linux/v7.2.2/source/drivers/gpu/drm/Makefile#L159
> Certainly it's highly likely that the next bridge in the chain is a
> panel, but there are no API calls that I see. Maybe the scripts got
> confused because the variable name didn't get updated in commit
No script, just this humble human got confused X-)
> 5486eac81004 ("drm/bridge: parade-ps8640: switch to
> devm_drm_of_get_bridge") ?
Luca
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2026-08-31 9:01 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-25 10:09 [PATCH v2 00/12] drm/bridge: Kconfig: cleanup DRM_PANEL usage Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 01/12] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: don't select unused DRM_PANEL Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 02/12] drm/bridge: tc358764: " Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 03/12] drm/bridge: ti-dlpc3433: don't depend on DRM_PANEL Luca Ceresoli
2026-08-25 10:22 ` sashiko-bot
2026-08-25 10:09 ` [PATCH v2 04/12] drm/bridge: ti-sn65dsi86: don't select unused DRM_PANEL Luca Ceresoli
2026-08-27 16:24 ` Doug Anderson
2026-08-31 8:55 ` Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 05/12] drm/bridge: ti-sn65dsi83: select DRM_PANEL_BRIDGE Luca Ceresoli
2026-08-25 12:24 ` Herve Codina
2026-08-25 13:40 ` Esben Haabendal
2026-08-25 10:09 ` [PATCH v2 06/12] drm/bridge: nxp-ptn3460: " Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 07/12] drm/bridge: parade-ps8622: " Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 08/12] drm/bridge: parade-ps8640: " Luca Ceresoli
2026-08-27 16:23 ` Doug Anderson
2026-08-31 9:01 ` Luca Ceresoli [this message]
2026-08-31 14:58 ` Doug Anderson
2026-08-31 15:26 ` Luca Ceresoli
2026-08-31 15:28 ` Doug Anderson
2026-08-25 10:09 ` [PATCH v2 09/12] drm/bridge: tc358775: " Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 10/12] drm/bridge: tc358767: " Luca Ceresoli
2026-08-25 10:32 ` sashiko-bot
2026-08-25 10:09 ` [PATCH v2 11/12] drm/bridge: tc358768: " Luca Ceresoli
2026-08-25 10:09 ` [PATCH v2 12/12] drm/bridge: ssd2825: " Luca Ceresoli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DL308Z0JXJ31.2V3H29DTG4MSM@bootlin.com \
--to=luca.ceresoli@bootlin.com \
--cc=Hui.Pu@gehealthcare.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=ck.hu@mediatek.com \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=eballetbo@kernel.org \
--cc=ian.ray@gehealthcare.com \
--cc=jernej.skrabec@gmail.com \
--cc=jitao.shi@mediatek.com \
--cc=jonas@kwiboo.se \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=matthias.bgg@gmail.com \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=simona@ffwll.ch \
--cc=thomas.petazzoni@bootlin.com \
--cc=treapking@chromium.org \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.