From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 25 Jun 2015 12:15:47 -0400 Subject: [U-Boot] [PATCH 7/7] Move defaults from config_cmd_default.h to Kconfig In-Reply-To: References: <20150612120855.GV1728@bill-the-cat> <1435007730-8451-1-git-send-email-joe.hershberger@ni.com> <1435007730-8451-8-git-send-email-joe.hershberger@ni.com> <20150623143351.GX28577@bill-the-cat> Message-ID: <20150625161547.GD28577@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Jun 23, 2015 at 10:36:20PM -0500, Joe Hershberger wrote: > Hi Tom, > > On Tue, Jun 23, 2015 at 9:33 AM, Tom Rini wrote: > > On Mon, Jun 22, 2015 at 04:15:30PM -0500, Joe Hershberger wrote: > > > >> This sets the default commands Kconfig to match > >> include/config_cmd_default.h commands in the common/Kconfig and removes > >> them from include/configs. > > [snip] > >> diff --git a/common/Kconfig b/common/Kconfig > >> index cb14592..2976cd7 100644 > >> --- a/common/Kconfig > >> +++ b/common/Kconfig > > [snip] > >> config CMD_IMLS > >> bool "imls" > >> + default y > >> help > >> List all images found in flash > > [snip] > >> > >> config CMD_FLASH > >> bool "flinfo, erase, protect" > >> + default y > >> help > >> NOR flash support. > >> flinfo - print FLASH memory information > > > > Today we only set these when !SYS_NO_FLASH so we need to Kconfig that > > first. > > While that's true, the moveconfig tool will de-select that option > based on SYS_NO_FLASH having been set for that board. While that may > not be ideal as far as reacting to an end-user changing that default > setting for a given board, the defconfig itself should be consistent > with the former default behavior. Also, once SYS_NO_FLASH is moved, > such defconfig entries will be removed automatically by savedefconfig. > I was trying to limit the number of changes included in this already > sizable undertaking. > > If you feel it's important to include moving this change at the same > time, then I can add a patch at the end of the series to include this > move. > > >> @@ -352,6 +371,7 @@ menu "Network commands" > >> config CMD_NET > >> bool "bootp, tftpboot" > >> select NET > >> + default y > >> help > >> Network commands. > >> bootp - boot image via network using BOOTP/TFTP protocol > >> @@ -379,6 +399,7 @@ config CMD_DHCP > >> > >> config CMD_NFS > >> bool "nfs" > >> + default y > >> help > >> Boot image via network using NFS protocol. > > > > I think we now have the smarts available to us to do this only if we > > have NET set, so "depends NET", yes? > > Sure. It could also be a follow-on, though, right? I'm just a bit > hesitant to redo all of this, since it really takes a long time to > validate (and still may not be perfect). Sorry for the delay. Yeah, OK, we can do these as follow-ups but lets not wait too long. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: