public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
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 --]

  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