[U-Boot] [PATCH] config_whitelist: remove false-positive CONFIG options

[Masahiro Yamada]

2018-02-21 21:19 GMT+09:00 Tom Rini <trini at konsulko.com>:
> On Wed, Feb 21, 2018 at 04:07:10PM +0900, Masahiro Yamada wrote:
>> 2018-01-06 3:17 GMT+09:00 Masahiro Yamada <yamada.masahiro at socionext.com>:
>> > U-Boot pulled in several core makefiles from Linux.  The following
>> > are not used in U-Boot:
>> >
>> >   - CONFIG_DEBUG_SECTION_MISMATCH
>> >   - CONFIG_FTRACE_MCOUNT_RECORD
>> >   - CONFIG_GCOV_KERNEL
>> >   - CONFIG_GCOV_PROFILE_ALL
>> >   - CONFIG_KASAN
>> >   - CONFIG_MODVERSIONS
>> >
>> > We can remove the unused code if we like. (although it will get the
>> > scripts out of sync)
>> >
>> > CONFIG_BOOM and CONFIG_HIS_DRIVER are just mentioned in the comment
>> > block of scripts/basic/fixdep.c
>> >
>> > CONFIG_SHELL is not configuration, but a variable for internal-use.
>> > It is just a historical misnomer in Kbuild.
>> >
>> > Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>>
>>
>> Tom,
>>
>> Can you check this please?
>
> The problem is that it will get put back in by
> ./scripts/build-whitelist.sh yes?

No.

If you incrementally run build-whitelist.sh,
they will not get back.


If you delete config_whitelist.txt and
build it from scratch, yes, they will get back
with other options we had already deleted.

For example, commit 8bb0f7c0c59e8c5b6f7a6869b802f593739c7ece

But we do not want to do this.



For detailed implementation, see below.

https://github.com/u-boot/u-boot/blob/v2018.03-rc3/scripts/build-whitelist.sh#L50






>  I try and run that at least every
> tag.  Thanks!
>
> --
> Tom
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
>



-- 
Best Regards
Masahiro Yamada