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

[Cyril Bur]

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.

Thanks,

Cyril

> >                 };
> >         };
> > 
> > --
> > 2.11.1
> >