[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2776792.KRxA6XjA2N@diego>
Date: Sun, 28 Dec 2025 18:41:06 +0100
From: Heiko Stübner <heiko@...ech.de>
To: Andy Yan <andyshrk@....com>, Icenowy Zheng <zhengxingda@...as.ac.cn>
Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Drew Fustini <fustini@...nel.org>,
Guo Ren <guoren@...nel.org>, Fu Wei <wefu@...hat.com>,
Philipp Zabel <p.zabel@...gutronix.de>,
Andrzej Hajda <andrzej.hajda@...el.com>,
Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>,
Michal Wilczynski <m.wilczynski@...sung.com>, devicetree@...r.kernel.org,
Yao Zi <ziyao@...root.org>, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org, Han Gao <rabenda.cn@...il.com>,
linux-riscv@...ts.infradead.org
Subject:
Re: [PATCH v4 5/9] drm/bridge: add a driver for T-Head TH1520 HDMI controller
Am Samstag, 27. Dezember 2025, 08:22:05 Mitteleuropäische Normalzeit schrieb Icenowy Zheng:
> 在 2025-12-27星期六的 15:03 +0800,Andy Yan写道:
> >
> >
> > Hello,
> >
> > 在 2025-12-26 21:27:53,"Icenowy Zheng" <zhengxingda@...as.ac.cn> 写道:
> > > 在 2025-12-25星期四的 19:07 +0800,Andy Yan写道:
> > > >
> > > >
> > > > Hello Icenowy:
> > > >
> > > > At 2025-12-25 00:12:01, "Icenowy Zheng" <zhengxingda@...as.ac.cn>
> > > > wrote:
> > > > > From: Icenowy Zheng <uwu@...nowy.me>
> > > > >
> > > > > T-Head TH1520 SoC contains a Synopsys DesignWare HDMI
> > > > > controller
> > > > > (paired
> > > > > with DesignWare HDMI TX PHY Gen2) that takes the "DP" output
> > > > > from
> > > > > the
> > > > > display controller.
> > > > >
> > > > > Add a driver for this controller utilizing the common
> > > > > DesignWare
> > > > > HDMI
> > > > > code in the kernel.
> > > > >
> > > > > Signed-off-by: Icenowy Zheng <uwu@...nowy.me>
> > > > > Signed-off-by: Icenowy Zheng <zhengxingda@...as.ac.cn>
> > > > > ---
> > > > > No changes in v3, v4.
> > > > >
> > > > > Changes in v2:
> > > > > - Created a new function to set PHY parameters and refactored
> > > > > the
> > > > > control flow of the configure_phy callback.
> > > > >
> > > > > MAINTAINERS | 1 +
> > > > > drivers/gpu/drm/bridge/Kconfig | 10 ++
> > > > > drivers/gpu/drm/bridge/Makefile | 1 +
> > > > > drivers/gpu/drm/bridge/th1520-dw-hdmi.c | 173
> > > > > ++++++++++++++++++++++++
> > > > > 4 files changed, 185 insertions(+)
> > > > > create mode 100644 drivers/gpu/drm/bridge/th1520-dw-hdmi.c
> > > >
> > > >
> > > > As this is a Synopsys DesignWare based IP, maybe it's better to
> > > > put
> > > > it under "drivers/gpu/drm/bridge/synopsys/"
> > > > Or just create a dir for thead, I think there will come other
> > > > display
> > > > related drivers for thead in the future.
> > >
> > > It's not proper to place vendor glues to synopsys/ .
> >
> >
> > TI has precedents for doing this with Cadence IP, see
> >
> > drivers/gpu/drm/bridge/cadence/cdns-dsi-j721e.c
> > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c
>
> Well for this kind of things, I prefer to hear from the maintainer of
> DW HDMI bridges (although it seems that this has no dedicated
> MAINTAINERS entry and get_maintainers just returns people for DRM
> DRIVERS FOR BRIDGE CHIPS).
I think the best way to resolve this question would be to actively ask
on the dri-devel IRC channel (on the OFTC network), because people
might not even realize this quite interesting question is in here, but
I guess most of the relevant people are on there.
But probably wait after new years, so people can actually return from
all the festivities happening right now.
Heiko
Powered by blists - more mailing lists