From mboxrd@z Thu Jan  1 00:00:00 1970
From: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Fri, 15 May 2015 00:36:44 +0900
Subject: [U-Boot] [PATCH v3 07/10] moveconfig: Ignore duplicate configs
 when moving
In-Reply-To: <1431556137-8426-7-git-send-email-joe.hershberger@ni.com>
References: <1431364998-2905-1-git-send-email-joe.hershberger@ni.com>
 <1431556137-8426-1-git-send-email-joe.hershberger@ni.com>
 <1431556137-8426-7-git-send-email-joe.hershberger@ni.com>
Message-ID: <CAK7LNARER1pSSpHX4HTB4Dz-kWRBvK5CiEDid7ra=PvGByE3Gg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

2015-05-14 7:28 GMT+09:00 Joe Hershberger <joe.hershberger@ni.com>:
> When moving configs, it is important to know what was defined in the
> config header even if it duplicates the configs coming from Kconfig.
>
> tools/moveconfig.py now defines KCONFIG_IGNORE_DUPLICATES to prevent the
> filtering from happening and selecting wrong values for the defconfig.
>
> Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
>
> ---
>
> Changes in v3:
> -New for version 3
>
> Changes in v2: None
>
>  scripts/Makefile.autoconf | 3 ++-
>  tools/moveconfig.py       | 1 +
>  2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/Makefile.autoconf b/scripts/Makefile.autoconf
> index f054081..36bfa17 100644
> --- a/scripts/Makefile.autoconf
> +++ b/scripts/Makefile.autoconf
> @@ -58,7 +58,8 @@ quiet_cmd_autoconf = GEN     $@
>         $(CPP) $(c_flags) $2 -DDO_DEPS_ONLY -dM $(srctree)/include/common.h > $@.tmp && { \
>                 sed -n -f $(srctree)/tools/scripts/define2mk.sed $@.tmp |               \
>                 while read line; do                                                     \
> -                       if ! grep -q "$${line%=*}=" include/config/auto.conf; then      \
> +                       if [ -n "${KCONFIG_IGNORE_DUPLICATES}" ] ||                     \
> +                          ! grep -q "$${line%=*}=" include/config/auto.conf; then      \
>                                 echo "$$line";                                          \
>                         fi                                                              \
>                 done > $@;                                                              \
> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
> index 9135e1d..97ff597 100755
> --- a/tools/moveconfig.py
> +++ b/tools/moveconfig.py
> @@ -340,6 +340,7 @@ class Slot:
>          cmd = list(self.make_cmd)
>          if self.cross_compile:
>              cmd.append('CROSS_COMPILE=%s' % self.cross_compile)
> +        cmd.append('KCONFIG_IGNORE_DUPLICATES=1')
>          cmd.append('include/autoconf.mk')
>          self.ps = subprocess.Popen(cmd, stdout=self.devnull, stderr=PIPE)
>          self.state = STATE_SILENTOLDCONFIG


This patch is not clear to me.

Isn't it the same concept as u-boot.cfg
instroduced by commit 741e58e0fc8.


-- 
Best Regards
Masahiro Yamada