Hi Nicolas,
2016-06-07 7:03 GMT+09:00 Nicolas Pitre :
> On Tue, 7 Jun 2016, Masahiro Yamada wrote:
>
>> Because config_enabled() is so useful,
>> maybe people tend to abuse it.
>>
>> I see one case where config_enabled() is used
>> for a non-config macro.
>>
>> #define __EXPORT_SYMBOL(sym, sec) \
On Tue, 7 Jun 2016, Masahiro Yamada wrote:
> Because config_enabled() is so useful,
> maybe people tend to abuse it.
>
> I see one case where config_enabled() is used
> for a non-config macro.
>
> #define __EXPORT_SYMBOL(sym, sec) \
>__cond_export_sym(sym, sec, config_enabled(__KSYM_
Hi Linus,
2016-06-07 1:36 GMT+09:00 Linus Torvalds :
> Side note:
>
> On Mon, Jun 6, 2016 at 2:28 AM, Masahiro Yamada
> wrote:
>>
>> -#define IS_REACHABLE(option) (config_enabled(option) || \
>> -(config_enabled(option##_MODULE) && config_enabled(MODULE)))
>> +#define IS_REACHABL
Side note:
On Mon, Jun 6, 2016 at 2:28 AM, Masahiro Yamada
wrote:
>
> -#define IS_REACHABLE(option) (config_enabled(option) || \
> -(config_enabled(option##_MODULE) && config_enabled(MODULE)))
> +#define IS_REACHABLE(option) (IS_BUILTIN(option) || \
> +(IS_MODULE(o
For the same reason as commit 02d699f1f464 ("include/linux/kconfig.h:
ese macros which are already defined"), it is better to use macros
IS_BUILTIN() and IS_MODULE() for defining IS_REACHABLE().
Signed-off-by: Masahiro Yamada
---
include/linux/kconfig.h | 4 ++--
1 file changed, 2 insertions(+)
5 matches
Mail list logo