[PATCH 00/16] Kconfig: Tidy up the top-level kconfig menu

Tom Rini trini at konsulko.com
Fri Sep 11 22:20:39 CEST 2020


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 <trini at konsulko.com> 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: <https://lists.denx.de/pipermail/u-boot/attachments/20200911/d8cb897e/attachment.sig>


More information about the U-Boot mailing list