From: Conor Dooley <conor@kernel.org>
To: Michal Wilczynski <m.wilczynski@samsung.com>
Cc: Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Emil Renner Berthing <kernel@esmil.dk>,
Hal Feng <hal.feng@starfivetech.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Xingyu Wu <xingyu.wu@starfivetech.com>,
Vinod Koul <vkoul@kernel.org>,
Kishon Vijay Abraham I <kishon@kernel.org>,
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>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
Lee Jones <lee@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Paul Walmsley <paul.walmsley@sifive.com>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>,
Marek Szyprowski <m.szyprowski@samsung.com>,
Icenowy Zheng <uwu@icenowy.me>,
Maud Spierings <maudspierings@gocontroll.com>,
Andy Yan <andyshrk@163.com>, Heiko Stuebner <heiko@sntech.de>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org, linux-phy@lists.infradead.org,
dri-devel@lists.freedesktop.org, linux-riscv@lists.infradead.org
Subject: Re: [PATCH RFC 00/13] drm: starfive: jh7110: Enable display subsystem
Date: Tue, 11 Nov 2025 18:14:48 +0000 [thread overview]
Message-ID: <20251111-footing-eclair-332f5f0769f2@spud> (raw)
In-Reply-To: <00e897dc-9966-439b-a74a-7604a1870027@samsung.com>
[-- Attachment #1: Type: text/plain, Size: 2185 bytes --]
On Tue, Nov 11, 2025 at 04:33:28PM +0100, Michal Wilczynski wrote:
>
>
> On 11/10/25 20:35, Conor Dooley wrote:
> > On Sat, Nov 08, 2025 at 02:04:34AM +0100, Michal Wilczynski wrote:
> >> This series enables the display subsystem on the StarFive JH7110 SoC.
> >> This hardware has a complex set of dependencies that this series aims to
> >> solve.
> >>
> >> I believe this is a PHY tuning issue that can be fixed in the new
> >> phy-jh7110-inno-hdmi.c driver without changing the overall architecture.
> >> I plan to continue debugging these modes and will submit follow up fixes
> >> as needed.
> >>
> >> The core architectural plumbing is sound and ready for review.
> >>
> >> Notes:
> >> - The JH7110 does not have a centralized MAINTAINERS entry like the
> >> TH1520, and driver maintainership seems fragmented. I have therefore
> >> added a MAINTAINERS entry for the display subsystem and am willing to
> >> help with its maintenance.
> >
> > Yeah, bunch of different folks wrote the drivers, so lots of entries.
> > Pretty much all as you've done here, authors are responsible for the
> > individual components and Emil is the platform maintainer but
> > responsible for most drivers.
> >
> > Do you need any feedback dt wise on the RFC, or is it too likely that
> > we'll both waste our breath if the DRM folks don't approve of your
> > approach for the rest of this series?
>
> Hi Conor,
>
> Thank you for your response.
>
> That's a fair point about the risk of the DRM approach being rejected.
> While I can't be certain, I'm hopeful that part is relatively
> straightforward, as it primarily integrates other recently reviewed
> (though not yet merged) components like the inno-hdmi bridge and dc8200
> drivers.
>
> To be honest, I was more concerned that the DT part of the series would
> be more problematic. Given that, I would find it very helpful to get
> your feedback on the DT aspects now, if you have the time.
Right. You'll definitely want some actual DRM people to weigh in though
before making changes, I am really not familiar enough with this type of
hardware to know if the breakdown is correct.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2025-11-11 18:14 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20251108010451eucas1p1c7bf340dbd2b1b7cbfb53d6debce7a2e@eucas1p1.samsung.com>
2025-11-08 1:04 ` [PATCH RFC 00/13] drm: starfive: jh7110: Enable display subsystem Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 01/13] dt-bindings: soc: starfive: Add vout-subsystem IP block Michal Wilczynski
2025-11-11 18:18 ` Conor Dooley
2025-11-11 18:36 ` Conor Dooley
2025-11-12 6:34 ` Icenowy Zheng
2025-11-12 18:36 ` Conor Dooley
2025-11-13 0:48 ` Icenowy Zheng
2025-11-13 19:44 ` Conor Dooley
2025-11-14 7:06 ` Icenowy Zheng
2025-11-08 1:04 ` [PATCH RFC 02/13] dt-bindings: clock: jh7110: Make power-domain optional Michal Wilczynski
2025-11-11 18:26 ` Conor Dooley
2025-11-08 1:04 ` [PATCH RFC 03/13] dt-bindings: phy: Add starfive,jh7110-inno-hdmi-phy Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 04/13] dt-bindings: display: bridge: Add starfive,jh7110-hdmi-controller Michal Wilczynski
2025-11-11 18:23 ` Conor Dooley
2025-11-08 1:04 ` [PATCH RFC 05/13] dt-bindings: mfd: Add starfive,jh7110-hdmi-mfd Michal Wilczynski
2025-11-11 18:29 ` Conor Dooley
2025-11-08 1:04 ` [PATCH RFC 06/13] drm: bridge: inno_hdmi: Refactor to support regmap and probe Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 07/13] drm: bridge: inno_hdmi: Add .disable platform operation Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 08/13] soc: starfive: Add jh7110-vout-subsystem driver Michal Wilczynski
2025-11-10 19:25 ` Conor Dooley
2025-11-08 1:04 ` [PATCH RFC 09/13] soc: starfive: Add jh7110-hdmi-mfd driver Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 10/13] clk: starfive: voutcrg: Update the voutcrg Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 11/13] drm: bridge: starfive: Add hdmi-controller driver Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 12/13] phy: starfive: Add jh7110-inno-hdmi-phy driver Michal Wilczynski
2025-11-08 1:04 ` [PATCH RFC 13/13] riscv: dts: starfive: jh7110: Update DT for display subsystem Michal Wilczynski
2025-11-11 16:07 ` Icenowy Zheng
2025-11-10 19:35 ` [PATCH RFC 00/13] drm: starfive: jh7110: Enable " Conor Dooley
2025-11-11 15:33 ` Michal Wilczynski
2025-11-11 18:14 ` Conor Dooley [this message]
2025-11-11 18:37 ` Conor Dooley
2025-11-13 14:57 ` Michal Wilczynski
2025-11-22 11:42 ` Michal Wilczynski
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=20251111-footing-eclair-332f5f0769f2@spud \
--to=conor@kernel.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=alex@ghiti.fr \
--cc=andrzej.hajda@intel.com \
--cc=andyshrk@163.com \
--cc=aou@eecs.berkeley.edu \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=hal.feng@starfivetech.com \
--cc=heiko@sntech.de \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=kernel@esmil.dk \
--cc=kishon@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-riscv@lists.infradead.org \
--cc=m.szyprowski@samsung.com \
--cc=m.wilczynski@samsung.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=maudspierings@gocontroll.com \
--cc=mripard@kernel.org \
--cc=mturquette@baylibre.com \
--cc=neil.armstrong@linaro.org \
--cc=p.zabel@pengutronix.de \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=rfoss@kernel.org \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
--cc=uwu@icenowy.me \
--cc=vkoul@kernel.org \
--cc=xingyu.wu@starfivetech.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox