[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2653568.Lt9SDvczpP@diego>
Date: Fri, 09 May 2025 14:54:00 +0200
From: Heiko StĂĽbner <heiko@...ech.de>
To: Sebastian Reichel <sebastian.reichel@...labora.com>,
Diederik de Haas <didi.debian@...ow.org>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
linux-kernel@...r.kernel.org, kernel@...labora.com
Subject: Re: [PATCH v2 2/5] arm64: dts: rockchip: move rock 5b to include file
Am Freitag, 9. Mai 2025, 14:44:57 Mitteleuropäische Sommerzeit schrieb Diederik de Haas:
> Hi,
>
> 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.
>
> 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.
>
> 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.
>
> > 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
>
> Or did I fall prey to the exact thing you described here?
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.
And merging them when applying then makes it again not break bisectability.
Heiko
> Cheers,
> Diederik
>
> > Signed-off-by: Sebastian Reichel <sebastian.reichel@...labora.com>
> > ---
> > 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..9407a7c9910ada1f6c803d2e15785a9cbd9bd655
> > --- /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 = "Radxa ROCK 5B";
> > + compatible = "radxa,rock-5b", "rockchip,rk3588";
> > +};
> > +
> > +&sdio {
> > + max-frequency = <200000000>;
> > + no-sd;
> > + no-mmc;
> > + non-removable;
> > + bus-width = <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 = <&vcc3v3_pcie2x1l0>;
> > + vqmmc-supply = <&vcc_1v8_s3>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&sdiom0_pins>;
> > + status = "okay";
> > +};
> > +
> > +&uart6 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&uart6m1_xfer &uart6m1_ctsn &uart6m1_rtsn>;
> > + status = "okay";
> > +};
> > +
> > +&pinctrl {
> > + usb {
> > + vcc5v0_host_en: vcc5v0-host-en {
> > + rockchip,pins = <4 RK_PB0 RK_FUNC_GPIO &pcfg_pull_none>;
> > + };
> > + };
> > +};
> > +
> > +&vcc5v0_host {
> > + enable-active-high;
> > + gpio = <&gpio4 RK_PB0 GPIO_ACTIVE_HIGH>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&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..6052787d2560978d2bae6cfbeea5fc1d419d583a 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"
> >
> > / {
> > - model = "Radxa ROCK 5B";
> > - compatible = "radxa,rock-5b", "rockchip,rk3588";
> > -
> > aliases {
> > mmc0 = &sdhci;
> > mmc1 = &sdmmc;
> > @@ -139,10 +136,6 @@ vcc5v0_host: regulator-vcc5v0-host {
> > regulator-always-on;
> > regulator-min-microvolt = <5000000>;
> > regulator-max-microvolt = <5000000>;
> > - enable-active-high;
> > - gpio = <&gpio4 RK_PB0 GPIO_ACTIVE_HIGH>;
> > - pinctrl-names = "default";
> > - pinctrl-0 = <&vcc5v0_host_en>;
> > vin-supply = <&vcc5v0_sys>;
> > };
> >
> > @@ -488,12 +481,6 @@ pcie3_vcc3v3_en: pcie3-vcc3v3-en {
> > rockchip,pins = <1 RK_PA4 RK_FUNC_GPIO &pcfg_pull_none>;
> > };
> > };
> > -
> > - usb {
> > - vcc5v0_host_en: vcc5v0-host-en {
> > - rockchip,pins = <4 RK_PB0 RK_FUNC_GPIO &pcfg_pull_none>;
> > - };
> > - };
> > };
> >
> > &pwm1 {
> > @@ -530,27 +517,6 @@ &sdmmc {
> > status = "okay";
> > };
> >
> > -&sdio {
> > - max-frequency = <200000000>;
> > - no-sd;
> > - no-mmc;
> > - non-removable;
> > - bus-width = <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 = <&vcc3v3_pcie2x1l0>;
> > - vqmmc-supply = <&vcc_1v8_s3>;
> > - pinctrl-names = "default";
> > - pinctrl-0 = <&sdiom0_pins>;
> > - status = "okay";
> > -};
> > -
> > &sfc {
> > pinctrl-names = "default";
> > pinctrl-0 = <&fspim2_pins>;
> > @@ -566,12 +532,6 @@ flash@0 {
> > };
> > };
> >
> > -&uart6 {
> > - pinctrl-names = "default";
> > - pinctrl-0 = <&uart6m1_xfer &uart6m1_ctsn &uart6m1_rtsn>;
> > - status = "okay";
> > -};
> > -
> > &spi2 {
> > status = "okay";
> > assigned-clocks = <&cru CLK_SPI2>;
>
>
Powered by blists - more mailing lists