Re: RFC: Support for U-Boot phases in Kconfig

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 3819 bytes --]

On Wed, Aug 11, 2021 at 12:28:13PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On Wed, 11 Aug 2021 at 09:40, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Wed, Aug 11, 2021 at 08:26:38AM -0600, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Wed, 11 Aug 2021 at 08:17, Tom Rini <trini@konsulko.com> wrote:
> > > >
> > > > On Wed, Aug 11, 2021 at 08:03:00AM -0600, Simon Glass wrote:
> > > > > Hi Tom,
> > > > >
> > > > > On Wed, 11 Aug 2021 at 07:47, Tom Rini <trini@konsulko.com> wrote:
> > > > > >
> > > > > > On Wed, Aug 11, 2021 at 06:56:31AM -0600, Simon Glass wrote:
> > > > > > > Hi Tom,
> > > > > > >
> > > > > > > On Tue, 10 Aug 2021 at 13:38, Tom Rini <trini@konsulko.com> wrote:
> > > > > > [snip]
> > > > > > > > I need to take another pass at converting a bunch of symbols, to see
> > > > > > > > where we're at.  Probably the biggest chunk of progress next would be to
> > > > > > > > start converting CONFIG_SYS_xxx to SYS_xxx and moving defines out of
> > > > > > > > config.h and in to something else.  I'm taking a peek at some of the
> > > > > > > > remaining PCI ones now.
> > > > > > >
> > > > > > > How about we set a deadline for this? It has gone on for too long and
> > > > > > > we just need to drop these CONFIGs. It's probably a higher priority
> > > > > > > than a Kconfig change.
> > > > > > >
> > > > > > > I was expecting that the config.h files would go away and we would use
> > > > > > > Kconfig (or DT) for everything. What sort of things don't fit into
> > > > > > > that model?
> > > > > >
> > > > > > Environment is the hard one to move out from config.h and in to, well, I
> > > > >
> > > > > Well you know my views on that :-)
> > > > >
> > > > > http://patchwork.ozlabs.org/project/uboot/patch/1382763695-2849-4-git-send-email-sjg@chromium.org/
> > > > >
> > > > > I still think it makes more sense than #defines and I can resurrect
> > > > > that series if you like.
> > > >
> > > > That might work, yeah.  I just also want to focus on less things in
> > > > progress at once.  That too I think has been part of why everything is
> > > > taking so long.
> > >
> > > OK I'll take a look. Other things in progress I can think of are:
> > >
> > > - drop common.h  (could be done in one series if we just go for it)
> > > - set up issue tracking so we can keep an eye on these things
> >
> > Well, all of the DM migrations that have deadlines, and the there's
> > probably some that don't still.  And the high level size concern about
> 
> Yes, I'm going to send a patch for serial and GPIO.

OK.  It would be good to know then how many platforms are unmigrated.
And I need to cycle back to the list of non-DM migrated boards thread.

> > using DM on platforms where dynamic selection/detection isn't a
> > functional win.
> 
> Assuming it is about 20KB added (plus devicetree), enabling OF_CONTROL
> seems worth it to me. For SPL I would like to see more adoption of
> your of-platdata idea.

20KiB is a lot.  It's not a huge deal when we start talking about some
sort of dynamic detection being useful.  Or "we were passed a dtb by
prior stage, use and pass along".  But there's still new designs where
neither of those are the case.  And tons of legacy designs we still have
users of.

But it's also one of those "not right this minute" topics.

[snip]
> > Ah, yes, so there's two things it does.  One is buildman.  Two is "spit
> > an error out if a defconfig lacks a MAINTAINER entry".  The latter is
> > important but could be its own tool, and then of course looking at
> > MAINTAINER files is how to see what config header a given defconfig
> > uses.
> 
> OK I see. The defconfig file seems to be in the Kconfig now...so I
> suppose I don't fully understand that.

Er, what?

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]