Message-ID: <3b5fd30554f91bcd083b2dc450cdf7fe2fa395e2.camel@iscas.ac.cn>
Date: Sat, 27 Dec 2025 15:22:05 +0800
From: Icenowy Zheng <zhengxingda@...as.ac.cn>
To: Andy Yan <andyshrk@....com>
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>, Heiko Stuebner <heiko@...ech.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: Re: [PATCH v4 5/9] drm/bridge: add a driver for T-Head TH1520
 HDMI controller

在 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).

> 
> 
> > 
> > As for T-Head, they stopped their embedded SoC business (sold to
> > another company) and now they only makes data-center chips, there
> > should be no T-Head display drivers in the future.
> > 
> > > 
> > > 
> > =========== 8< ============
> > 
> > 
> > _______________________________________________
> > linux-riscv mailing list
> > linux-riscv@...ts.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-riscv

Powered by blists - more mailing lists