[patch] fix compile warning and initialization of static tmsisa

From: Adrian Bunk (bunk@fs.tum.de)
Date: Thu Dec 05 2002 - 20:11:21 EST


Hi Jochen,

as already discussed some months ago the patch below contains the
follosing fixes for the static compile of tmsisa:
- fixes a "`portlist' defined but not used" compile time warning by
  moving portlist below an #ifdef MODULE
- call tms_isa_probe in Space.c for intialization

The patch compiles against 2.5.50 and applies against 2.4.20.

Please comment on whether it's correct or not and if it's correct
please apply.

TIA
Adrian

--- l/drivers/net/Space.c.old Mon Dec 17 00:44:29 2001
+++ l/drivers/net/Space.c Mon Dec 17 00:49:00 2001
@@ -540,6 +540,7 @@
 #ifdef CONFIG_TR
 /* Token-ring device probe */
 extern int ibmtr_probe(struct net_device *);
+extern int tms_isa_probe(struct net_device *dev);
 extern int smctr_probe(struct net_device *);

 static int
@@ -548,6 +549,9 @@
     if (1
 #ifdef CONFIG_IBMTR
         && ibmtr_probe(dev)
+#endif
+#ifdef CONFIG_TMSISA
+ && tms_isa_probe(dev)
 #endif
 #ifdef CONFIG_SMCTR
         && smctr_probe(dev)

--- l/drivers/net/tokenring/tmsisa.c.old 2002-12-06 01:51:02.000000000 +0100
+++ l/drivers/net/tokenring/tmsisa.c 2002-12-06 01:54:58.000000000 +0100
@@ -39,12 +39,6 @@
 
 #define TMS_ISA_IO_EXTENT 32
 
-/* A zero-terminated list of I/O addresses to be probed. */
-static unsigned int portlist[] __initdata = {
- 0x0A20, 0x1A20, 0x0B20, 0x1B20, 0x0980, 0x1980, 0x0900, 0x1900,// SK
- 0
-};
-
 /* A zero-terminated list of IRQs to be probed.
  * Used again after initial probe for sktr_chipset_init, called from sktr_open.
  */
@@ -367,6 +361,12 @@
 
 #define ISATR_MAX_ADAPTERS 3
 
+/* A zero-terminated list of I/O addresses to be probed. */
+static unsigned int portlist[] __initdata = {
+ 0x0A20, 0x1A20, 0x0B20, 0x1B20, 0x0980, 0x1980, 0x0900, 0x1900,// SK
+ 0
+};
+
 static int io[ISATR_MAX_ADAPTERS];
 static int irq[ISATR_MAX_ADAPTERS];
 static int dma[ISATR_MAX_ADAPTERS];
-
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/



This archive was generated by hypermail 2b29 : Sat Dec 07 2002 - 22:00:25 EST