[PATCH v2 1/3] ARM: dts: stm32: add SRAM binding

[Ahmad Fatoum]

Hello Steffen,

On 16.05.22 14:16, Steffen Trumtrar wrote:
> The STM32 has an SRAM from 0x1000000 to 0x10060000.

s/STM32/STM32MP15x/, also append an 'in stm32mp151.dtsi' to the title,
to clarify it's not about the MCUs.

> Make it available via mmio-sram.
> 
> Signed-off-by: Steffen Trumtrar <s.trumtrar at pengutronix.de>
> ---
>  arch/arm/boot/dts/stm32mp151.dtsi | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index 08708346d583..1816d9b02bb8 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -122,6 +122,14 @@ soc {
>  		interrupt-parent = <&intc>;
>  		ranges;
>  
> +		sram: sram at 10000000 {

We already have /ahb/m4 at 10000000, which is status = "disabled", but can be
used by remoteproc. Of course, boards are free to use the SRAM from Linux
too, but status should be disabled to avoid breaking existing boards.

Also, this is not the only SRAM on the SoC, could you change the label
to the name used in the datasheet/reference manual?

Thanks,
Ahmad

> +			compatible = "mmio-sram";
> +			reg = <0x10000000 0x60000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0x10000000 0x60000>;
> +		};
> +
>  		timers2: timer at 40000000 {
>  			#address-cells = <1>;
>  			#size-cells = <0>;


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |