Hi all,

patch for lp486e.c network driver attached. 
Changes: check_region() call removed, added missing __init and __exit.

Best regards.

-- 
Andrey Panin            | Embedded systems software engineer
[EMAIL PROTECTED]        | PGP key: http://www.orbita1.ru/~pazke/AndreyPanin.asc
diff -ur -X /usr/dontdiff linux.vanilla/drivers/net/lp486e.c 
/linux/drivers/net/lp486e.c
--- linux.vanilla/drivers/net/lp486e.c  Mon May 21 23:49:44 2001
+++ /linux/drivers/net/lp486e.c Fri May 25 22:23:22 2001
@@ -971,16 +971,17 @@
        volatile struct i596_private *lp;
        unsigned char eth_addr[6] = { 0, 0xaa, 0, 0, 0, 0 };
        unsigned char *bios;
-       int i,j;
-       static int probed = 0;
+       int i, j;
+       int ret = -ENOMEM;
+       static int probed;
 
        if (probed)
                return -ENODEV;
        probed++;
 
-       if (check_region(IOADDR, LP486E_TOTAL_SIZE)) {
-               printk("lp486e: IO address 0x%x in use\n", IOADDR);
-               return -ENODEV;
+       if (!request_region(IOADDR, LP486E_TOTAL_SIZE, dev->name)) {
+               printk(KERN_ERR "lp486e: IO address 0x%x in use\n", IOADDR);
+               return -EBUSY;
        }
 
        /*
@@ -989,7 +990,7 @@
        dev->mem_start = (unsigned long)
                kmalloc(sizeof(struct i596_private) + 0x0f, GFP_KERNEL);
        if (!dev->mem_start)
-               return -ENOMEM;
+               goto err_out;
        dev->priv = (void *)((dev->mem_start + 0xf) & 0xfffffff0);
        lp = (struct i596_private *) dev->priv;
        memset((void *)lp, 0, sizeof(struct i596_private));
@@ -998,12 +999,10 @@
         * Do we really have this thing?
         */
        if (i596_scp_setup(dev)) {
-               kfree ((void *) dev->mem_start);
-               return -ENODEV;
+               ret = -ENODEV;
+               goto err_out_kfree;
        }
 
-       request_region(IOADDR, LP486E_TOTAL_SIZE, "lp486e");
-
        dev->base_addr = IOADDR;
        dev->irq = IRQ;
 
@@ -1053,6 +1052,12 @@
        i596_port_do(dev, PORT_DUMP, "dump");
 #endif
        return 0;
+
+err_out_kfree:
+       kfree ((void *) dev->mem_start);
+err_out:
+       release_region(IOADDR, LP486E_TOTAL_SIZE);
+       return ret;
 }
 
 static void inline
@@ -1330,7 +1335,7 @@
 static int io = IOADDR;
 static int irq = IRQ;
 
-static int lp486e_init_module(void) {
+static int __init lp486e_init_module(void) {
        struct net_device *dev = &dev_lp486e;
        dev->irq = irq;
        dev->base_addr = io;
@@ -1342,7 +1347,7 @@
        return 0;
 }
 
-static void lp486e_cleanup_module(void) {
+static void __exit lp486e_cleanup_module(void) {
        unregister_netdev(&dev_lp486e);
        kfree((void *)dev_lp486e.mem_start);
        dev_lp486e.priv = NULL;

PGP signature

Reply via email to