On Sun, Jan 17, 2016 at 06:23:45PM +0100, Peter Korsgaard wrote: > Commit 1e3d640316 (ARM: sheevaplug: redefine MTDPARTS) changed the partition > layout (without any description why), but didn't change the offset/size to > load the kernel from or the root=/dev/mtdblockX in the bootargs. > > The 3MB forseen for a kernel is furthermore too little. A 4.4 build of > mvebu_v5_defconfig is 3.6MB: > > -rw-r--r-- 1 peko peko 3.6M Jan 16 20:24 uImage.kirkwood-sheevaplug > > When device tree support for sheevaplug was added to the kernel in commit > ee514b381e (ARM: Kirkwood: Add dts files for Sheevaplug and eSATA > Sheevaplug) a default flash partition layout (used if mtdparts= isn't passed > on the command line / CONFIG_MTD_CMDLINE_PARTS isn't enabled) with 1MB for > u-boot / environment, 4MB for the kernel and the rest for the rootfs, so use > that layout here and adjust the kernel loading to match. > > Signed-off-by: Peter Korsgaard <pe...@korsgaard.com> > --- > include/configs/sheevaplug.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/configs/sheevaplug.h b/include/configs/sheevaplug.h > index c529636..f9fb9bc 100644 > --- a/include/configs/sheevaplug.h > +++ b/include/configs/sheevaplug.h > @@ -64,12 +64,12 @@ > > #define CONFIG_MTDPARTS \ > "orion_nand:512K(uboot)," \ > - "512K(env),1M(script),6M(kernel)," \ > - "12M(ramdisk),4M(spare),-(rootfs)\0" > + "512K(env),4M(kernel)," \ > + "-(rootfs)\0" > > #define CONFIG_EXTRA_ENV_SETTINGS "x_bootargs=console" \ > "=ttyS0,115200 mtdparts="CONFIG_MTDPARTS \ > - "x_bootcmd_kernel=nand read 0x6400000 0x100000 0x300000\0" \ > + "x_bootcmd_kernel=nand read 0x6400000 0x100000 0x400000\0" \ > "x_bootcmd_usb=usb start\0" \ > "x_bootargs_root=root=/dev/mtdblock3 rw rootfstype=jffs2\0"
Can I suggest future proofing this by using the names set in MTDPARTS instead of direct offsets? Thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
