[U-Boot] [PATCH 6/7] omap3: zoom1: enable bootz

[Tom Rini]

On Tue, Apr 08, 2014 at 09:35:29AM -0500, Nishanth Menon wrote:
> On Mon, Apr 7, 2014 at 9:29 AM, Tom Rini <trini at ti.com> wrote:
> > On Thu, Apr 03, 2014 at 06:19:25PM -0500, Nishanth Menon wrote:
> >
> >> Boot from zImage and fdt_file if uImage is not available to maintain
> >> the legacy behavior.
> > [snip]
> >>  #define CONFIG_EXTRA_ENV_SETTINGS \
> >>       "loadaddr=0x82000000\0" \
> >> +     "fdt_high=0xffffffff\0" \
> >> +     "fdtaddr=0x80f80000\0" \
> >
> > OK, how much memory does this board ever have?  If less than 512MB,
> > don't do fdt_high.  If 256MB or more, please put fdtaddr at base+128MB.
> > Thanks!
> 
> As per the board dts in kernel, this platform has 128MB. As per the
> configuration files, CS1 might not be populated on many boards as
> well. so 128MB seems to be the max memory available here.

I never trust the dts in the kernel wrt memory size, at least for TI
platforms as it's always 128MB and we've always gotten by with a
run-time fixup (or people not noticing their board had less memory than
it really does).  And, aside, they ought to adapt the PowerPC convention
of saying 0 for memory size so it's clear that the value is fixed up
elsewhere.

> just remove fdt_high?

Yup.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140408/5ddd2f99/attachment.pgp>