From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from xenotime.net (xenotime.net [66.160.160.81]) by ozlabs.org (Postfix) with SMTP id 2AEA367DCF for ; Sat, 7 Oct 2006 07:33:21 +1000 (EST) Received: from midway.site ([71.117.236.95]) by xenotime.net for ; Fri, 6 Oct 2006 14:33:09 -0700 Date: Fri, 6 Oct 2006 14:34:37 -0700 From: Randy Dunlap To: Judith Lebzelter Subject: Re: [PATCH] Add Kconfig dependency !VT for VIOCONS Message-Id: <20061006143437.f7338860.rdunlap@xenotime.net> In-Reply-To: <20061006200007.GD3684@shell0.pdx.osdl.net> References: <20061006180549.GB3684@shell0.pdx.osdl.net> <20061006200007.GD3684@shell0.pdx.osdl.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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 This is the old virtual console driver for legacy iSeries. You should use the iSeries Hypervisor Virtual Console --- linux-2619-rc1g2.orig/drivers/char/Kconfig +++ linux-2619-rc1g2/drivers/char/Kconfig @@ -7,7 +7,6 @@ menu "Character devices" config VT bool "Virtual terminal" if EMBEDDED select INPUT - default y if !VIOCONS ---help--- If you say Y here, you will get support for terminal devices with display and keyboard devices. These are called "virtual" because you > Thanks, > Judith > > On Fri, Oct 06, 2006 at 11:05:49AM -0700, Judith Lebzelter wrote: > > From: Judith Lebzelter > > > > Add Kconfig dependency !VT for VIOCONS > > > > I would like to avoid this compile error in 'allmodconfig': > > drivers/char/viocons.c:52:2: error: #error You must turn off CONFIG_VT to use CONFIG_VIOCONS > > > > Signed-off-by: Judith Lebzelter > > --- > > > > Index: linux/arch/powerpc/platforms/iseries/Kconfig > > =================================================================== > > --- linux.orig/arch/powerpc/platforms/iseries/Kconfig 2006-10-05 09:35:09.000000000 -0700 > > +++ linux/arch/powerpc/platforms/iseries/Kconfig 2006-10-06 10:30:19.333425703 -0700 > > @@ -4,6 +4,7 @@ > > > > config VIOCONS > > tristate "iSeries Virtual Console Support (Obsolete)" > > + depends on !VT > > help > > This is the old virtual console driver for legacy iSeries. > > You should use the iSeries Hypervisor Virtual Console --- ~Randy