[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 6 Jun 2016 18:03:54 -0400 (EDT)
From: Nicolas Pitre <nicolas.pitre@...aro.org>
To: Masahiro Yamada <yamada.masahiro@...ionext.com>
cc: Linus Torvalds <torvalds@...ux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Michal Simek <michal.simek@...inx.com>,
Rusty Russell <rusty@...tcorp.com.au>,
Michal Marek <mmarek@...e.cz>, Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH] kconfig.h: use already defined macros for IS_REACHABLE()
define
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_##sym))
Here the need is for a macro that returns 1 or 0 whether given
symbol is defined or not, exactly as explained in the comment above the
definition for config_enabled() which in itself has nothing to do with
config.
So maybe config_enabled() should be renamed to __is_defined() or
similar, and then config_enabled() or its replacement defined in termps
of it.
Nicolas
Powered by blists - more mailing lists