[PATCH 4/5] ARM: vexpress: Initial RS1 memory map support
Pawel Moll
pawel.moll at arm.com
Fri Nov 18 23:20:50 EST 2011
On Thu, 2011-11-17 at 15:36 +0000, Russell King - ARM Linux wrote:
> > +config ARCH_VEXPRESS_LEGACY
> > + bool
> > + select AUTO_ZRELADDR
> > + select ARM_PATCH_PHYS_VIRT
> > +
> > +config ARCH_VEXPRESS_RS1
> > + bool
> > +
> >
> This doesn't make sense. Why should CA9x4 be forced to use the P2V stuff
> and the AUTO_ZRELADDR stuff? Hint: it works fine today without.
>
> > @@ -1,3 +1,3 @@
> > - zreladdr-y += 0x60008000
> > -params_phys-y := 0x60000100
> > -initrd_phys-y := 0x60800000
> > + zreladdr-y += 0x80008000
> > +params_phys-y := 0x80000100
> > +initrd_phys-y := 0x80800000
>
> So is this the reason - you don't want to turn this into a config, so you
> want to force all the automatic zreladdr stuff on?
Yes, exactly.
> You do know that this
> will probably cause uboot to load the uImage at 0x80008000 instead of
> 0x60008000, and therefore we'll lose half the RAM on this board?
I missed that - I don't use uboot, so I don't build uImages.
As the ARCH_VEXPRESS_LEGACY will be no more (see my other response) I'll
revert the order, so the _RS1 will enforce P2V, AUTO_ZRELADDR and change
the zrealaddr-y & friends (as in: 0x6* when no _RS1, 0x8* when _RS1=y).
Is there any other way of having single binary kernel booting from both
0x60008000 and 0x80008000?
All comments appreciated as always, thanks!
Pawel
More information about the devicetree-discuss
mailing list