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

[Nishanth Menon]

On Tue, Apr 8, 2014 at 11:22 AM, Tom Rini <trini at ti.com> wrote:
> 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.
Done in v2.