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 23660C5321D for ; Fri, 23 Aug 2024 16:04:57 +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:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject: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=LiXLrXKncmNXfGaPZXyDA2QvOhA5CtlHasi1n6qAHgo=; b=rATqoShV1w00UY/3z2x1GUosBf TznSjQcIDV82JIXy3jLuzzQhYrNYK4N3SmFiPh2HGQeZfFFhE23pLvl6wLXSUhOCgYZ6ouWMv6kxL pcDuut1m/UadNNwATh2DdsNUKUi/id9hyZumLLNNWBd8gC6ioHcIFOgrHbwnjlzmTryw3pxDqHaBm VXY49JP4NGhmIDL4wVaExCHmMrfJDQ1oVD22T2Q+FHuZdzwaHSUuOMxn3Ov7pzmdGZ5fcprCmLbix 2pygci4+2Etg8aEiIazVk++4JRi3rxBwbkyyUm+5r2HIg8RDZcrTxaz1yliyBFe/+fUd/FUaVb+05 gOBsGV6w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1shWmU-0000000HSLr-0zpc; Fri, 23 Aug 2024 16:04:54 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1shWkU-0000000HRko-1lJC; Fri, 23 Aug 2024 16:02:52 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 1624B6068D; Fri, 23 Aug 2024 16:02:49 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 62512C4AF09; Fri, 23 Aug 2024 16:02:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1724428968; bh=+J5v2a1KYipoTMfE/Qk79+k0+U/RnOiCO1QF6+MROOg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=kSm0hgM6IJekKLoVVE4ON58EmoKIOJoNWoxufF4+bNJCT+g/pUd3wbRhW9P+J3VsW fnGGRjDaXr/tZL+228+l2+0IFrz6qcMxcLs8//zybPvjRTkyK7Bz2vgcR72ZDAV50u CQuV1ZVGFQ6p/rQ4SV7TuntA62tP3i88VsDCslQiBM+5gQRgN/9KUPWVFsbZVRGSLN GsJkCgGYKV4Vrb7i/FIs7cs0Ob2isoRA+fO8AdHDroZTA9YahiaIY5Mdmp73ywISI/ L2UykFD9Yrr1rToETOdzTrEDkUP6B1fGBCMu+N1/L7uhVvLFj6UXpW9t9Iaz8T7Vpl wi50RmyyR6eOw== Date: Fri, 23 Aug 2024 17:02:41 +0100 From: Conor Dooley To: Andy Yan Subject: Re: Re: [PATCH v4 3/4] dt-bindings: display: rockchip: Add schema for RK3588 HDMI TX Controller Message-ID: <20240823-swifter-smugly-c51447514f96@spud> References: <20240819-b4-rk3588-bridge-upstream-v4-0-6417c72a2749@collabora.com> <7E8109D4-A353-4FE3-9152-3C3C6CB7D634@sntech.de> <2085e998-a453-4893-9e80-3be68b0fb13d@collabora.com> <4167579.6PsWsQAL7t@diego> <20240822-pushchair-premises-f4055779216a@spud> <7fc8cbc3-43d0-43d2-9272-350ac556e2b2@collabora.com> <4140b55c.a48.1917cc1095f.Coremail.andyshrk@163.com> MIME-Version: 1.0 In-Reply-To: <4140b55c.a48.1917cc1095f.Coremail.andyshrk@163.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240823_090250_860583_7BD830EB X-CRM114-Status: GOOD ( 28.72 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Heiko =?iso-8859-1?Q?St=FCbner?= , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Laurent Pinchart , Andrzej Hajda , kernel@collabora.com, David Airlie , Rob Herring , Robert Foss , Jernej Skrabec , linux-rockchip@lists.infradead.org, Luis de Arquer , devicetree@vger.kernel.org, Conor Dooley , Daniel Vetter , Jonas Karlman , Sascha Hauer , Alexandre ARNOUD , Maarten Lankhorst , Maxime Ripard , Mark Yao , linux-arm-kernel@lists.infradead.org, Neil Armstrong , Sandy Huang , Thomas Zimmermann , Andy Yan , Krzysztof Kozlowski Content-Type: multipart/mixed; boundary="===============5217919092264239942==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============5217919092264239942== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="tX5i8q7vWJU61EV6" Content-Disposition: inline --tX5i8q7vWJU61EV6 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Aug 23, 2024 at 09:01:51AM +0800, Andy Yan wrote: >=20 > Hi=EF=BC=8C >=20 > =E5=9C=A8 2024-08-22 19:59:43=EF=BC=8C"Cristian Ciocaltea" =E5=86=99=E9=81=93=EF=BC=9A > >On 8/22/24 11:41 AM, Conor Dooley wrote: > >> On Thu, Aug 22, 2024 at 09:01:34AM +0200, Heiko St=C3=BCbner wrote: > >>> @Conor: just for me, did some shift happen in our understanding of dt- > >>> best-practices in terms of syscon via phandle vs. syscon via compatib= le? > >>> > >>> Because Rockchip boards are referencing their GRFs via phandes forever > >>> but similar to the soc vs non-soc node thing, I'd like to stay on top= of > >>> best-practices ;-) > >>=20 > >> If IP blocks, and thus drivers, are going to be reused between devices, > >> using the phandles makes sense given that it is unlikely that syscon > >> nodes can make use of fallback compatibles due to bits within that "gl= ue" > >> changing between devices. It also makes sense when there are multiple > >> instances of an IP on the device, which need to use different syscons. > >> My goal is to ask people why they are using these type of syscons > >> phandle properties, cos often they are not required at all - for examp= le > >> with clocks where you effectively need a whole new driver for every > >> single soc and having a phandle property buys you nothing. > > > >That would be also the case for this HDMI controller - need to check the > >specs for the newer RK3576 SoC, but I expect the syscons would be quite > >different when compared to RK3588, hence we should keep making use of > >the phandles. >=20 >=20 > Yes=EF=BC=8Cfor rk3576=EF=BC=8Cit shares the same HDMI IP block=EF=BC=88h= dmi controller and PHY=EF=BC=89=EF=BC=8C > of course reuse the driver of rk3588=EF=BC=8C but it has different GRF to= depends on[0]: > which calls ioc_grf and vo0_grf: >=20 > I also believe that makeing use of phandle beneficial for different devic= es to reuse the same code. >=20 > hdmi: hdmi@27da0000 { > compatible =3D "rockchip,rk3576-dw-hdmi"; > reg =3D <0x0 0x27da0000 0x0 0x10000>, <0x0 0x27db0000 0x0= 0x10000>; > interrupts =3D , > , > , > , > ; > , rockchip,grf =3D <&ioc_grf>; > rockchip,vo0_grf =3D <&vo0_grf>; btw, I don't particularly like this naming - on another soc in the future "vo0_grf" could be "vo1_grf". It is better to name them after what they are providing to the hdmi controller, rather than what the grf itself is called. That said, if the grf is changing between socs, the offset within the grf and what it provides to the hdmi controller may vary completely, which makes having generic grf reference properties redundant. > phys =3D <&hdptxphy_hdmi>; > phy-names =3D "hdmi"; >=20 >=20 > [0]https://github.com/armbian/linux-rockchip/blob/rk-6.1-rkr3/arch/arm64/= boot/dts/rockchip/rk3576.dtsi#L3122C2-L3123C33 >=20 > > > >_______________________________________________ > >Linux-rockchip mailing list > >Linux-rockchip@lists.infradead.org > >http://lists.infradead.org/mailman/listinfo/linux-rockchip --tX5i8q7vWJU61EV6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZsiyoQAKCRB4tDGHoIJi 0oGzAP41PotnHGP7pOq28w8lA6nlhNpPPnf7XLGOSlUCEoZ4HwEAp3PJGursxfJy RIkB52Ug7Xev2iSA98QEF4jPYJGtKw4= =eT1G -----END PGP SIGNATURE----- --tX5i8q7vWJU61EV6-- --===============5217919092264239942== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip --===============5217919092264239942==--