Re: [PATCH v3 2/2] printk: Remove redundant CONFIG_BASE_SMALL

[Masahiro Yamada]

On Mon, Feb 5, 2024 at 8:30 AM Yoann Congal <yoann.congal@xxxxxxxx> wrote:
>
> CONFIG_BASE_SMALL is currently a type int but is only used as a boolean
> equivalent to !CONFIG_BASE_FULL.
>
> So, remove it entirely and move every usage to !CONFIG_BASE_FULL:
> Since CONFIG_BASE_FULL is a type bool config,
> CONFIG_BASE_SMALL == 0 becomes  IS_ENABLED(CONFIG_BASE_FULL) and
> CONFIG_BASE_SMALL != 0 becomes !IS_ENABLED(CONFIG_BASE_FULL).
>
> Signed-off-by: Yoann Congal <yoann.congal@xxxxxxxx>



Reviewed-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>




-- 
Best Regards
Masahiro Yamada