[U-Boot] [PATCH 08/10] imx6: standardise filesystem and boot options

[Soeren Moch <smoch@web.de>]

On 05/11/15 19:21, Soeren Moch wrote:
> [...]
>
> > diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
> > index 08ae39c..862f668 100644
> > --- a/include/configs/mx6_common.h
> > +++ b/include/configs/mx6_common.h
> > @@ -58,6 +58,18 @@
> >  #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
>
> [...]
>
> > diff --git a/include/configs/tbs2910.h b/include/configs/tbs2910.h
> > index 34e12eb..5ec2ba6 100644
> > --- a/include/configs/tbs2910.h
> > +++ b/include/configs/tbs2910.h
> > @@ -58,15 +58,6 @@
> >  #define CONFIG_CMD_TIME
> >
> >  /* Filesystems / image support */
> > -#define CONFIG_CMD_EXT4
> > -#define CONFIG_CMD_FAT
> > -#define CONFIG_DOS_PARTITION
> > -#define CONFIG_EFI_PARTITION
> > -#define CONFIG_CMD_FS_GENERIC
> > -
> > -#define CONFIG_OF_LIBFDT
> > -#define CONFIG_CMD_BOOTZ
> > -#define CONFIG_SUPPORT_RAW_INITRD
> >  #define CONFIG_FIT
> >
>
> Peter,
>
> you removed CONFIG_EFI_PARTITION from tbs2910.h without adding it to mx6_common.h .
> Please fix this in your next version of the patch series.
>
> Thanks,
> Soeren
>
One additional comment:
Do we really need CONFIG_CMD_EXT2 when CONFIG_CMD_EXT4 is defined? My understanding
was so far, that CONFIG_CMD_EXT4 alone is enough.

Regards,
Soeren