On Tue, Apr 8, 2014 at 11:22 AM, Tom Rini trini@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@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.