From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4CA4BCD98D0 for ; Thu, 13 Nov 2025 19:45:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=7nwPC53c6NuGA7opVopOAARRYAre4g7MgQ3+2yMO2Ag=; b=K9nUFplD9zq+KGJKAlXk/Yu51f V54mKOAFigplEb3V6XbuWVC2w3zcseast2b30v2UKPI9ZjbNSVV1ha0sQmP0RU3GI3U5YOpSoI86o I7BgBoPeGy35A+QCab5zSslhj2pTsEIqOwmWeNyjmG2/US+Z6n0kIct0nJ6vdQkoYwTmjoxhfF3lR vFnRCDQLZ3weG/hSAArP9sFM5jIRkPkC+Hw39QBl5IIyuybH/+lczUTCYECG2sx8gjyz0Enw5Pnk0 q/vr1tLQg1tRpGPrnZoYTdY4Mn0QX9GhmrL08gzt8EkMuo1LY1qa5feq/SvSyHkckKhGrNnuKWypA 8YC8WGuA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vJdFg-0000000B2Au-0FOF; Thu, 13 Nov 2025 19:45:04 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vJdFe-0000000B2Ah-1cro; Thu, 13 Nov 2025 19:45:02 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id BE38B600CB; Thu, 13 Nov 2025 19:45:01 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 70FE0C4CEF1; Thu, 13 Nov 2025 19:44:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763063101; bh=0VwKaYc35bZVUgu90hr/jsC2a+mJC8giJD4+WvmKtmE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=CYdJIPPSOEOvnf/bgsK11R2E89RQ7+vEc5reOI2FhvVe9UVPfAItJfBhMQSYjoGv0 TNXFpYwpGAm3kQazKFBeEFhmJBXOw2jeikG1SNneO4gp0FOhTlV3pYjUDrryqGGvVW cF5VIdqba1jBzZ6YvMAoSjLCEVCBO8pk8A10cnNxpONJ1ru5wnLwbGyKKF/S1cYyFK kyt0oasNjxwpqKj75d72zVvVJgRpLLL++cfso3ieHF0qj4Hasq9dUqbELt+sp1roY3 W1mhVVQ/6D80E7ChaldiPuY4sQVYEzinVKabCn9zyBlXxO0pqxcAgaPojRyQZWFrVk 9FNGg4ljOwW3w== Date: Thu, 13 Nov 2025 19:44:52 +0000 From: Conor Dooley To: Icenowy Zheng Cc: Michal Wilczynski , Rob Herring , Krzysztof Kozlowski , Emil Renner Berthing , Hal Feng , Michael Turquette , Stephen Boyd , Conor Dooley , Xingyu Wu , Vinod Koul , Kishon Vijay Abraham I , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Lee Jones , Philipp Zabel , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Marek Szyprowski , Maud Spierings , Andy Yan , Heiko Stuebner , 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 01/13] dt-bindings: soc: starfive: Add vout-subsystem IP block Message-ID: <20251113-irritable-unfold-da619d52026c@spud> References: <20251108-jh7110-clean-send-v1-0-06bf43bb76b1@samsung.com> <20251108-jh7110-clean-send-v1-1-06bf43bb76b1@samsung.com> <20251111-massager-twistable-1e88f03d82f8@spud> <20251111-unsaid-rockslide-67b88b2e34bd@spud> <0d8e3a626b037dd348378e5ebca8005c1e715871.camel@icenowy.me> <20251112-skating-robust-81be8dee0a8b@spud> <1d54524d7ef939232b4512ca59c78794484103ca.camel@icenowy.me> MIME-Version: 1.0 In-Reply-To: <1d54524d7ef939232b4512ca59c78794484103ca.camel@icenowy.me> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============0186208498609741836==" Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org --===============0186208498609741836== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="ts3gmjGSebbDWFBm" Content-Disposition: inline --ts3gmjGSebbDWFBm Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 13, 2025 at 08:48:33AM +0800, Icenowy Zheng wrote: > =E5=9C=A8 2025-11-12=E6=98=9F=E6=9C=9F=E4=B8=89=E7=9A=84 18:36 +0000=EF= =BC=8CConor Dooley=E5=86=99=E9=81=93=EF=BC=9A > > On Wed, Nov 12, 2025 at 02:34:39PM +0800, Icenowy Zheng wrote: > > > =E5=9C=A8 2025-11-11=E6=98=9F=E6=9C=9F=E4=BA=8C=E7=9A=84 18:36 +0000= =EF=BC=8CConor Dooley=E5=86=99=E9=81=93=EF=BC=9A > > > > On Tue, Nov 11, 2025 at 06:18:16PM +0000, Conor Dooley wrote: > > > > > On Sat, Nov 08, 2025 at 02:04:35AM +0100, Michal Wilczynski > > > > > wrote: > > > > > > Add the dt-binding documentation for the StarFive JH7110 > > > > > > Video > >=20 > > > > > > +patternProperties: > > > > > > +=C2=A0 "^display@[0-9a-f]+$": > > > > >=20 > > > > > Personally I'd like to see these being regular properties, > > > > > since > > > > > there's > > > > > exactly one possible setup for this. > > > > >=20 > > > > > > +=C2=A0=C2=A0=C2=A0 type: object > > > > > > +=C2=A0=C2=A0=C2=A0 description: Verisilicon DC8200 Display Con= troller node. > > > > >=20 > > > > > Can you add the relevant references here instead of allowing > > > > > any > > > > > object? > > > >=20 > > > > I don't think that if you did, this would pass the binding > > > > checks, > > > > because there's no "verisilicon,dc" binding. I think I saw one in > > > > progress, but without the soc-specific compatible that I am going > > > > to > > > > require here - if for no reason other than making sure that the > > > > clocks > > > > etc are provided correctly for this device. > > >=20 > > > Well I didn't specify any soc-specific compatible because that IP > > > has > > > its own identification registers. > >=20 > > I still require one because I want to make sure that clocks etc are > > handled correctly. You can ignore it in the driver if you wish, but > > when > > the next user comes along with one more or less clock, I want the > > jh7110 one to be forced to use the correct configuration. >=20 > I don't think for those generic IPs requiring a SoC-specific compatible > is a good idea. I disagree. If things are complex enough to end up with different numbers of clocks or power-domains etc on different platforms (which I believe GPUs are) then I want one for validation purposes on platforms I care about. What you do in the driver is up to you. --ts3gmjGSebbDWFBm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCaRY1NAAKCRB4tDGHoIJi 0k9zAQDvbUM7s4AEGxVxu+wdxmQIgr0MmVDfS6YxVkmZ+4CJIAEAiwMj6RElCUx6 g7Zzf1f3tUc15pHEt8NIVOAt/SqlRQc= =tccX -----END PGP SIGNATURE----- --ts3gmjGSebbDWFBm-- --===============0186208498609741836== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy --===============0186208498609741836==--