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

[Andre Przywara]

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@xxxxxxx>

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

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>;
> -	};
>   };