Re: [PATCH linux 1/3] ARM: dts: aspeed: Update palmetto device tree

[Cyril Bur <cyrilbur@gmail.com>]

On Tue, 2016-12-13 at 14:29 +1100, Joel Stanley wrote:
> On Tue, Dec 13, 2016 at 2:11 PM, Cyril Bur <cyrilbur@gmail.com>
> wrote:
> > > > --- a/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> > > > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> > > > @@ -17,7 +17,30 @@
> > > >         };
> > > > 
> > > >         memory {
> > > > -               reg = <0x40000000 0x10000000>;
> > > > +               reg = <0x40000000 0x20000000>;
> > > > +       };
> > > 
> > > Can you send this in a separate patch? It should go into the tree
> > > now.
> > > 
> > 
> > Yep, should I send just for palmetto?
> > 
> > Do you have a list of other platforms? I figure all the ast2400
> > based
> > aspeed-bmc-opp-* should get this change, should I include them all
> > in
> > one patch or all individually?
> 
> One patch will do. I checked all of the machines and they all have
> 512MB (minus bits that I assume are reserved for framebuffers, etc.):
> 
> Barreleye: 499020 kB
> Palmetto: 424204 kB
> Firestone: 416020 kB
> Garrison: 416012 kB
> 
> 

Ok great thanks!

Final question, I assume the same thing in a separate patch is good for
the framebuffer, I'd forgotten that the size of VGA memory can be
altered by a hardware strapping reg '0x1E6E2070 Hardware Strapping
Table' bits 3:2 define it. The default is for 16MB, is it safe assume
the platforms will come up with strapping reg in default state?

> Cheers,
> 
> Joel