Re: [PATCH v1 16/19] arm: dts: mt7623: fixup available memory size on bananapi-r2

From: Sean Wang
Date: Fri Mar 02 2018 - 18:27:22 EST


On Fri, 2018-03-02 at 09:42 -0600, Rob Herring wrote:
> On Fri, Feb 23, 2018 at 06:16:36PM +0800, sean.wang@xxxxxxxxxxxx wrote:
> > From: Sean Wang <sean.wang@xxxxxxxxxxxx>
> >
> > There is 2GB DDR3 available on bananapi-r2 board as [1] specified.
> >
> > [1] http://www.banana-pi.org/r2.html
> >
> > Signed-off-by: Sean Wang <sean.wang@xxxxxxxxxxxx>
> > ---
> > arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> > index 140ff78..3e8d02c 100644
> > --- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> > +++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> > @@ -38,6 +38,11 @@
> > default-state = "off";
> > };
> > };
> > +
> > + memory {
>
> Unit address?
>

If I did it with adding unit address

- memory {
+ memory@80000000 {
device_type = "memory";
reg = <0 0x80000000 0 0x80000000>;
};


bad dtc blob is being generated and contains two memory nodes, one is
memory and the other is memory@80000000 whose blob disassembly detail is
as the following.

memory {
device_type = "memory";
reg = <0x0 0x0 0x0 0x0>;
};



memory@80000000 {
device_type = "memory";
reg = <0x0 0x80000000 0x0 0x80000000>;
};


and bad memory node with size 0 would cause the boot fails.


is it a dtc compiler problem ?

> > + device_type = "memory";
> > + reg = <0 0x80000000 0 0x80000000>;
> > + };
> > };
> >
> > &afe {
> > --
> > 2.7.4
> >