Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932180AbWJJQIl (ORCPT ); Tue, 10 Oct 2006 12:08:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932182AbWJJQIl (ORCPT ); Tue, 10 Oct 2006 12:08:41 -0400 Received: from xenotime.net ([66.160.160.81]:38588 "HELO xenotime.net") by vger.kernel.org with SMTP id S932180AbWJJQIk (ORCPT ); Tue, 10 Oct 2006 12:08:40 -0400 Date: Tue, 10 Oct 2006 09:10:04 -0700 From: Randy Dunlap To: Adrian Bunk Cc: Judith Lebzelter , linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Add Kconfig dependency !VT for VIOCONS Message-Id: <20061010091004.d6d91f8d.rdunlap@xenotime.net> In-Reply-To: <20061010045534.GA3650@stusta.de> References: <20061006180549.GB3684@shell0.pdx.osdl.net> <20061006200007.GD3684@shell0.pdx.osdl.net> <20061006143437.f7338860.rdunlap@xenotime.net> <20061010045534.GA3650@stusta.de> Organization: YPO4 X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.8.10; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2355 Lines: 71 On Tue, 10 Oct 2006 06:55:34 +0200 Adrian Bunk wrote: > On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote: > > On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote: > > > > > Actually, this gets rid of the CONFIG_VIOCONS from my .config, but > > > then I get another warning when I build: > > > > > > Warning! Found recursive dependency: VT VIOCONS VT > > > > > > Can anyone suggest something? > > > > I think that your patch is mostly good/correct, but one more line > > is needed on the VT side: a deletion. > > > > This works for me: > > > > From: Randy Dunlap > > > > Make allmodconfig .config build successfully by making VIOCONS > > available only if VT=n. VT need not check VIOCONS. > > > > Signed-off-by: Randy Dunlap > > --- > > arch/powerpc/platforms/iseries/Kconfig | 2 +- > > drivers/char/Kconfig | 1 - > > 2 files changed, 1 insertion(+), 2 deletions(-) > > > > --- linux-2619-rc1g2.orig/arch/powerpc/platforms/iseries/Kconfig > > +++ linux-2619-rc1g2/arch/powerpc/platforms/iseries/Kconfig > > @@ -3,7 +3,7 @@ menu "iSeries device drivers" > > depends on PPC_ISERIES > > > > config VIOCONS > > - tristate "iSeries Virtual Console Support (Obsolete)" > > + tristate "iSeries Virtual Console Support (Obsolete)" if !VT > > help > >... > > config VT > > bool "Virtual terminal" if EMBEDDED > > With this dependency on EMBEDDED, you could as well simply remove > VIOCONS... > > > select INPUT > > - default y if !VIOCONS > > Removing the "default y" is wrong. Oops, yes, agreed. I don't see a way (using: make ARCH=powerpc iseries_defconfig, which wants to enable VIOCONS) to prevent VT from being enabled so that VIOCONS can be enabled. However, since VIOCONS is marked (Obsolete) and since the powerpc people don't comment on this patch & problem, maybe Judith is the only person who cares. > > ---help--- > > If you say Y here, you will get support for terminal devices with > > display and keyboard devices. These are called "virtual" because you > >... --- ~Randy - 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/