diff -urN /mnt/disk/linux/drivers/net/ac3200.c /linux/drivers/net/ac3200.c
--- /mnt/disk/linux/drivers/net/ac3200.c	Fri Oct 20 22:03:22 2000
+++ /linux/drivers/net/ac3200.c	Sun Oct 22 18:45:59 2000
@@ -116,14 +116,20 @@
 
 static int __init ac_probe1(int ioaddr, struct net_device *dev)
 {
-	int i;
+	int i, retval;
 
-	if (inb_p(ioaddr + AC_ID_PORT) == 0xff) 
+	if (!request_region(ioaddr, AC_IO_EXTENT, "ac3200"))
 		return -ENODEV;
 
-	if (inl(ioaddr + AC_ID_PORT) != AC_EISA_ID)
-		return -ENODEV;
+	if (inb_p(ioaddr + AC_ID_PORT) == 0xff) {
+		retval = -ENODEV;
+		goto out;
+	}
 
+	if (inl(ioaddr + AC_ID_PORT) != AC_EISA_ID) {
+		retval = -ENODEV;
+		goto out;
+	}
 
 #ifndef final_version
 	printk(KERN_DEBUG "AC3200 ethercard configuration register is %#02x,"
@@ -132,15 +138,6 @@
 		   inb(ioaddr + AC_ID_PORT + 2), inb(ioaddr + AC_ID_PORT + 3));
 #endif
 
-	/* We should have a "dev" from Space.c or the static module table. */
-	if (dev == NULL) {
-		printk("ac3200.c: Passed a NULL device.\n");
-		dev = init_etherdev(0, 0);
-
-		if (!dev)
-			return -ENOMEM;
-	}
-
 	printk("AC3200 in EISA slot %d, node", ioaddr/0x1000);
 	for(i = 0; i < 6; i++)
 		printk(" %02x", dev->dev_addr[i] = inb(ioaddr + AC_SA_PROM + i));
@@ -151,14 +148,16 @@
 		|| inb(ioaddr + AC_SA_PROM + 1) != AC_ADDR1
 		|| inb(ioaddr + AC_SA_PROM + 2) != AC_ADDR2 ) {
 		printk(", not found (invalid prefix).\n");
-		return -ENODEV;
+		retval = -ENODEV;
+		goto out;
 	}
 #endif
 
 	/* Allocate dev->priv and fill in 8390 specific dev fields. */
 	if (ethdev_init(dev)) {
 		printk (", unable to allocate memory for dev->priv.\n");
-		return -ENOMEM;
+		retval = -ENOMEM;
+		goto out;
 	}
 
 	/* Assign and allocate the interrupt now. */
@@ -172,15 +171,12 @@
 
 	if (request_irq(dev->irq, ei_interrupt, 0, "ac3200", dev)) {
 		printk (" nothing! Unable to get IRQ %d.\n", dev->irq);
-		kfree(dev->priv);
-		dev->priv = NULL;
-		return -EAGAIN;
+		retval = -EAGAIN;
+		goto out1;
 	}
 
 	printk(" IRQ %d, %s port\n", dev->irq, port_name[dev->if_port]);
 
-	request_region(ioaddr, AC_IO_EXTENT, "ac3200");
-
 	dev->base_addr = ioaddr;
 
 #ifdef notyet
@@ -210,20 +206,16 @@
 			printk(KERN_CRIT "ac3200.c: Use EISA SCU to set card memory below 1MB,\n");
 			printk(KERN_CRIT "ac3200.c: or to an address above 0x%lx.\n", virt_to_bus(high_memory));
 			printk(KERN_CRIT "ac3200.c: Driver NOT installed.\n");
-			free_irq(dev->irq, dev);
-			kfree(dev->priv);
-			dev->priv = NULL;
-			return -EINVAL;
+			retval = -EINVAL;
+			goto out2;
 		}
 		dev->mem_start = (unsigned long)ioremap(dev->mem_start, AC_STOP_PG*0x100);
 		if (dev->mem_start == 0) {
 			printk(KERN_ERR "ac3200.c: Unable to remap card memory above 1MB !!\n");
 			printk(KERN_ERR "ac3200.c: Try using EISA SCU to set memory below 1MB.\n");
 			printk(KERN_ERR "ac3200.c: Driver NOT installed.\n");
-			free_irq(dev->irq, dev);
-			kfree(dev->priv);
-			dev->priv = NULL;
-			return -EAGAIN;
+			retval = -EINVAL;
+			goto out2;
 		}
 		ei_status.reg0 = 1;	/* Use as remap flag */
 		printk("ac3200.c: remapped %dkB card memory to virtual address %#lx\n",
@@ -252,6 +244,14 @@
 	dev->stop = &ac_close_card;
 	NS8390_init(dev, 0);
 	return 0;
+out2:
+	free_irq(dev->irq, dev);
+out1:
+	kfree(dev->priv);
+	dev->priv = NULL;
+out:
+	release_region(ioaddr, AC_IO_EXTENT);
+	return retval;
 }
 
 static int ac_open(struct net_device *dev)
@@ -353,15 +353,15 @@
 	},
 };
 
-static int io[MAX_AC32_CARDS] = { 0, };
-static int irq[MAX_AC32_CARDS]  = { 0, };
-static int mem[MAX_AC32_CARDS] = { 0, };
+static int io[MAX_AC32_CARDS];
+static int irq[MAX_AC32_CARDS];
+static int mem[MAX_AC32_CARDS];
+
 MODULE_PARM(io, "1-" __MODULE_STRING(MAX_AC32_CARDS) "i");
 MODULE_PARM(irq, "1-" __MODULE_STRING(MAX_AC32_CARDS) "i");
 MODULE_PARM(mem, "1-" __MODULE_STRING(MAX_AC32_CARDS) "i");
 
-int
-init_module(void)
+static int ac3200_init(void)
 {
 	int this_dev, found = 0;
 
@@ -389,8 +389,7 @@
 	return 0;
 }
 
-void
-cleanup_module(void)
+static void ac3200_cleanup(void)
 {
 	int this_dev;
 
@@ -409,6 +408,10 @@
 	}
 	unload_8390_module();
 }
+
+module_init(ac3200_init);
+module_exit(ac3200_cleanup);
+
 #endif /* MODULE */
 
 /*