[U-Boot] [PATCH] Kconfig: Use imply for network commands in distro default
Tom Rini
trini at konsulko.com
Tue Feb 6 03:02:52 UTC 2018
On Wed, Jan 31, 2018 at 04:09:57PM +0100, Michal Simek wrote:
> There is no reason to unconditially select network commands as distro
> defaults. Systems without ethernet but with sd, sata, usb should also be
> able to use distro defaults. Use imply instead of select.
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
> Kconfig | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Kconfig b/Kconfig
> index 9b8a8077993a..55bb00936cf7 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -69,14 +69,14 @@ config DISTRO_DEFAULTS
> imply USE_BOOTCOMMAND
> select CMD_BOOTZ if ARM && !ARM64
> select CMD_BOOTI if ARM64
> - select CMD_DHCP
> - select CMD_PXE
> + imply CMD_DHCP
> + imply CMD_PXE
> select CMD_EXT2
> select CMD_EXT4
> select CMD_FAT
> select CMD_FS_GENERIC
> - select CMD_MII
> - select CMD_PING
> + imply CMD_MII
> + imply CMD_PING
> select CMD_PART
> select HUSH_PARSER
> help
Should we perhaps use select CMD_DHCP if NET, etc? There's a certain
level of non-opt-outedness that's supposed to be here, so long as the
underlying functionality is there. And a follow-up might be that a
bunch more CMDs need to be nested under / depend on NET to start with.
But that's just a thought. Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180205/5e3ffa07/attachment.sig>
More information about the U-Boot
mailing list