Skip to contentSkip to conversation

Message about PCILynx in kernel config

PATCH Applied Jean Delvare 2 participants 2 messages Jan 23, 2004

lore
Share & tools
lore
#0 lore raw

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.

drivers/ieee1394/Kconfig renamed from drivers/ieee1394/Kconfig.orig +1 -1
@@ -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"
Signature · 2 lines
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/
Applied
#1 lore raw
Quoted 3 lines

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.

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