On Mon, Oct 22, 2012 at 10:19:33AM -0600, Stephen Warren wrote: > From: Stephen Warren <swar...@nvidia.com> > > Add an ASSERT() to u-boot.lds to detect an SPL that doesn't fit within > SPL_TEXT_BASE..SPL_MAX_SIZE. > > Different .lds files implement this check in two possible ways: > 1) An ASSERT() like this > 2) Defining a MEMORY region of size SPL_MAX_SIZE, and re-directing all > linker output into that region. Since u-boot.lds is used for both > SPL and main U-Boot, this would entail only sometimes defining a > MEMORY region, and only sometimes performing that redirection, and > hence option (1) was deemed much simpler, and hence implemented. > > Note that this causes build failures at least for NVIDIA Tegra Seaboard > and Ventana. However, these are legitimate; the SPL doesn't fit within > the required space, and this does cause runtime issues. > > Signed-off-by: Stephen Warren <swar...@nvidia.com> > Acked-by: Simon Glass <s...@chromium.org> > Acked-by: Allen Martin <amar...@nvidia.com>
This isn't quite what I envisoned at first (see arch/arm/cpu/armv7/omap-common/u-boot-spl.lds) but I think for the generic linker script, this is the least instrusive method. Acked-by: Tom Rini <tr...@ti.com> And since parts 1 and 2 are generic code, I've assigned them to Albert in patchwork. It's his call if he wants to take them or have them all come via the tegra tree. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
