Re: [PATCH 3/5] ARM: dts: at91: sama5d2: Add memory node in devicetree

Clément Léger
2021-08-03T09:05:17Z

Le Tue, 3 Aug 2021 08:58:17 +0000,
<Eugen.Hristev@microchip.com> a écrit :

> On 8/2/21 3:39 PM, Clément Léger wrote:
> > sama5d2 support from DRAM will be modified to use device tree
> > instead of hardcoded addresses. In order to prepare that, add the
> > memory node to sama5d2.dtsi file since the boards supported by
> > U-Boot with sama5d2 all contains 512Mb of DDR.
> >
> > Signed-off-by: Clément Léger <clement.leger@bootlin.com>
> > ---
> > arch/arm/dts/sama5d2.dtsi | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm/dts/sama5d2.dtsi b/arch/arm/dts/sama5d2.dtsi
> > index 6fb2cb25f9..299aefc496 100644
> > --- a/arch/arm/dts/sama5d2.dtsi
> > +++ b/arch/arm/dts/sama5d2.dtsi
> > @@ -4,6 +4,10 @@
> > model = "Atmel SAMA5D2 family SoC";
> > compatible = "atmel,sama5d2";
> >
> > + memory {
> > + reg = <0x20000000 0x20000000>;
> > + };
>
> Hi,
>
> Memory is external to the SoC. You have to move this node to the
> board specific DTS.

Ok, I'll move this to sama5d2_xplained only since this series has been
made and tested on sama5d2_xplained/sama5d27_som1_ek.

Clément

>
> Even if all current boards have 512 MB of DRAM , this does not belong
> here.
>
> And by the way this is not true, since this SoC can work in a SiP
> with only 256 or 128 MB of DRAM for example and this is true for
> certain boards ( wlsom1_ek comes to my mind now).
>
> Eugen
>
> > +
> > aliases {
> > spi0 = &spi0;
> > spi1 = &qspi0;
> > --
> > 2.32.0
> >
>

--
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin