Re: [PATCH][RFC] check for select dependency errors on config load

From: Steven Rostedt <rostedt@goodmis.org>
To: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-kbuild@vger.kernel.org,
	Frederic Weisbecker <fweisbec@gmail.com>,
	Ingo Molnar <mingo@elte.hu>,
	zippel@linux-m68k.org
Subject: Re: [PATCH][RFC] check for select dependency errors on config load
Date: Sat, 21 Feb 2009 15:07:24 -0500 (EST)	[thread overview]
Message-ID: <alpine.DEB.2.00.0902211504200.30069@gandalf.stny.rr.com> (raw)
In-Reply-To: <499F9D25.8010300@oracle.com>

On Fri, 20 Feb 2009, Randy Dunlap wrote:
> 
> EMBEDDED is misnamed.  It means "those who think that they know enough
> to use all of the power of kconfig."
> Some people spell that EXPERT etc.
> Or it means "let me shoot myself in the foot."
> 
> So HOTPLUG, INPUT, FW_LOADER, etc. should not be modified by Aunt Tillie,
> but you and I can play with them.

We really should go and rename it then :-/

> 
> 
> for SERIO:
> 
> config SERIO
> 	tristate "Serial I/O support" if EMBEDDED || !X86
> 	default y
> 
> Experts can modify it.  !X86 can modify it.
> It's usually needed on X86 for keyboard controllers etc.,
> but if one sets EMBEDDED, you can muck up your config and not be
> able to use the keyboard.
> 
> I don't know where the !S390 comes from, but it's not surprising.

Anyway, to avoid having these spit out all the time, I'll see if I can 
make it not warn if the dependency was on EMBEDDED/EXPERT. I'll try to 
work on it when I have time.

Note, the config I used was originally generated by randconfig.

Thanks,

-- Steve