From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Thu, 8 Feb 2018 11:31:09 +0530 Subject: [U-Boot] [PATCH v2] Kconfig: Select networking commands only when NET is enabled In-Reply-To: <6617f17b-8e5c-df00-3580-d66b7a022651@monstr.eu> References: <6617f17b-8e5c-df00-3580-d66b7a022651@monstr.eu> Message-ID: <7eca8b76-860b-29e6-0d6c-3a2a46443f81@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Michal, On Wednesday 07 February 2018 01:14 PM, Michal Simek wrote: > Hi Lokesh, > > On 6.2.2018 13:28, Michal Simek wrote: >> There is no reason to unconditially select network commands as distro >> defaults without networking enable. >> >> Signed-off-by: Michal Simek >> --- >> >> Changes in v2: >> - Use if NET instead of imply (suggested by Tom) >> - Change patch subject and description >> >> Kconfig | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/Kconfig b/Kconfig >> index f713c6a9b188..166e7841b11f 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 >> + select CMD_DHCP if NET >> + select CMD_PXE if NET >> select CMD_EXT2 >> select CMD_EXT4 >> select CMD_FAT >> select CMD_FS_GENERIC >> - select CMD_MII >> - select CMD_PING >> + select CMD_MII if NET >> + select CMD_PING if NET >> select CMD_PART >> select HUSH_PARSER >> help >> > > Based on > https://travis-ci.org/michalsimek/u-boot/jobs/338051973 > > Lokesh: > There is an issue with omap4_sdp4430_defconfig where CONFIG_NET is not > enabled but PXE and DHCP configs are present. > This is quite weird and IMHO CONFIG_NET should be enabled for this target. > This can be one resolution. Yeah, resolution looks good to me. Tom, any comments? > > diff --git a/configs/omap4_sdp4430_defconfig > b/configs/omap4_sdp4430_defconfig > index ac495710633c..59526fd1dc35 100644 > --- a/configs/omap4_sdp4430_defconfig > +++ b/configs/omap4_sdp4430_defconfig > @@ -23,7 +23,10 @@ CONFIG_CMD_SPI=y > # CONFIG_CMD_NET is not set > # CONFIG_CMD_NFS is not set > CONFIG_CMD_EXT4_WRITE=y > +# CONFIG_EFI_PARTITION is not set > +CONFIG_SPL_PARTITION_UUIDS=y > CONFIG_ENV_IS_IN_MMC=y > +CONFIG_NET=y > CONFIG_MMC_OMAP_HS=y > CONFIG_SYS_NS16550=y > CONFIG_OMAP3_SPI=y > @@ -31,4 +34,6 @@ CONFIG_USB=y > CONFIG_USB_MUSB_UDC=y > CONFIG_USB_OMAP3=y > CONFIG_FAT_WRITE=y > +# CONFIG_REGEX is not set > CONFIG_OF_LIBFDT=y > +# CONFIG_EFI_LOADER is not set Thanks and regards, Lokesh > > > Thanks, > Michal > >