Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 12 Dec 2022 16:47:10 +0300
From:   Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
CC:     linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v2_1/6=5D_arm64=3A_dts=3A_qcom=3A_sc7?= =?US-ASCII?Q?180=3A_order_top-level_nodes_alphabetically?=
User-Agent: K-9 Mail for Android
In-Reply-To: <20221212100232.138519-1-krzysztof.kozlowski@linaro.org>
References: <20221212100232.138519-1-krzysztof.kozlowski@linaro.org>
Message-ID: <725A4567-0024-451A-99F1-30E6FED04E5B@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk


On 12 December 2022 13:02:27 GMT+03:00, Krzysztof Kozlowski <krzysztof=2Ek=
ozlowski@linaro=2Eorg> wrote:
>Order top-level nodes like memory, reserved-memory, opp-table-cpu
>alphabetically for easier code maintenance=2E  No functional change (same
>dtx_diff)=2E
>
>Signed-off-by: Krzysztof Kozlowski <krzysztof=2Ekozlowski@linaro=2Eorg>

Reviewed-by: Dmitry Baryshkov<dmitry=2Ebaryshkov@linaro=2Eorg>

>
>---
>
>Changes since v1:
>1=2E New patch
>---
> arch/arm64/boot/dts/qcom/sc7180=2Edtsi | 144 +++++++++++++--------------
> 1 file changed, 72 insertions(+), 72 deletions(-)
>
>diff --git a/arch/arm64/boot/dts/qcom/sc7180=2Edtsi b/arch/arm64/boot/dts=
/qcom/sc7180=2Edtsi
>index ea886cf08b4d=2E=2E6d3e86ce2936 100644
>--- a/arch/arm64/boot/dts/qcom/sc7180=2Edtsi
>+++ b/arch/arm64/boot/dts/qcom/sc7180=2Edtsi
>@@ -27,8 +27,6 @@ / {
> 	#address-cells =3D <2>;
> 	#size-cells =3D <2>;
>=20
>-	chosen { };
>-
> 	aliases {
> 		mmc1 =3D &sdhc_1;
> 		mmc2 =3D &sdhc_2;
>@@ -54,6 +52,8 @@ aliases {
> 		spi11 =3D &spi11;
> 	};
>=20
>+	chosen { };
>+
> 	clocks {
> 		xo_board: xo-board {
> 			compatible =3D "fixed-clock";
>@@ -68,62 +68,6 @@ sleep_clk: sleep-clk {
> 		};
> 	};
>=20
>-	reserved_memory: reserved-memory {
>-		#address-cells =3D <2>;
>-		#size-cells =3D <2>;
>-		ranges;
>-
>-		hyp_mem: memory@80000000 {
>-			reg =3D <0x0 0x80000000 0x0 0x600000>;
>-			no-map;
>-		};
>-
>-		xbl_mem: memory@80600000 {
>-			reg =3D <0x0 0x80600000 0x0 0x200000>;
>-			no-map;
>-		};
>-
>-		aop_mem: memory@80800000 {
>-			reg =3D <0x0 0x80800000 0x0 0x20000>;
>-			no-map;
>-		};
>-
>-		aop_cmd_db_mem: memory@80820000 {
>-			reg =3D <0x0 0x80820000 0x0 0x20000>;
>-			compatible =3D "qcom,cmd-db";
>-			no-map;
>-		};
>-
>-		sec_apps_mem: memory@808ff000 {
>-			reg =3D <0x0 0x808ff000 0x0 0x1000>;
>-			no-map;
>-		};
>-
>-		smem_mem: memory@80900000 {
>-			reg =3D <0x0 0x80900000 0x0 0x200000>;
>-			no-map;
>-		};
>-
>-		tz_mem: memory@80b00000 {
>-			reg =3D <0x0 0x80b00000 0x0 0x3900000>;
>-			no-map;
>-		};
>-
>-		ipa_fw_mem: memory@8b700000 {
>-			reg =3D <0 0x8b700000 0 0x10000>;
>-			no-map;
>-		};
>-
>-		rmtfs_mem: memory@94600000 {
>-			compatible =3D "qcom,rmtfs-mem";
>-			reg =3D <0x0 0x94600000 0x0 0x200000>;
>-			no-map;
>-
>-			qcom,client-id =3D <1>;
>-			qcom,vmid =3D <15>;
>-		};
>-	};
>-
> 	cpus {
> 		#address-cells =3D <2>;
> 		#size-cells =3D <0>;
>@@ -398,6 +342,18 @@ CLUSTER_SLEEP_0: cluster-sleep-0 {
> 		};
> 	};
>=20
>+	firmware {
>+		scm {
>+			compatible =3D "qcom,scm-sc7180", "qcom,scm";
>+		};
>+	};
>+
>+	memory@80000000 {
>+		device_type =3D "memory";
>+		/* We expect the bootloader to fill in the size */
>+		reg =3D <0 0x80000000 0 0>;
>+	};
>+
> 	cpu0_opp_table: opp-table-cpu0 {
> 		compatible =3D "operating-points-v2";
> 		opp-shared;
>@@ -538,20 +494,69 @@ cpu6_opp16: opp-2553600000 {
> 		};
> 	};
>=20
>-	memory@80000000 {
>-		device_type =3D "memory";
>-		/* We expect the bootloader to fill in the size */
>-		reg =3D <0 0x80000000 0 0>;
>-	};
>-
> 	pmu {
> 		compatible =3D "arm,armv8-pmuv3";
> 		interrupts =3D <GIC_PPI 5 IRQ_TYPE_LEVEL_HIGH>;
> 	};
>=20
>-	firmware {
>-		scm {
>-			compatible =3D "qcom,scm-sc7180", "qcom,scm";
>+	psci {
>+		compatible =3D "arm,psci-1=2E0";
>+		method =3D "smc";
>+	};
>+
>+	reserved_memory: reserved-memory {
>+		#address-cells =3D <2>;
>+		#size-cells =3D <2>;
>+		ranges;
>+
>+		hyp_mem: memory@80000000 {
>+			reg =3D <0x0 0x80000000 0x0 0x600000>;
>+			no-map;
>+		};
>+
>+		xbl_mem: memory@80600000 {
>+			reg =3D <0x0 0x80600000 0x0 0x200000>;
>+			no-map;
>+		};
>+
>+		aop_mem: memory@80800000 {
>+			reg =3D <0x0 0x80800000 0x0 0x20000>;
>+			no-map;
>+		};
>+
>+		aop_cmd_db_mem: memory@80820000 {
>+			reg =3D <0x0 0x80820000 0x0 0x20000>;
>+			compatible =3D "qcom,cmd-db";
>+			no-map;
>+		};
>+
>+		sec_apps_mem: memory@808ff000 {
>+			reg =3D <0x0 0x808ff000 0x0 0x1000>;
>+			no-map;
>+		};
>+
>+		smem_mem: memory@80900000 {
>+			reg =3D <0x0 0x80900000 0x0 0x200000>;
>+			no-map;
>+		};
>+
>+		tz_mem: memory@80b00000 {
>+			reg =3D <0x0 0x80b00000 0x0 0x3900000>;
>+			no-map;
>+		};
>+
>+		ipa_fw_mem: memory@8b700000 {
>+			reg =3D <0 0x8b700000 0 0x10000>;
>+			no-map;
>+		};
>+
>+		rmtfs_mem: memory@94600000 {
>+			compatible =3D "qcom,rmtfs-mem";
>+			reg =3D <0x0 0x94600000 0x0 0x200000>;
>+			no-map;
>+
>+			qcom,client-id =3D <1>;
>+			qcom,vmid =3D <15>;
> 		};
> 	};
>=20
>@@ -640,11 +645,6 @@ ipa_smp2p_in: ipa-modem-to-ap {
> 		};
> 	};
>=20
>-	psci {
>-		compatible =3D "arm,psci-1=2E0";
>-		method =3D "smc";
>-	};
>-
> 	soc: soc@0 {
> 		#address-cells =3D <2>;
> 		#size-cells =3D <2>;

--=20
With best wishes
Dmitry