On Tue, Nov 15, 2016 at 09:59:22AM +0000, Max Krummenacher wrote: > Hi Stefan > > On Mon, 2016-11-14 at 18:06 -0800, Stefan Agner wrote: > > From: Stefan Agner <stefan.ag...@toradex.com> > > > > When building SPL with eMMC boot support, specify standard padding > > for payload to match the load sector during boot. This allows to > > use the u-boot-with-spl.imx build target to generate a direcly > > flashable image which can be flashed using: > > > > dd if=u-boot-with-spl.imx of=/dev/mmcblk0 bs=512 skip=2 > > > > Signed-off-by: Stefan Agner <stefan.ag...@toradex.com> > > > > --- > > > > include/configs/imx6_spl.h | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h > > index 76d1ca0..ca3ed19 100644 > > --- a/include/configs/imx6_spl.h > > +++ b/include/configs/imx6_spl.h > > @@ -42,6 +42,12 @@ > > #define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1 > > #define > > CONFIG_SYS_MONITOR_LEN (CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS/2*1024) > > #define CONFIG_SPL_ABORT_ON_RAW_IMAGE > > +/* > > + * SPL starts at 1KB (ROM offset), hence we need to pad SPL up to > > 68KB so > > + * that payload (U-Boot) will be at 69KB (sector 138) on the eMMC/SD > > card. > > + * Required for targets such as u-boot-with-spl.imx. > > + */ > > +#define CONFIG_SPL_PAD_TO 0x11000 > > #endif > > > > /* SATA support */ > > This has been brought up already: > https://www.mail-archive.com/u-boot@lists.denx.de/msg228246.html > I agreed with Marek that this might disrupt other boot sources > (NOR/SPI/ ...) which may or may not decide to use another offset > between SPL and U-Boot. > Thus it is advisable to move that define to the individual board > configuration where one knows for what boot device one wants to > use the combined u-boot-with-spl.imx binary.
I'm going to have to disagree with Marek here then. Looking over the explanation in include/configs/imx6_spl.h we have: 1KiB up front "wasted" for a partition table 4KiB up next for the ROM-mandated header 64KiB for SPL itself, which is our max size, unconditionally set above this bit of context. But it is the lowest common denominator in terms of IRAM size in the families. Which brings us to the SPL_PAD_TO above, 68KiB. We cannot go lower than this. No one is changing any of this either today nor setting SPL_PAD_TO so u-boot-with-spl.imx isn't usable today. None of this breaks using separate files. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
