[PATCH 2/3] buildman: Detect Kconfig loops

[Tom Rini]

On Thu, Oct 21, 2021 at 10:09:56AM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On Thu, 21 Oct 2021 at 06:13, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Tue, Oct 19, 2021 at 09:43:24PM -0600, Simon Glass wrote:
> >
> > > Hex and int Kconfig options are supposed to have defaults. This is so we
> > > can configure U-Boot without having to enter particular values for the
> > > items that don't have specific values in the board's defconfig file.
> >
> > That's not true.  All symbols that we can make reasonable defaults for
> > get them.  It's just that for boolean n is a reasonable default.
> > int/hex often just need to be entered, period.
> >
> > Everything else is fine however, thanks for digging in to this.
> 
> I got that from there and it made sense as to why we have this problem:
> 
> https://docs.zephyrproject.org/2.4.0/guides/kconfig/tips.html#redundant-defaults

Yeah, we're just in the bad spot because of things not being fully
migrated.

> But I think you are right if we should update the commit message here
> to avoid confusion. Shall I send v2?

Yes please.

> 
> Regards,
> Simon

-- 
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/20211021/d84a9c71/attachment.sig>