Messages in this thread Patch in this message | | | Date | Fri, 1 Jun 2007 22:25:56 +0200 | From | Sam Ravnborg <> | Subject | Re: [RFC 1/4] CONFIG_STABLE: Define it |
| |
> > With the following behaviour: > > DEVEL_KERNEL = 0 in Makefile: > > DEVEL_KERNEL=n in Kconfig > CONFIG_DEVEL_KERNEL is not set in cpp > > DEVEL_KERNEL = 1 in Makefile: > > DEVEL_KERNEL=y in Kconfig > CONFIG_DEVEL_KERNEL is set in cpp > > however the above patch doesn't do this correctly and I got bored of > fiddling with it. Help?
My first try below. It does the kconfig stuff as expected. But the CONFIG_KERNEL_DEVEL is NOT updated unless you touch .config (or in other ways change the config).
I did not see an easy way to fix that - Roman?
I only had to add SYMBOL_VALID as falg to get it working - but it took me a while to figure out. Somehow all the comments describing the data structures for kconfig has got lost.
Sam
diff --git a/Makefile b/Makefile index 562a909..362668c 100644 --- a/Makefile +++ b/Makefile @@ -3,6 +3,7 @@ PATCHLEVEL = 6 SUBLEVEL = 22 EXTRAVERSION = -rc3 NAME = Jeff Thinks I Should Change This, But To What? +KERNEL_DEVEL = # *DOCUMENTATION* # To see a list of typical targets execute "make help" @@ -320,7 +321,7 @@ AFLAGS := -D__ASSEMBLY__ KERNELRELEASE = $(shell cat include/config/kernel.release 2> /dev/null) KERNELVERSION = $(VERSION).$(PATCHLEVEL).$(SUBLEVEL)$(EXTRAVERSION) -export VERSION PATCHLEVEL SUBLEVEL KERNELRELEASE KERNELVERSION +export VERSION PATCHLEVEL SUBLEVEL KERNELRELEASE KERNELVERSION KERNEL_DEVEL export ARCH CONFIG_SHELL HOSTCC HOSTCFLAGS CROSS_COMPILE AS LD CC export CPP AR NM STRIP OBJCOPY OBJDUMP MAKE AWK GENKSYMS PERL UTS_MACHINE export HOSTCXX HOSTCXXFLAGS LDFLAGS_MODULE CHECK CHECKFLAGS diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig index 8770a5d..5373d58 100644 --- a/arch/i386/Kconfig +++ b/arch/i386/Kconfig @@ -91,6 +91,14 @@ source "init/Kconfig" menu "Processor type and features" +config MY_KERNEL_DEVEL + bool "Needs Kernel devel" + depends on KERNEL_DEVEL + +config MY_KERNEL_DEVEL2 + bool "Do not need kernel devel" + depends on !KERNEL_DEVEL + source "kernel/time/Kconfig" config SMP diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c index c35dcc5..fb4d5b8 100644 --- a/scripts/kconfig/symbol.c +++ b/scripts/kconfig/symbol.c @@ -68,6 +68,15 @@ void sym_init(void) if (p) sym_add_default(sym, p); + sym = sym_lookup("KERNEL_DEVEL", 0); + sym->type = S_BOOLEAN; + sym->flags |= SYMBOL_VALID|SYMBOL_AUTO; + p = getenv("KERNEL_DEVEL"); + if (p && atoi(p)) + sym_add_default(sym, "y"); + else + sym_add_default(sym, "n"); + sym = sym_lookup("UNAME_RELEASE", 0); sym->type = S_STRING; sym->flags |= SYMBOL_AUTO; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |