From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 8 Apr 2014 12:22:18 -0400 Subject: [U-Boot] [PATCH 6/7] omap3: zoom1: enable bootz In-Reply-To: References: <1396567167-17596-1-git-send-email-nm@ti.com> <1396567167-17596-7-git-send-email-nm@ti.com> <20140407142959.GE23803@bill-the-cat> Message-ID: <20140408162218.GP23803@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Apr 08, 2014 at 09:35:29AM -0500, Nishanth Menon wrote: > On Mon, Apr 7, 2014 at 9:29 AM, Tom Rini 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: