[U-Boot] [PATCH] bootcount: Migrate CONFIG_SYS_BOOTCOUNT_ADDR

[Alex Kiernan]

On Sun, Feb 25, 2018 at 1:23 PM, Tom Rini <trini at konsulko.com> wrote:
> On Sun, Feb 25, 2018 at 11:29:30AM +0100, Lukasz Majewski wrote:
>> Hi Tom,
>>
>> > Migrate the users of CONFIG_SYS_BOOTCOUNT_ADDR to Kconfig.  We can
>> > provide a default for BOOTCOUNT_AM33XX as that's a specific part of
>> > the RTC in use.  We can also provide a default for ARCH_LS1043A and
>> > ARCH_LS1021A as they had been previously calculated and their symbols
>> > are in Kconfig.  In the case of other QE-enabled platforms, they are
>> > not so we must update the relevant defconfig files.  The previous
>> > default only applied in some cases, even on a specific SoC family.
>>
>> Thanks for your patch.
>>
>> I suppose, that it is a last puzzle to convert BOOTCOUNT to Kconfig.
>>
>> Please correct me if I'm wrong, but it shall be applied on top of
>> Alex's series:
>>
>> http://patchwork.ozlabs.org/cover/874539/
>
> Correct.  I tested that and the only part I wasn't 100% confident in
> after transition was the addresses.
>

Having looked at it both ways (migrate CONFIG_SYS_BOOTCOUNT_ADDR into
Kconfig, migrate everything out of the CONFIG_ namespace) they both
seem to have downsides... either you have magic numbers in Kconfig or
you have configuration littered through include files. It feels like
it wants some piece of machinery we don't really have today.

-- 
Alex Kiernan