Re: [PATCH 2/2] ARM: dts: stm32mp13: add SCMI nodes

2022-07-07 Thread Patrice CHOTARD
Hi Patrick

On 7/6/22 18:20, Patrick Delaunay wrote:
> Add the node for SCMI firmware with the associated reserved memory nodes
> 
> Signed-off-by: Patrick Delaunay 
> ---
> 
>  arch/arm/dts/stm32mp13-u-boot.dtsi | 20 +++
>  arch/arm/dts/stm32mp131.dtsi   | 31 ++
>  2 files changed, 51 insertions(+)
> 
> diff --git a/arch/arm/dts/stm32mp13-u-boot.dtsi 
> b/arch/arm/dts/stm32mp13-u-boot.dtsi
> index adc7e67ee58..01552adb7c4 100644
> --- a/arch/arm/dts/stm32mp13-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp13-u-boot.dtsi
> @@ -90,6 +90,26 @@
>   u-boot,dm-pre-reloc;
>  };
>  
> + {
> + u-boot,dm-pre-reloc;
> +};
> +
> +_clk {
> + u-boot,dm-pre-reloc;
> +};
> +
> +_reset {
> + u-boot,dm-pre-reloc;
> +};
> +
> +_shm {
> + u-boot,dm-pre-reloc;
> +};
> +
> +_sram {
> + u-boot,dm-pre-reloc;
> +};
> +
>   {
>   u-boot,dm-pre-reloc;
>  };
> diff --git a/arch/arm/dts/stm32mp131.dtsi b/arch/arm/dts/stm32mp131.dtsi
> index 652743fd06e..8f7af65e3ed 100644
> --- a/arch/arm/dts/stm32mp131.dtsi
> +++ b/arch/arm/dts/stm32mp131.dtsi
> @@ -27,11 +27,42 @@
>   interrupt-parent = <>;
>   };
>  
> + scmi_sram: sram@2000 {
> + compatible = "mmio-sram";
> + reg = <0x2000 0x1000>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges = <0 0x2000 0x1000>;
> +
> + scmi_shm: scmi_shm@0 {
> + compatible = "arm,scmi-shmem";
> + reg = <0 0x80>;
> + };
> + };
> +
>   firmware {
>   optee: optee {
>   method = "smc";
>   compatible = "linaro,optee-tz";
>   };
> +
> + scmi: scmi {
> + compatible = "linaro,scmi-optee";
> + #address-cells = <1>;
> + #size-cells = <0>;
> + linaro,optee-channel-id = <0>;
> + shmem = <_shm>;
> +
> + scmi_clk: protocol@14 {
> + reg = <0x14>;
> + #clock-cells = <1>;
> + };
> +
> + scmi_reset: protocol@16 {
> + reg = <0x16>;
> + #reset-cells = <1>;
> + };
> + };
>   };
>  
>   clocks {
Reviewed-by: Patrice Chotard 

Thanks
Patrice


[PATCH 2/2] ARM: dts: stm32mp13: add SCMI nodes

2022-07-06 Thread Patrick Delaunay
Add the node for SCMI firmware with the associated reserved memory nodes

Signed-off-by: Patrick Delaunay 
---

 arch/arm/dts/stm32mp13-u-boot.dtsi | 20 +++
 arch/arm/dts/stm32mp131.dtsi   | 31 ++
 2 files changed, 51 insertions(+)

diff --git a/arch/arm/dts/stm32mp13-u-boot.dtsi 
b/arch/arm/dts/stm32mp13-u-boot.dtsi
index adc7e67ee58..01552adb7c4 100644
--- a/arch/arm/dts/stm32mp13-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp13-u-boot.dtsi
@@ -90,6 +90,26 @@
u-boot,dm-pre-reloc;
 };
 
+ {
+   u-boot,dm-pre-reloc;
+};
+
+_clk {
+   u-boot,dm-pre-reloc;
+};
+
+_reset {
+   u-boot,dm-pre-reloc;
+};
+
+_shm {
+   u-boot,dm-pre-reloc;
+};
+
+_sram {
+   u-boot,dm-pre-reloc;
+};
+
  {
u-boot,dm-pre-reloc;
 };
diff --git a/arch/arm/dts/stm32mp131.dtsi b/arch/arm/dts/stm32mp131.dtsi
index 652743fd06e..8f7af65e3ed 100644
--- a/arch/arm/dts/stm32mp131.dtsi
+++ b/arch/arm/dts/stm32mp131.dtsi
@@ -27,11 +27,42 @@
interrupt-parent = <>;
};
 
+   scmi_sram: sram@2000 {
+   compatible = "mmio-sram";
+   reg = <0x2000 0x1000>;
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges = <0 0x2000 0x1000>;
+
+   scmi_shm: scmi_shm@0 {
+   compatible = "arm,scmi-shmem";
+   reg = <0 0x80>;
+   };
+   };
+
firmware {
optee: optee {
method = "smc";
compatible = "linaro,optee-tz";
};
+
+   scmi: scmi {
+   compatible = "linaro,scmi-optee";
+   #address-cells = <1>;
+   #size-cells = <0>;
+   linaro,optee-channel-id = <0>;
+   shmem = <_shm>;
+
+   scmi_clk: protocol@14 {
+   reg = <0x14>;
+   #clock-cells = <1>;
+   };
+
+   scmi_reset: protocol@16 {
+   reg = <0x16>;
+   #reset-cells = <1>;
+   };
+   };
};
 
clocks {
-- 
2.25.1