Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762190AbXETAsT (ORCPT ); Sat, 19 May 2007 20:48:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756529AbXETAsM (ORCPT ); Sat, 19 May 2007 20:48:12 -0400 Received: from einhorn.in-berlin.de ([192.109.42.8]:34531 "EHLO einhorn.in-berlin.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756199AbXETAsL (ORCPT ); Sat, 19 May 2007 20:48:11 -0400 X-Envelope-From: stefanr@s5r6.in-berlin.de Message-ID: <464F9A7B.1030708@s5r6.in-berlin.de> Date: Sun, 20 May 2007 02:46:51 +0200 From: Stefan Richter User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.2) Gecko/20070408 SeaMonkey/1.1.1 MIME-Version: 1.0 To: Satyam Sharma CC: Andrew Morton , Sam Ravnborg , LKML , Roman Zippel , Kumar Gala , Simon Horman , Adrian Bunk Subject: Re: RFC: kconfig select warnings bogus? References: <20070519151523.GA26724@uranus.ravnborg.org> <20070519110944.2a52061e.akpm@linux-foundation.org> In-Reply-To: X-Enigmail-Version: 0.94.1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2922 Lines: 73 Satyam Sharma wrote: > A combined patch for all of those on top of 22-rc2 (adding Signed-off-by's > for original authors too): > > Signed-off-by: Kumar Gala > Signed-off-by: Simon Horman > Signed-off-by: Satyam Sharma [...] > diff -ruNp a/drivers/input/keyboard/Kconfig > b/drivers/input/keyboard/Kconfig > --- a/drivers/input/keyboard/Kconfig 2007-05-20 04:11:55.000000000 +0530 > +++ b/drivers/input/keyboard/Kconfig 2007-05-20 04:24:54.000000000 +0530 > @@ -167,7 +167,6 @@ config KEYBOARD_AMIGA > config KEYBOARD_ATARI > tristate "Atari keyboard" > depends on ATARI > - select ATARI_KBD_CORE > help > Say Y here if you are running Linux on any Atari and have a keyboard > attached. > diff -ruNp a/drivers/input/mouse/Kconfig b/drivers/input/mouse/Kconfig > --- a/drivers/input/mouse/Kconfig 2007-05-20 04:11:55.000000000 +0530 > +++ b/drivers/input/mouse/Kconfig 2007-05-20 04:25:09.000000000 +0530 > @@ -179,7 +179,6 @@ config MOUSE_AMIGA > config MOUSE_ATARI > tristate "Atari mouse" > depends on ATARI > - select ATARI_KBD_CORE > help > Say Y here if you have an Atari and want its native mouse > supported by the kernel. > diff -ruNp a/drivers/macintosh/Kconfig b/drivers/macintosh/Kconfig > --- a/drivers/macintosh/Kconfig 2007-05-20 04:11:55.000000000 +0530 > +++ b/drivers/macintosh/Kconfig 2007-05-20 04:18:57.000000000 +0530 > @@ -113,7 +113,6 @@ config PMAC_SMU > > config PMAC_APM_EMU > tristate "APM emulation" > - select SYS_SUPPORTS_APM_EMULATION > select APM_EMULATION > depends on ADB_PMU && PM > > diff -ruNp a/drivers/net/Kconfig b/drivers/net/Kconfig > --- a/drivers/net/Kconfig 2007-05-20 04:11:56.000000000 +0530 > +++ b/drivers/net/Kconfig 2007-05-20 04:22:15.000000000 +0530 > @@ -2280,7 +2280,6 @@ config GFAR_NAPI > config UCC_GETH > tristate "Freescale QE Gigabit Ethernet" > depends on QUICC_ENGINE > - select UCC_FAST > help > This driver supports the Gigabit Ethernet mode of the QUICC Engine, > which is available on some Freescale SOCs. Adrian already commented on this. Let me just add that these above changes are in so far *totally wrong* as they remove the "depends on XYZ" which is implied by "select XYZ". Remember, "select" is like "depends on", just with the added twist that the 'make snafuconfig' scripts are instructed to (a) not hide the depending option from the user and (b) try to enable the dependency for the user if he tries to enable the depending option. -- Stefan Richter -=====-=-=== -=-= =-=-- http://arcgraph.de/sr/ - 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/