On Sat, Jun 11, 2016 at 04:54:47PM +0200, Hans de Goede wrote: > DISTRO_DEFAULTS is intended to mirror / replace > include/config_distro_defaults.h. > > The intend is for boards which include this file to select this from > their Kconfig files and when moving setting to Kconfig which are #define-ed > in config_distro_defaults.h to select this from DISTRO_DEFAULTS so that > boards which have selected DISTRO_DEFAULTS will keep the same configuration > as before without needing any defconfig file changes. > > The initial list of selected things matches all settings recently removed > from config_distro_defaults.h because they have been converted to Kconfig, > with the exception of CMD_ELF and CMD_NET, which have a default of y, if > the default of these ever changes they should be selected by DISTRO_DEFAULTS > too. > > For testing and example purposes this commit also converts ARCH_SUNXI > to use DISTRO_DEFAULT instead of selecting everything it needs itself. > > Signed-off-by: Hans de Goede <hdego...@redhat.com> > --- > Kconfig | 16 ++++++++++++++++ > arch/arm/Kconfig | 10 +--------- > common/Kconfig | 1 + > 3 files changed, 18 insertions(+), 9 deletions(-) > > diff --git a/Kconfig b/Kconfig > index 4b46216..cc2902c 100644 > --- a/Kconfig > +++ b/Kconfig > @@ -53,6 +53,22 @@ config CC_OPTIMIZE_FOR_SIZE > > This option is enabled by default for U-Boot. > > +config DISTRO_DEFAULTS > + bool "Select defaults suitable for booting general purpose Linux > distributions" > + default n > + select CMD_BOOTZ > + select CMD_DHCP > + select CMD_EXT2 > + select CMD_EXT4 > + select CMD_FAT > + select CMD_FS_GENERIC > + select CMD_MII > + select CMD_PING > + select HUSH_PARSER > + help > + Select this to enable various options and commands which are suitable > + for building u-boot for booting general purpose Linux distributions.
Thanks for doing this, this part here looks good. > + > config SYS_MALLOC_F > bool "Enable malloc() pool before relocation" > default y if DM > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 4aeb4d2..4f6a3b3 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -584,24 +584,16 @@ config TARGET_CM_T43 > > config ARCH_SUNXI > bool "Support sunxi (Allwinner) SoCs" > - select CMD_BOOTZ > - select CMD_DHCP > - select CMD_EXT2 > - select CMD_EXT4 > - select CMD_FAT > - select CMD_FS_GENERIC > select CMD_GPIO > - select CMD_MII > select CMD_MMC if MMC > - select CMD_PING > select CMD_USB > + select DISTRO_DEFAULTS Here's what I don't like, and is a double edged sword. I'd rather see this default y if ARCH_SUNXI above (and likely to add in a bunch of other SoCs/etc) so that people that are doing a custom board based on sunxi or whatever can still turn off what they want to turn off and the defconfig growth burden goes on them. Thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot