[U-Boot] [PATCH 00/11] zap: Do not use macros that are equivalent to IS_ENABLED(CONFIG_...)

Simon Glass sjg at chromium.org
Sat Dec 19 21:29:47 CET 2015


Hi Masahiro,

On 19 December 2015 at 01:39, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> Hi Simon,
>
>
> 2015-12-19 3:57 GMT+09:00 Simon Glass <sjg at chromium.org>:
>
>>>>  6 files changed, 16 insertions(+), 70 deletions(-)
>>
>> I definitely agree it would be good to drop this. But does it work? I
>> thought IS_ENABLED() only worked for Kconfig options?
>
>
> Ah, I missed that.
>
> IS_ENABLED() only works with macros defined as 1.
>
> Most of include/configs/*.h define macros without values,
> so this series does not work.
>
> I've marked it as Rejected.

I suppose we need to convert them all to Kconfig.

Regards,
Simon


More information about the U-Boot mailing list