[PATCH 2/2] checkpatch.pl: Make CONFIG_IS_ENABLED(CONFIG_*) an error

Simon Glass sjg at chromium.org
Mon Oct 5 03:52:18 CEST 2020


Hi Alper,

On Wed, 30 Sep 2020 at 05:46, Alper Nebi Yasak <alpernebiyasak at gmail.com> wrote:
>
> CONFIG_IS_ENABLED() takes the kconfig name without the CONFIG_ prefix,
> e.g. CONFIG_IS_ENABLED(CLK) for CONFIG_CLK. Make including the prefix
> an error in checkpatch.pl so calls in the wrong format aren't
> accidentally reintroduced.
>
> Signed-off-by: Alper Nebi Yasak <alpernebiyasak at gmail.com>
> ---
>
>  scripts/checkpatch.pl | 6 ++++++
>  1 file changed, 6 insertions(+)
>

This looks good but please add a test to patman too.

Regards,
Simon


More information about the U-Boot mailing list