From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andi Kleen Subject: Re: [PATCH] [2.6.13-mm2] set IBM ThinkPad extras to default n in Kconfig Date: Sat, 10 Sep 2005 12:20:39 +0200 Message-ID: <200509101220.40008.ak@suse.de> References: <20050910094259.GA16051@gollum.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20050910094259.GA16051@gollum.tnic> Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org To: Borislav Petkov Cc: Roman Zippel , "Brown, Len" , akpm@osdl.org, acpi-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org List-Id: linux-acpi@vger.kernel.org On Saturday 10 September 2005 11:42, Borislav Petkov wrote: > On Fri, Sep 09, 2005 at 06:25:00PM +0200, Roman Zippel wrote: > > The best would be to avoid using defaults completely, unless the > > resulting kernel is non-functional (e.g. it doesn't compile or boot). > > So far it's still the responsibility of the user to explicitly turn > > everything on he needs (at least until we have a functional autoconfig). > > BTW distros are not the only users, from them I would expect how to > > configure a kernel. > > Actually, this sounds pretty sane and IMHO is somehow the biggest common > denominator concerning linux users and their kernel configuration > recreational activities :); but seriously, going all over the menus of > Kbuild and turning everything off is a lot of work compared to turning on > the several things I need on my system. "default m" is also not a good > thing since compiling of unnecessary modules is simply dumb for a system > that's just not going to use them. The new driver dcdbas driver in -git9 seems to have inherited that bad habit too. Grr ... Patch appended. Roman - can you perhaps just forbod default m in Kconfig? I don't think it makes any sense. -Andi Don't set dcdbas driver to default m It's nasty to set random drivers to default m because people who just press enter on make oldconfig get these. Remove the default m Signed-off-by: Andi Kleen Index: linux/drivers/firmware/Kconfig =================================================================== --- linux.orig/drivers/firmware/Kconfig +++ linux/drivers/firmware/Kconfig @@ -71,7 +71,6 @@ config DELL_RBU config DCDBAS tristate "Dell Systems Management Base Driver" depends on X86 || X86_64 - default m help The Dell Systems Management Base Driver provides a sysfs interface for systems management software to perform System Management