2004-01-23 20:06:55

by Jean Delvare

[permalink] [raw]
Subject: Message about PCILynx in kernel config

Hi all,

This is a proposed patch to alter the conditions under which the PCILynx
message is displayed while configuring Linux 2.6. The way it is now, a
user with PCI and I2C both disabled will see a message stating that
"PCILynx requires I2C". If the user goes to the I2C section and selects
I2C, then comes back to the IEEE 1394 section, the message will have
disappeared (because I2C isn't missing anymore) but the PCILynx option
will not show (because PCI is still not selected).

I think it might be confusing, so I suggest that the message shouldn't
be displayed at all if PCI isn't selected. And here's a simple patch
that does just this. Built against and tested on Linux 2.6.2-rc1.

Thanks for your attention.

--- linux-2.6.2-rc1/drivers/ieee1394/Kconfig.orig 2004-01-22 10:37:43 +0100
+++ linux-2.6.2-rc1/drivers/ieee1394/Kconfig 2004-01-22 10:43:10 +0100
@@ -52,7 +52,7 @@
depends on IEEE1394

comment "Texas Instruments PCILynx requires I2C"
- depends on IEEE1394 && I2C=n
+ depends on PCI && IEEE1394 && I2C=n

config IEEE1394_PCILYNX
tristate "Texas Instruments PCILynx support"


--
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/


2004-01-23 20:33:54

by Ben Collins

[permalink] [raw]
Subject: Re: Message about PCILynx in kernel config

> I think it might be confusing, so I suggest that the message shouldn't
> be displayed at all if PCI isn't selected. And here's a simple patch
> that does just this. Built against and tested on Linux 2.6.2-rc1.

Applied, thanks.

--
Debian - http://www.debian.org/
Linux 1394 - http://www.linux1394.org/
Subversion - http://subversion.tigris.org/
WatchGuard - http://www.watchguard.com/