[U-Boot] [PATCH v2 01/44] Correct defconfigs using savedefconfig

Masahiro Yamada yamada.masahiro at socionext.com
Thu Sep 8 11:24:23 CEST 2016


2016-09-07 22:05 GMT+09:00 Tom Rini <trini at konsulko.com>:
> On Wed, Sep 07, 2016 at 01:03:23PM +0900, Masahiro Yamada wrote:
>> 2016-09-07 3:01 GMT+09:00 Tom Rini <trini at konsulko.com>:
>> > On Mon, Sep 05, 2016 at 11:47:00AM +0900, Masahiro Yamada wrote:
>> >> 2016-08-30 9:21 GMT+09:00 Simon Glass <sjg at chromium.org>:
>> >> > Update the defconfig files to match their canonical form, as produced by
>> >> > 'make safedefconfig'.
>> >> >
>> >> > This is the result of running 'tools/moveconfig.py -s' on the tree.
>> >> >
>> >> > Signed-off-by: Simon Glass <sjg at chromium.org>
>> >>
>> >> >
>> >> > diff --git a/configs/10m50_defconfig b/configs/10m50_defconfig
>> >> > index 15952af..0e3ad96 100644
>> >> > --- a/configs/10m50_defconfig
>> >> > +++ b/configs/10m50_defconfig
>> >> > @@ -1,7 +1,7 @@
>> >> >  CONFIG_NIOS2=y
>> >> >  CONFIG_SYS_CONFIG_NAME="10m50_devboard"
>> >> > -CONFIG_DM_SERIAL=y
>> >> >  CONFIG_DM_GPIO=y
>> >> > +CONFIG_DM_SERIAL=y
>> >> >  CONFIG_DEFAULT_DEVICE_TREE="10m50_devboard"
>> >> >  CONFIG_FIT=y
>> >> >  CONFIG_OF_BOARD_SETUP=y
>> >> > diff --git a/configs/3c120_defconfig b/configs/3c120_defconfig
>> >> > index b19c956..0c3fbde 100644
>> >> > --- a/configs/3c120_defconfig
>> >> > +++ b/configs/3c120_defconfig
>> >> > @@ -1,7 +1,7 @@
>> >> >  CONFIG_NIOS2=y
>> >> >  CONFIG_SYS_CONFIG_NAME="3c120_devboard"
>> >> > -CONFIG_DM_SERIAL=y
>> >> >  CONFIG_DM_GPIO=y
>> >> > +CONFIG_DM_SERIAL=y
>> >> >  CONFIG_DEFAULT_DEVICE_TREE="3c120_devboard"
>> >> >  CONFIG_FIT=y
>> >> >  CONFIG_OF_BOARD_SETUP=y
>> >>
>> >>
>> >>
>> >> If the following patch is applied
>> >> http://patchwork.ozlabs.org/patch/664076/
>> >> the savedefconfig sync will produce completely different
>> >> results for these defconfigs.
>> >>
>> >>
>> >> The "make savedefconfig" sorts CONFIGs
>> >> in the order as Kconfig entries are parsed.
>> >>
>> >> In other words, the canonical order changes
>> >> every time bare default entries are added/removed
>> >> in board/*/Kconfig.
>> >>
>> >>
>> >> We are just repeating mad churn...
>> >
>> > Yes, but it's churn we have to live with until everything is migrated,
>> > no?
>> >
>>
>> I am OK with adding CONFIG to defconfig files.
>>
>> But I often see CONFIGs moving up/down
>> every time we sync defconfigs,
>> most of them are unrelated to what we want to move.
>>
>>
>> The cause is that people often add
>>   config FOO
>>         default y
>>
>> around in board/*/Kconfig
>> (and eventually I fix it).
>
> OK, fair point, I need to be better about catching those when they sneak
> in.


Anyway, current defconfigs are heavily out of sync,
so we need to sync them at some point.
(maybe right after the upcoming release?)


BTW,
This kind of patch becomes stale soon.
I guess Tom can do global sync any time suitable for him,
instead of letting a huge patch fly in the ML.
Or somebody can request it like
"Hey, I want to work on Kconfig moves, but before that, please sync
the mainline!".



-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list