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

Tom Rini trini at konsulko.com
Wed Sep 7 15:05:48 CEST 2016


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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160907/3491010c/attachment.sig>


More information about the U-Boot mailing list