From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 1 Aug 2014 16:43:07 -0400 Subject: [U-Boot] [PATCH 7/7] sunxi: Add environment settings to make extlinux.conf booting work In-Reply-To: <53DBF710.2000708@wwwdotorg.org> References: <1406879207-12322-1-git-send-email-hdegoede@redhat.com> <1406879207-12322-7-git-send-email-hdegoede@redhat.com> <53DBE31B.9000701@wwwdotorg.org> <20140801150533.4f49589e@adria.ausil.us> <53DBF710.2000708@wwwdotorg.org> Message-ID: <20140801204307.GA19374@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 Fri, Aug 01, 2014 at 02:22:40PM -0600, Stephen Warren wrote: > On 08/01/2014 02:05 PM, Dennis Gilmore wrote: > >On Fri, 01 Aug 2014 12:57:31 -0600 > >Stephen Warren wrote: > > > >>On 08/01/2014 01:46 AM, Hans de Goede wrote: > >>>Automatic booting using an extlinux.conf file requires various > >>>environment variables to be set. > >> > >>Acked-by: Stephen Warren > >> > >>I'd personally be tempted to set fdt_high=0xffffffff, > >>initrd_high=0xffffffff to stop U-Boot copying the DT/initrd from the > >>load location to some other location under 256M, but that's just an > >>optimization and entirely optional. > > > >There has been quite a few times where using 0xffffff has caused > >issues. > > What kind of issues? > > At least for Tegra, I've carefully chosen the values for the various > load addresses so that there won't be issues. (Without that I can > easily see the potential for issues.) I've seen far more repeated > problems when U-Boot moves the DT/initrd around than than when it > didn't (none in that case). Besides, it's completely redundant and > unnecessary work if the blobs are already loaded at sane addresses, > which they are on Tegra at least. Just how large of a kernel have you thrown on a Tegra? 32MB might seem reasonable at first but it wouldn't be overly surprised if someone can shove a BSS into there. I know I shoved DT into 128MB by default because a 32bit ARM kernel isn't functional at that size. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: