From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 11 Sep 2020 16:20:39 -0400 Subject: [PATCH 00/16] Kconfig: Tidy up the top-level kconfig menu In-Reply-To: References: <20200911022127.3451713-1-sjg@chromium.org> <20200911161106.GR7259@bill-the-cat> Message-ID: <20200911202039.GC5110@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 Fri, Sep 11, 2020 at 02:15:50PM -0600, Simon Glass wrote: > Hi Tom, > > On Fri, 11 Sep 2020 at 10:11, Tom Rini wrote: > > > > On Thu, Sep 10, 2020 at 08:21:11PM -0600, Simon Glass wrote: > > > > > > > > At present this menu is pretty messy, with quite a few minor options shown > > > at the top level. This series creates a few new menus and moves things > > > around so that the top-level menu is cleaner. > > > > > > There is more to do, but this is a start. > > [snip] > > > Kconfig | 340 +--------------- > > > cmd/Kconfig | 117 ------ > > > common/Kconfig | 505 ++---------------------- > > > common/Kconfig.boot | 894 +++++++++++++++++++++++++++++++++++++++++++ > > > drivers/core/Kconfig | 11 + > > > dts/Kconfig | 9 - > > > env/Kconfig | 9 + > > > tools/Kconfig | 12 + > > > 8 files changed, 955 insertions(+), 942 deletions(-) > > > create mode 100644 common/Kconfig.boot > > > create mode 100644 tools/Kconfig > > > > And after a resync of the defconfigs: > > 555 files changed, 941 insertions(+), 941 deletions(-) > > and a partial wc -l: > > 449 Kconfig > > 697 common/Kconfig > > 894 common/Kconfig.boot > > 2190 cmd/Kconfig > > > > So in the end, yes, this is I think making things easier to maintain but > > will cause a few merge hiccups. > > Oh dear, I completely forgot about that aspect. Should I add a resync > patch at the end, or is it better to do it when reviewed/applied? Don't worry about re-sync patches, those are trivially generated and fail to apply easily. Really I'm fine with most cases not touching defconfigs EXCEPT when we're changing requires / depends stuff. That requires a defconfig change at the same time to not break things. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: