[U-Boot] [U-Boot, 3/4] tools: moveconfig: warn loudly if moved option has no entry in Kconfig
Tom Rini
trini at konsulko.com
Wed Sep 7 19:58:05 CEST 2016
On Mon, Aug 22, 2016 at 10:18:21PM +0900, Masahiro Yamada wrote:
> Currently, the tool gives up moving an option quietly if its entry
> was not found in Kconfig.
>
> If the option is not defined in the config header in the first
> place, it is no problem (as the Kconfig entry may have been hidden
> by reasonable "depends on").
>
> However, if the option is defined in the config header, the missing
> Kconfig entry is a sign of possible behavior change. It is highly
> recommended to manually check if the option has been moved as
> expected. In this case, let's add "suspicious" in the log and
> change the log color (if --color option is given) to make it stand
> out.
>
> This was suggested by Tom in [1].
>
> [1] http://lists.denx.de/pipermail/u-boot/2016-July/261988.html
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> Suggested-by: Tom Rini <trini at konsulko.com>
> Reviewed-by: Tom Rini <trini at konsulko.com>
Applied to u-boot/master, thanks!
--
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/b1200012/attachment.sig>
More information about the U-Boot
mailing list