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 C480CC3ABC3 for ; Fri, 9 May 2025 14:22:00 +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:References:To:From:Subject:Cc:Message-Id:Date:Mime-Version: 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=z9123sDSMRghCWSK8nEdZrDCYl94DmP97JK7vkT/a4Y=; b=O7dE3hIf5jFkecIrnuI0Rhdwuv Ou9D1qd0fN9kFBJdR4IEVSjiLYg+/1Bizk+wDvhD9aSxoBWAkydxUplokBg4SCyMGMAoZWPacPF8h HgC8WIcoMs4eHqwlxBfdXqBivNUNE9w13OR5xFQRUIDmoM4cn3ofKphEdMsBOgE4jJQzdoA/e1+kW PvadTVUBCyCOXwAKKNCTx0rQo+Kalzi7Ev0LFF0SojIyAfg76gFfSMYD0SCWEc7TMURfowyfLwNee DzmStUxzoR3GXMzm6CKYcBk5JVkC106r1/mKGMXpDFax+Y+ARb4cpd++llvORcjDwpO4cyj00s2nW jbH3/kyA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uDObr-00000003vxh-1kPM; Fri, 09 May 2025 14:21:55 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uDNZJ-00000003iZq-2Se4 for linux-rockchip@bombadil.infradead.org; Fri, 09 May 2025 13:15:13 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:References:To:From:Subject :Cc:Message-Id:Date:Content-Type:Mime-Version:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=5axwq5ukixnZASkqSTZMmJAzeb3lPrAspoP9HlhWjSw=; b=Qrcq3YcsZ8qgLPhCVqr0GlI6T7 0XVUnJNcEwec5bWdryoXAXOV8S0aUzFE7w6c4U9kD2WzP+NHlgJGuxl/PRK/mSz461UNi+icvAZew kdyMh4XuFn5MVBdg7DVtemX28rG5WFMYIuB1EVrbHkIWYEqnvVh2U+iB1SouC8cEwhoyKLovIv7FP eCwYMS1Zi3G/ADzGF7b2BMTftUwJTuUYHg40Ex0FqjDMwq9U1xmi90GXcSvbrN5PgdV1NdSoiRZW+ fUYQ8bk87MczWXKxXNh0uHLuS20hADqGJ4Dboz82f9GXAL7Hr4aLSBGaBhrVUTlbHCngiOVUZXQ+a FU75GIUQ==; Received: from out-173.mta1.migadu.com ([95.215.58.173]) by desiato.infradead.org with esmtps (Exim 4.98.1 #2 (Red Hat Linux)) id 1uDNZG-0000000GHXv-19Iu for linux-rockchip@lists.infradead.org; Fri, 09 May 2025 13:15:12 +0000 Mime-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cknow.org; s=key1; t=1746796490; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=5axwq5ukixnZASkqSTZMmJAzeb3lPrAspoP9HlhWjSw=; b=mDeG47eJZKpstpn0HYUuRIY91JnYd5NZlJsQdZU8dfctwKLiqcdr1UBqGKUi/8MKZ6TH6i HC7cPnMlBRYNcejsm5W7v6RQAjDnPvVGdMg8rzgAYIL9s0evaEgaySXv2reXFhdgv7AjBB odwBm9ZrQ+OQBwOnCtRgXKjZydn2zFW7HbT2G/f7pfyQtLc/Qik2J6QeXpRSdMU1vyk6Gw qJHQ/2dvYh2SSrEZXRmJG3mhf/ypiDAHfB6aRsrbf/KvS0129LJMqhXMbWO3doGZ5CzJvf csT4PkppY88Y6LhsP5xz0mij8XoGukg4vAK3JL86bQEwf1pIIukusDDAtukryA== Date: Fri, 09 May 2025 15:14:31 +0200 Message-Id: Cc: "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , , , , , Subject: Re: [PATCH v2 2/5] arm64: dts: rockchip: move rock 5b to include file X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "Diederik de Haas" To: "Sebastian Reichel" , =?utf-8?q?Heiko_St=C3=BCbner?= References: <20250508-rock5bp-for-upstream-v2-0-677033cc1ac2@kernel.org> <20250508-rock5bp-for-upstream-v2-2-677033cc1ac2@kernel.org> <2653568.Lt9SDvczpP@diego> In-Reply-To: X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250509_141510_402225_5F26801A X-CRM114-Status: GOOD ( 33.28 ) 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: , Content-Type: multipart/mixed; boundary="===============1793171661575825185==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============1793171661575825185== Content-Type: multipart/signed; boundary=6655860ab50d87daa0204e489dd2504ca4fc9154ff9510bb077e41da56ae; micalg=pgp-sha512; protocol="application/pgp-signature" --6655860ab50d87daa0204e489dd2504ca4fc9154ff9510bb077e41da56ae Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Hi, On Fri May 9, 2025 at 3:08 PM CEST, Sebastian Reichel wrote: > On Fri, May 09, 2025 at 02:54:00PM +0200, Heiko St=C3=BCbner wrote: >> Am Freitag, 9. Mai 2025, 14:44:57 Mitteleurop=C3=A4ische Sommerzeit schr= ieb Diederik de Haas: >> > On Thu May 8, 2025 at 7:48 PM CEST, Sebastian Reichel wrote: >> > > Radxa released some more boards, which are based on the original >> > > Rock 5B. Move its board description into an include file to avoid >> > > unnecessary duplication. >> >=20 >> > Aren't you moving it *out of* an/the include file? >> > If so, the patch Subject and the above line should be updated so that >> > they correctly reflect what is changed in this patch. >> >=20 >> > The above text is correct (and the same ...) as patch 1, but in this >> > patch you move things out of the dtsi which are unique per board. >> >=20 >> > > NOTE: this should be merged with the previous commit to ensure >> > > bisectability. The rename happens in a separete commit during >> > > development because git does not properly detect the rename when >> > > the original filename is reused in the same commit. This means >> > > >> > > 1. it's a lot harder to review the changes >> > > 2. it's a lot harder to rebase the patch series >> >=20 >> > Or did I fall prey to the exact thing you described here? >>=20 >> I think Sebastian's idea is, that I squash both patches when applying. >> This split makes it easy(er) to review because patch1 is just a rename. >>=20 >> And merging them when applying then makes it again not break bisectabili= ty. > > Correct. This is a lot easier to review than what git generates when > having these two patches squashed together, which is a huge diff of > all 1000+ lines in the file (I tried really hard to convince it that > this is mostly a rename with --find-renames and --find-copies). You > can see this kind of mess in patch 2 of the ROCK 5T series that > Nicolas just send (I will comment on that and suggest to do the same > thing I did to ease review. In his case it should even be possible > to do it in a bisectable way without needing a squash :)). Thank you both for the clarification :-) Cheers, Diederik >> > > Signed-off-by: Sebastian Reichel >> > > --- >> > > arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 52 +++++++++++++= +++++++++++ >> > > arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dtsi | 40 -------------= ----- >> > > 2 files changed, 52 insertions(+), 40 deletions(-) >> > > >> > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/= arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> > > new file mode 100644 >> > > index 0000000000000000000000000000000000000000..9407a7c9910ada1f6c80= 3d2e15785a9cbd9bd655 >> > > --- /dev/null >> > > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> > > @@ -0,0 +1,52 @@ >> > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) >> > > + >> > > +/dts-v1/; >> > > + >> > > +#include "rk3588-rock-5b.dtsi" >> > > + >> > > +/ { >> > > + model =3D "Radxa ROCK 5B"; >> > > + compatible =3D "radxa,rock-5b", "rockchip,rk3588"; >> > > +}; >> > > + >> > > +&sdio { >> > > + max-frequency =3D <200000000>; >> > > + no-sd; >> > > + no-mmc; >> > > + non-removable; >> > > + bus-width =3D <4>; >> > > + cap-sdio-irq; >> > > + disable-wp; >> > > + keep-power-in-suspend; >> > > + wakeup-source; >> > > + sd-uhs-sdr12; >> > > + sd-uhs-sdr25; >> > > + sd-uhs-sdr50; >> > > + sd-uhs-sdr104; >> > > + vmmc-supply =3D <&vcc3v3_pcie2x1l0>; >> > > + vqmmc-supply =3D <&vcc_1v8_s3>; >> > > + pinctrl-names =3D "default"; >> > > + pinctrl-0 =3D <&sdiom0_pins>; >> > > + status =3D "okay"; >> > > +}; >> > > + >> > > +&uart6 { >> > > + pinctrl-names =3D "default"; >> > > + pinctrl-0 =3D <&uart6m1_xfer &uart6m1_ctsn &uart6m1_rtsn>; >> > > + status =3D "okay"; >> > > +}; >> > > + >> > > +&pinctrl { >> > > + usb { >> > > + vcc5v0_host_en: vcc5v0-host-en { >> > > + rockchip,pins =3D <4 RK_PB0 RK_FUNC_GPIO &pcfg_pull_none>; >> > > + }; >> > > + }; >> > > +}; >> > > + >> > > +&vcc5v0_host { >> > > + enable-active-high; >> > > + gpio =3D <&gpio4 RK_PB0 GPIO_ACTIVE_HIGH>; >> > > + pinctrl-names =3D "default"; >> > > + pinctrl-0 =3D <&vcc5v0_host_en>; >> > > +}; >> > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dtsi b/arch= /arm64/boot/dts/rockchip/rk3588-rock-5b.dtsi >> > > index 17f4fd054cd3d1c4e23ccfe014a9c4b9d7ad1a06..6052787d2560978d2bae= 6cfbeea5fc1d419d583a 100644 >> > > --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dtsi >> > > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dtsi >> > > @@ -8,9 +8,6 @@ >> > > #include "rk3588.dtsi" >> > > =20 >> > > / { >> > > - model =3D "Radxa ROCK 5B"; >> > > - compatible =3D "radxa,rock-5b", "rockchip,rk3588"; >> > > - >> > > aliases { >> > > mmc0 =3D &sdhci; >> > > mmc1 =3D &sdmmc; >> > > @@ -139,10 +136,6 @@ vcc5v0_host: regulator-vcc5v0-host { >> > > regulator-always-on; >> > > regulator-min-microvolt =3D <5000000>; >> > > regulator-max-microvolt =3D <5000000>; >> > > - enable-active-high; >> > > - gpio =3D <&gpio4 RK_PB0 GPIO_ACTIVE_HIGH>; >> > > - pinctrl-names =3D "default"; >> > > - pinctrl-0 =3D <&vcc5v0_host_en>; >> > > vin-supply =3D <&vcc5v0_sys>; >> > > }; >> > > =20 >> > > @@ -488,12 +481,6 @@ pcie3_vcc3v3_en: pcie3-vcc3v3-en { >> > > rockchip,pins =3D <1 RK_PA4 RK_FUNC_GPIO &pcfg_pull_none>; >> > > }; >> > > }; >> > > - >> > > - usb { >> > > - vcc5v0_host_en: vcc5v0-host-en { >> > > - rockchip,pins =3D <4 RK_PB0 RK_FUNC_GPIO &pcfg_pull_none>; >> > > - }; >> > > - }; >> > > }; >> > > =20 >> > > &pwm1 { >> > > @@ -530,27 +517,6 @@ &sdmmc { >> > > status =3D "okay"; >> > > }; >> > > =20 >> > > -&sdio { >> > > - max-frequency =3D <200000000>; >> > > - no-sd; >> > > - no-mmc; >> > > - non-removable; >> > > - bus-width =3D <4>; >> > > - cap-sdio-irq; >> > > - disable-wp; >> > > - keep-power-in-suspend; >> > > - wakeup-source; >> > > - sd-uhs-sdr12; >> > > - sd-uhs-sdr25; >> > > - sd-uhs-sdr50; >> > > - sd-uhs-sdr104; >> > > - vmmc-supply =3D <&vcc3v3_pcie2x1l0>; >> > > - vqmmc-supply =3D <&vcc_1v8_s3>; >> > > - pinctrl-names =3D "default"; >> > > - pinctrl-0 =3D <&sdiom0_pins>; >> > > - status =3D "okay"; >> > > -}; >> > > - >> > > &sfc { >> > > pinctrl-names =3D "default"; >> > > pinctrl-0 =3D <&fspim2_pins>; >> > > @@ -566,12 +532,6 @@ flash@0 { >> > > }; >> > > }; >> > > =20 >> > > -&uart6 { >> > > - pinctrl-names =3D "default"; >> > > - pinctrl-0 =3D <&uart6m1_xfer &uart6m1_ctsn &uart6m1_rtsn>; >> > > - status =3D "okay"; >> > > -}; >> > > - >> > > &spi2 { >> > > status =3D "okay"; >> > > assigned-clocks =3D <&cru CLK_SPI2>; >> >=20 >> >=20 >>=20 >>=20 >>=20 >>=20 --6655860ab50d87daa0204e489dd2504ca4fc9154ff9510bb077e41da56ae Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQT1sUPBYsyGmi4usy/XblvOeH7bbgUCaB3/wwAKCRDXblvOeH7b biJWAPwP5+nFvMYlxo+Uf4SxONp4EbIR9ujXYtqplfzzKyENUAD/R5g1zPNEXBx4 wQAeV1eNLGfh/kvi5dsnxCAolMKa5Qs= =Gk0H -----END PGP SIGNATURE----- --6655860ab50d87daa0204e489dd2504ca4fc9154ff9510bb077e41da56ae-- --===============1793171661575825185== 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 --===============1793171661575825185==--