On 23.05.2015 01:06, Peter Robinson wrote: > > On 22 May 2015 19:48, "Soeren Moch" <sm...@web.de <mailto:sm...@web.de>> > wrote: >>> >>> Move all standard filesystem, partition and fdt options to mx6_common. >>> >>> Signed-off-by: Peter Robinson <pbrobinson at gmail.com > <http://gmail.com> <http://lists.denx.de/mailman/listinfo/u-boot>> >> >> >> [...] >> >> >>> diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h >>> index 07fc3e8..c323038 100644 >>> --- a/include/configs/mx6_common.h >>> +++ b/include/configs/mx6_common.h >>> @@ -67,6 +67,17 @@ >>> #define CONFIG_CONS_INDEX 1 >>> #define CONFIG_BAUDRATE 115200 >>> >>> +/* Filesystems and image support */ >>> +#define CONFIG_OF_LIBFDT >>> +#define CONFIG_CMD_BOOTZ >>> +#define CONFIG_SUPPORT_RAW_INITRD >>> +#define CONFIG_CMD_FS_GENERIC >>> +#define CONFIG_DOS_PARTITION >>> +#define CONFIG_CMD_EXT2 >>> +#define CONFIG_CMD_EXT4 >>> +#define CONFIG_CMD_EXT4_WRITE >>> +#define CONFIG_CMD_FAT >>> + >>> /* Miscellaneous configurable options */ >>> #define CONFIG_SYS_NO_FLASH >>> #undef CONFIG_CMD_IMLS >> >> >> >> Peter, >> >> Thanks for fixing EFI_PARTITION in tbs2910. >> >> You still define CONFIG_CMD_EXT2 and CONFIG_CMD_EXT4, while >> only CONFIG_CMD_EXT4 is required. That is not really a problem, >> but since you are doing cleanup here... >> > > Yes, i looked at this and a good half were including ext2. I plan on > doing another pass so I figure there's enough churn already in this > series so get it merged and ensure there's no regression and then > re-review. >
Ah ok, if you want to run ext2ls / ext2load you need CONFIG_CMD_EXT2 explicitly. I thought that this would also be enabled by CONFIG_CMD_EXT4 automatically. Sorry for the noise, Soeren _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot