From: Judith Lebzelter <redacted>
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 <redacted>
---
Index: linux/arch/powerpc/platforms/iseries/Kconfig
===================================================================
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?
Thanks,
Judith
On Fri, Oct 06, 2006 at 11:05:49AM -0700, Judith Lebzelter wrote:
quoted hunk
From: Judith Lebzelter <redacted>
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 <redacted>
---
Index: linux/arch/powerpc/platforms/iseries/Kconfig
===================================================================
From: Randy Dunlap <hidden> Date: 2006-10-06 21:33:19
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelpThisistheoldvirtualconsoledriverforlegacyiSeries.YoushouldusetheiSeriesHypervisorVirtualConsole---linux-2619-rc1g2.orig/drivers/char/Kconfig+++linux-2619-rc1g2/drivers/char/Kconfig
@@ -7,7 +7,6 @@ menu "Character devices"configVTbool"Virtual terminal"ifEMBEDDEDselectINPUT-defaultyif!VIOCONS---help---IfyousayYhere,youwillgetsupportforterminaldeviceswithdisplayandkeyboarddevices.Thesearecalled"virtual"becauseyou
Thanks,
Judith
On Fri, Oct 06, 2006 at 11:05:49AM -0700, Judith Lebzelter wrote:
quoted
From: Judith Lebzelter <redacted>
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 <redacted>
---
Index: linux/arch/powerpc/platforms/iseries/Kconfig
===================================================================
From: Adrian Bunk <hidden> Date: 2006-10-10 04:55:38
On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote:
quoted hunk
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
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.
---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
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
From: Randy Dunlap <hidden> Date: 2006-10-10 16:08:41
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:
quoted
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
config VT
bool "Virtual terminal" if EMBEDDED
With this dependency on EMBEDDED, you could as well simply remove
VIOCONS...
quoted
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.
quoted
---help---
If you say Y here, you will get support for terminal devices with
display and keyboard devices. These are called "virtual" because you
...
From: Adrian Bunk <hidden> Date: 2006-10-11 03:56:33
On Tue, Oct 10, 2006 at 09:10:04AM -0700, Randy Dunlap wrote:
On Tue, 10 Oct 2006 06:55:34 +0200 Adrian Bunk wrote:
quoted
On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote:
quoted
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
config VT
bool "Virtual terminal" if EMBEDDED
With this dependency on EMBEDDED, you could as well simply remove
VIOCONS...
quoted
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.
With iseries_defconfig, CONFIG_VT is not 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.
allmodconfig turns on CONFIG_EMBEDDED.
~Randy
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
From: Randy Dunlap <hidden> Date: 2006-10-11 04:07:34
On Wed, 11 Oct 2006 05:56:29 +0200 Adrian Bunk wrote:
On Tue, Oct 10, 2006 at 09:10:04AM -0700, Randy Dunlap wrote:
quoted
On Tue, 10 Oct 2006 06:55:34 +0200 Adrian Bunk wrote:
quoted
On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote:
quoted
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
config VT
bool "Virtual terminal" if EMBEDDED
With this dependency on EMBEDDED, you could as well simply remove
VIOCONS...
quoted
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.
With iseries_defconfig, CONFIG_VT is not enabled.
quoted
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.
allmodconfig turns on CONFIG_EMBEDDED.
So am I doing this incorrectly?
$ make ARCH=powerpc iseries_defconfig
$editor .config
CONFIG_VT=y
...
#
# iSeries device drivers
#
CONFIG_VIODASD=y
CONFIG_VIOCD=m
CONFIG_VIOTAPE=m
CONFIG_VIOPATH=y
---
~Randy
From: Adrian Bunk <hidden> Date: 2006-10-11 04:30:13
On Tue, Oct 10, 2006 at 09:08:58PM -0700, Randy Dunlap wrote:
On Wed, 11 Oct 2006 05:56:29 +0200 Adrian Bunk wrote:
quoted
On Tue, Oct 10, 2006 at 09:10:04AM -0700, Randy Dunlap wrote:
quoted
On Tue, 10 Oct 2006 06:55:34 +0200 Adrian Bunk wrote:
quoted
On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote:
quoted
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
config VT
bool "Virtual terminal" if EMBEDDED
With this dependency on EMBEDDED, you could as well simply remove
VIOCONS...
quoted
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.
With iseries_defconfig, CONFIG_VT is not enabled.
quoted
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.
allmodconfig turns on CONFIG_EMBEDDED.
So am I doing this incorrectly?
$ make ARCH=powerpc iseries_defconfig
$editor .config
CONFIG_VT=y
...
#
# iSeries device drivers
#
CONFIG_VIODASD=y
CONFIG_VIOCD=m
CONFIG_VIOTAPE=m
CONFIG_VIOPATH=y
CONFIG_VIOCONS is not enabled.
~Randy
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
From: Randy Dunlap <hidden> Date: 2006-10-11 04:33:47
On Wed, 11 Oct 2006 06:30:06 +0200 Adrian Bunk wrote:
On Tue, Oct 10, 2006 at 09:08:58PM -0700, Randy Dunlap wrote:
quoted
On Wed, 11 Oct 2006 05:56:29 +0200 Adrian Bunk wrote:
quoted
On Tue, Oct 10, 2006 at 09:10:04AM -0700, Randy Dunlap wrote:
quoted
On Tue, 10 Oct 2006 06:55:34 +0200 Adrian Bunk wrote:
quoted
On Fri, Oct 06, 2006 at 02:34:37PM -0700, Randy Dunlap wrote:
quoted
On Fri, 6 Oct 2006 13:00:07 -0700 Judith Lebzelter wrote:
quoted
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 <redacted>
Make allmodconfig .config build successfully by making VIOCONS
available only if VT=n. VT need not check VIOCONS.
Signed-off-by: Randy Dunlap <redacted>
---
arch/powerpc/platforms/iseries/Kconfig | 2 +-
drivers/char/Kconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
@@ -3,7 +3,7 @@ menu "iSeries device drivers"depends onPPC_ISERIESconfigVIOCONS-tristate"iSeries Virtual Console Support (Obsolete)"+tristate"iSeries Virtual Console Support (Obsolete)"if!VThelp
...
config VT
bool "Virtual terminal" if EMBEDDED
With this dependency on EMBEDDED, you could as well simply remove
VIOCONS...
quoted
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.
With iseries_defconfig, CONFIG_VT is not enabled.
quoted
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.
allmodconfig turns on CONFIG_EMBEDDED.
So am I doing this incorrectly?
$ make ARCH=powerpc iseries_defconfig
$editor .config
CONFIG_VT=y
...
#
# iSeries device drivers
#
CONFIG_VIODASD=y
CONFIG_VIOCD=m
CONFIG_VIOTAPE=m
CONFIG_VIOPATH=y
CONFIG_VIOCONS is not enabled.
Yep. That was with my patch.
But I see what you mean. Thanks.
---
~Randy