lkml.org 
[lkml]   [2011]   [Aug]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] kconfig: Introduce IS_ENABLED(), IS_BUILTIN() and IS_MODULE()
    On Fri, Jul 29, 2011 at 03:51:37PM +0200, Michal Marek wrote:
    > Replace the config_is_*() macros with a variant that allows for grepping
    > for usage of CONFIG_* options in the code. Usage:
    >
    > if (IS_ENABLED(CONFIG_NUMA))
    >
    > or
    >
    > #if IS_ENABLED(CONFIG_NUMA)
    >
    > The IS_ENABLED() macro evaluates to 1 if the argument is set (to either 'y'
    > or 'm'), IS_BUILTIN() tests if the option is 'y' and IS_MODULE() test if
    > the option is 'm'. Only boolean and tristate options are supported.
    >
    > Signed-off-by: Michal Marek <mmarek@suse.cz>
    > ---
    >
    > Everybody happy with the above naming?
    Naming is good - has my ack.


    >
    > ---
    > Makefile | 2 +-
    > include/linux/kconfig.h | 32 ++++++++++++++++++++
    > scripts/kconfig/confdata.c | 71 ++++++++-----------------------------------
    > 3 files changed, 47 insertions(+), 58 deletions(-)
    > create mode 100644 include/linux/kconfig.h
    >
    > diff --git a/Makefile b/Makefile
    > index afb8e0d..bd7629b 100644
    > --- a/Makefile
    > +++ b/Makefile
    > @@ -360,7 +360,7 @@ CFLAGS_GCOV = -fprofile-arcs -ftest-coverage
    > LINUXINCLUDE := -I$(srctree)/arch/$(hdr-arch)/include \
    > -Iarch/$(hdr-arch)/include/generated -Iinclude \
    > $(if $(KBUILD_SRC), -I$(srctree)/include) \
    > - -include include/generated/autoconf.h
    > + -include $(srctree)/include/linux/kconfig.h

    If we do it like this then how about adding this to autoconf.h and avoid the extra file,
    and the extra flag to gcc?

    Sam


    \
     
     \ /
      Last update: 2011-08-02 19:37    [W:3.395 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site