arm64: dts: zena: Refactor Devicetree nodes

[PATCH] arm64: dts: zena: Refactor Devicetree nodes

Posted by Debbie Horsfall 3 weeks, 6 days ago

Move the SRAM node into the SoC node. Move the memory node out of
the include to make it customizable for each platform variant.

Signed-off-by: Debbie Horsfall <debbie.horsfall@arm.com>
---
This targets for-next/juno/updates in response to
https://lore.kernel.org/linux-arm-kernel/20260309-manipulative-inescapable-labradoodle-7a76e4@sudeepholla/
 arch/arm64/boot/dts/arm/zena-css-fvp.dts |  8 +++++
 arch/arm64/boot/dts/arm/zena-css.dtsi    | 44 ++++++++++--------------
 2 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/arch/arm64/boot/dts/arm/zena-css-fvp.dts b/arch/arm64/boot/dts/arm/zena-css-fvp.dts
index b75204a91882..53c5412d92b2 100644
--- a/arch/arm64/boot/dts/arm/zena-css-fvp.dts
+++ b/arch/arm64/boot/dts/arm/zena-css-fvp.dts
@@ -14,6 +14,14 @@ / {
 	chosen {
 		stdout-path = &soc_serial0;
 	};
+
+	memory@80000000 {
+		device_type = "memory";
+
+		/* ~2GB mapped at 2GB, another 2GB at 2TB */
+		reg = <0x00000000 0x80000000 0x00000000 0x7f000000>,
+		      <0x00000200 0x00000000 0x00000000 0x80000000>;
+	};
 };
 
 &soc {
diff --git a/arch/arm64/boot/dts/arm/zena-css.dtsi b/arch/arm64/boot/dts/arm/zena-css.dtsi
index 9899d2883337..0b41ee4bf4c6 100644
--- a/arch/arm64/boot/dts/arm/zena-css.dtsi
+++ b/arch/arm64/boot/dts/arm/zena-css.dtsi
@@ -634,6 +634,24 @@ soc: soc {
 		#size-cells = <2>;
 		ranges;
 
+		sram: sram@104000 {
+			compatible = "mmio-sram";
+			reg = <0x0 0x00104000 0x0 0x00001000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0x0 0x00104000 0x00001000>;
+
+			scmi_shmem_tx: scpshmem-sram-section@0 {
+				compatible = "arm,scmi-shmem";
+				reg = <0x0 0x100>;
+			};
+
+			scmi_shmem_rx: scpshmem-sram-section@100 {
+				compatible = "arm,scmi-shmem";
+				reg = <0x100 0x100>;
+			};
+		};
+
 		timer@1a810000 {
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0 0x1a810000 0x0 0x10000>;
@@ -748,30 +766,4 @@ timer {
 			     <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>,
 			     <GIC_PPI 12 IRQ_TYPE_LEVEL_LOW>;
 	};
-
-	sram: sram@104000 {
-		compatible = "mmio-sram";
-		reg = <0x0 0x00104000 0x0 0x00001000>;
-		#address-cells = <1>;
-		#size-cells = <1>;
-		ranges = <0 0x0 0x00104000 0x00001000>;
-
-		scmi_shmem_tx: scpshmem-sram-section@0 {
-			compatible = "arm,scmi-shmem";
-			reg = <0x0 0x100>;
-		};
-
-		scmi_shmem_rx: scpshmem-sram-section@100 {
-			compatible = "arm,scmi-shmem";
-			reg = <0x100 0x100>;
-		};
-	};
-
-	memory@80000000 {
-		device_type = "memory";
-
-		/* ~2GB mapped at 2GB, another 2GB at 2TB */
-		reg = <0x00000000 0x80000000 0x00000000 0x7f000000>,
-		      <0x00000200 0x00000000 0x00000000 0x80000000>;
-	};
 };
-- 
2.43.0

Re: [PATCH] arm64: dts: zena: Refactor Devicetree nodes

Posted by Sudeep Holla 3 weeks ago

On Wed, 11 Mar 2026 17:39:48 +0000, Debbie Horsfall wrote:
> Move the SRAM node into the SoC node. Move the memory node out of
> the include to make it customizable for each platform variant.
> 

Applied to sudeep.holla/linux (for-next/juno/updates), thanks!

[1/1] arm64: dts: zena: Refactor Devicetree nodes
      https://git.kernel.org/sudeep.holla/c/021915c7885f
--
Regards,
Sudeep

Re: [PATCH] arm64: dts: zena: Refactor Devicetree nodes

Posted by Andre Przywara 3 weeks, 4 days ago

Hi,

On 3/11/26 18:39, Debbie Horsfall wrote:
> Move the SRAM node into the SoC node. Move the memory node out of
> the include to make it customizable for each platform variant.

Looks good to me, indeed just moving the nodes around.

> Signed-off-by: Debbie Horsfall <debbie.horsfall@arm.com>

Reviewed-by: Andre Przywara <andre.przywara@arm.com>

Cheers,
Andre

> ---
> This targets for-next/juno/updates in response to
> https://lore.kernel.org/linux-arm-kernel/20260309-manipulative-inescapable-labradoodle-7a76e4@sudeepholla/
>   arch/arm64/boot/dts/arm/zena-css-fvp.dts |  8 +++++
>   arch/arm64/boot/dts/arm/zena-css.dtsi    | 44 ++++++++++--------------
>   2 files changed, 26 insertions(+), 26 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/arm/zena-css-fvp.dts b/arch/arm64/boot/dts/arm/zena-css-fvp.dts
> index b75204a91882..53c5412d92b2 100644
> --- a/arch/arm64/boot/dts/arm/zena-css-fvp.dts
> +++ b/arch/arm64/boot/dts/arm/zena-css-fvp.dts
> @@ -14,6 +14,14 @@ / {
>   	chosen {
>   		stdout-path = &soc_serial0;
>   	};
> +
> +	memory@80000000 {
> +		device_type = "memory";
> +
> +		/* ~2GB mapped at 2GB, another 2GB at 2TB */
> +		reg = <0x00000000 0x80000000 0x00000000 0x7f000000>,
> +		      <0x00000200 0x00000000 0x00000000 0x80000000>;
> +	};
>   };
>   
>   &soc {
> diff --git a/arch/arm64/boot/dts/arm/zena-css.dtsi b/arch/arm64/boot/dts/arm/zena-css.dtsi
> index 9899d2883337..0b41ee4bf4c6 100644
> --- a/arch/arm64/boot/dts/arm/zena-css.dtsi
> +++ b/arch/arm64/boot/dts/arm/zena-css.dtsi
> @@ -634,6 +634,24 @@ soc: soc {
>   		#size-cells = <2>;
>   		ranges;
>   
> +		sram: sram@104000 {
> +			compatible = "mmio-sram";
> +			reg = <0x0 0x00104000 0x0 0x00001000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0x0 0x00104000 0x00001000>;
> +
> +			scmi_shmem_tx: scpshmem-sram-section@0 {
> +				compatible = "arm,scmi-shmem";
> +				reg = <0x0 0x100>;
> +			};
> +
> +			scmi_shmem_rx: scpshmem-sram-section@100 {
> +				compatible = "arm,scmi-shmem";
> +				reg = <0x100 0x100>;
> +			};
> +		};
> +
>   		timer@1a810000 {
>   			compatible = "arm,armv7-timer-mem";
>   			reg = <0x0 0x1a810000 0x0 0x10000>;
> @@ -748,30 +766,4 @@ timer {
>   			     <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>,
>   			     <GIC_PPI 12 IRQ_TYPE_LEVEL_LOW>;
>   	};
> -
> -	sram: sram@104000 {
> -		compatible = "mmio-sram";
> -		reg = <0x0 0x00104000 0x0 0x00001000>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		ranges = <0 0x0 0x00104000 0x00001000>;
> -
> -		scmi_shmem_tx: scpshmem-sram-section@0 {
> -			compatible = "arm,scmi-shmem";
> -			reg = <0x0 0x100>;
> -		};
> -
> -		scmi_shmem_rx: scpshmem-sram-section@100 {
> -			compatible = "arm,scmi-shmem";
> -			reg = <0x100 0x100>;
> -		};
> -	};
> -
> -	memory@80000000 {
> -		device_type = "memory";
> -
> -		/* ~2GB mapped at 2GB, another 2GB at 2TB */
> -		reg = <0x00000000 0x80000000 0x00000000 0x7f000000>,
> -		      <0x00000200 0x00000000 0x00000000 0x80000000>;
> -	};
>   };