[PATCH] ARM: dts: aspeed: Update reserved memory nodes for zaius and witherspoon

[Cyril Bur]

On Fri, 2017-02-17 at 13:51 +1100, Cyril Bur wrote:
> On Thu, 2017-02-16 at 14:14 +0800, Mine wrote:
> > Hi Cyril,
> > 
> > On Thu, Feb 16, 2017 at 1:49 PM, Cyril Bur <cyrilbur at gmail.com> wrote:
> > > Signed-off-by: Cyril Bur <cyrilbur at gmail.com>
> > > ---
> > >  arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts | 9 +++++++--
> > >  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts       | 9 +++++++--
> > >  2 files changed, 14 insertions(+), 4 deletions(-)
> > > 
> > > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> > > index 4b2d8aa8abb6..2d76bfee1262 100644
> > > --- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> > > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> > > @@ -25,9 +25,14 @@
> > >                 #size-cells = <1>;
> > >                 ranges;
> > > 
> > > -               flash_memory: region at 94000000 {
> > > +               flash_memory: region at 98000000 {
> > >                         no-map;
> > > -                       reg = <0x94000000 0x04000000>; /* 64M */
> > > +                       reg = <0x98000000 0x04000000>; /* 64M */
> > > +               };
> > > +
> > > +               vga_memory: framebuffer at 9c000000 {
> > > +                       no-map;
> > > +                       reg = <0x9c000000 0x04000000>; /* 64MB */
> > 
> > This assumes that Witherspoon has 512MiB RAM, then why the memory part
> > indicates it has 1GiB?
> > ```
> >         memory {
> >                 reg = <0x80000000 0x40000000>;
> >         };
> > ```
> 
> Oops, you're correct. I'll adjust the numbers.
> 
> > 
> > >                 };
> > >         };
> > > 
> > > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> > > index e1c9b3f4fe44..f30288f735bc 100644
> > > --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> > > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> > > @@ -25,9 +25,14 @@
> > >                 #size-cells = <1>;
> > >                 ranges;
> > > 
> > > -               flash_memory: region at 94000000 {
> > > +               flash_memory: region at 98000000 {
> > >                         no-map;
> > > -                       reg = <0x94000000 0x04000000>; /* 64M */
> > > +                       reg = <0x98000000 0x04000000>; /* 64M */
> > > +               };
> > > +
> > > +               vga_memory: framebuffer at 9c000000 {
> > > +                       no-map;
> > > +                       reg = <0x9c000000 0x04000000>; /* 64MB */
> > 
> > As I know Google decides to not support VGA on Zaius, so maybe
> > vga_memory is not needed to Zaius at all.
> > 
> 
> Happy to drop that if there won't be any VGA on the platform.

Sorry for the followup, I assume it would then make sense to put the
flash_memory at the top of RAM?

> 
> Thanks,
> 
> Cyril
> 
> > >                 };
> > >         };
> > > 
> > > --
> > > 2.11.1
> > >