On Thu, May 19, 2016 at 1:51 AM, Masahiro Yamada <yamada.masah...@socionext.com> wrote: > Since 96464badc794 ("moveconfig: Always run savedefconfig on the > moved config"), this tool can not correctly move bool configs with > the default value y. > > The reason is like follows: > > We are supposed to add the config entries in Kconfig for the options > we are moving before running this tool. Otherwise, the moved options > would all disappear during the "make savedefconfig" stage. > > Let's say we want to move CONFIG_FOO to Kconfig, making it an option > with "default y". The first thing we need to do is to create an entry > like follows: > > config FOO > bool "foo" > default y > > So, CONFIG_FOO=y will be set in the .config for every defconfig. > > Commit 7740f653e6b3 ("moveconfig: Ignore duplicate configs when > moving") introduced KCONFIG_IGNORE_DUPLICATES to fix the false > negative (= boards that should be converted to "=y" are misconverted > to "not set") problem. With that commit, the CONFIGs in the .config > are now duplicated to include/autoconf.mk unless they are defined to > a different value or #undef'ed in the board header. It causes false > positive problem this time; boards without #define/#undef CONFIG_FOO > in their header should be converted to "not set", but misconverted > to "=y" actually. > > This commit intends to handle such cases correctly. Anyway, we need > to create a Kconfig entry beforehand to persevere savedefconfig. > But, we do not want to propagate CONFIGs coming from "default y" > to the include/autoconf.mk. To achieve it, the new option --undef > is here! > > We can move bool option with default y in this way: > 1. create an entry in Kconfig > 2. run "tools/moveconfig.py --undef=CONFIG_FOO CONFIG_FOO"
This seems tedious to use. The way I solved this didn't require any extra specification of what was expected to happen. http://lists.denx.de/pipermail/u-boot/2015-June/215724.html > This option can appear multiple times in the command line, so that > we can move multiple options at a time. > > Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot