Drivers: net: ethernet: 3com: 3c589_cs fixed coding style issues
authorJustin van Wijngaarden <justinvanwijngaarden@gmail.com>
Mon, 17 Feb 2014 22:58:46 +0000 (23:58 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 Feb 2014 21:59:46 +0000 (16:59 -0500)
checkpatch.pl clean-up, from 14 error/ 277 warnings, to 0 errors, 7 warnings

Signed-off-by: Justin van Wijngaarden <justinvanwijngaarden@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/3com/3c589_cs.c

index 5992860a39c97747e64befccd939f2e1f99e78f4..063557e037f21b8b43fa3d1dfe64a19c870d4789 100644 (file)
@@ -1,23 +1,24 @@
-/*======================================================================
-
-    A PCMCIA ethernet driver for the 3com 3c589 card.
-
-    Copyright (C) 1999 David A. Hinds -- dahinds@users.sourceforge.net
-
-    3c589_cs.c 1.162 2001/10/13 00:08:50
-
-    The network driver code is based on Donald Becker's 3c589 code:
-
-    Written 1994 by Donald Becker.
-    Copyright 1993 United States Government as represented by the
-    Director, National Security Agency.  This software may be used and
-    distributed according to the terms of the GNU General Public License,
-    incorporated herein by reference.
-    Donald Becker may be reached at becker@scyld.com
-
-    Updated for 2.5.x by Alan Cox <alan@lxorguk.ukuu.org.uk>
-
-======================================================================*/
+/* ======================================================================
+ *
+ * A PCMCIA ethernet driver for the 3com 3c589 card.
+ *
+ * Copyright (C) 1999 David A. Hinds -- dahinds@users.sourceforge.net
+ *
+ * 3c589_cs.c 1.162 2001/10/13 00:08:50
+ *
+ * The network driver code is based on Donald Becker's 3c589 code:
+ *
+ * Written 1994 by Donald Becker.
+ * Copyright 1993 United States Government as represented by the
+ * Director, National Security Agency.  This software may be used and
+ * distributed according to the terms of the GNU General Public License,
+ * incorporated herein by reference.
+ * Donald Becker may be reached at becker@scyld.com
+ *
+ * Updated for 2.5.x by Alan Cox <alan@lxorguk.ukuu.org.uk>
+ *
+ * ======================================================================
+ */
 
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
 #include <linux/ioport.h>
 #include <linux/bitops.h>
 #include <linux/jiffies.h>
+#include <linux/uaccess.h>
+#include <linux/io.h>
 
 #include <pcmcia/cistpl.h>
 #include <pcmcia/cisreg.h>
 #include <pcmcia/ciscode.h>
 #include <pcmcia/ds.h>
 
-#include <asm/uaccess.h>
-#include <asm/io.h>
 
 /* To minimize the size of the driver source I only define operating
-   constants if they are used several times.  You'll need the manual
-   if you want to understand driver details. */
+ * constants if they are used several times. You'll need the manual
+ * if you want to understand driver details.
+ */
+
 /* Offsets from base I/O address. */
 #define EL3_DATA       0x00
 #define EL3_TIMER      0x0a
@@ -65,7 +68,9 @@
 #define EL3WINDOW(win_num) outw(SelectWindow + (win_num), ioaddr + EL3_CMD)
 
 /* The top five bits written to EL3_CMD are a command, the lower
-   11 bits are the parameter, if applicable. */
+ * 11 bits are the parameter, if applicable.
+ */
+
 enum c509cmd {
        TotalReset      = 0<<11,
        SelectWindow    = 1<<11,
@@ -190,138 +195,142 @@ static const struct net_device_ops el3_netdev_ops = {
 
 static int tc589_probe(struct pcmcia_device *link)
 {
-    struct el3_private *lp;
-    struct net_device *dev;
+       struct el3_private *lp;
+       struct net_device *dev;
 
-    dev_dbg(&link->dev, "3c589_attach()\n");
+       dev_dbg(&link->dev, "3c589_attach()\n");
 
-    /* Create new ethernet device */
-    dev = alloc_etherdev(sizeof(struct el3_private));
-    if (!dev)
-        return -ENOMEM;
-    lp = netdev_priv(dev);
-    link->priv = dev;
-    lp->p_dev = link;
+       /* Create new ethernet device */
+       dev = alloc_etherdev(sizeof(struct el3_private));
+       if (!dev)
+               return -ENOMEM;
+       lp = netdev_priv(dev);
+       link->priv = dev;
+       lp->p_dev = link;
 
-    spin_lock_init(&lp->lock);
-    link->resource[0]->end = 16;
-    link->resource[0]->flags |= IO_DATA_PATH_WIDTH_16;
+       spin_lock_init(&lp->lock);
+       link->resource[0]->end = 16;
+       link->resource[0]->flags |= IO_DATA_PATH_WIDTH_16;
 
-    link->config_flags |= CONF_ENABLE_IRQ;
-    link->config_index = 1;
+       link->config_flags |= CONF_ENABLE_IRQ;
+       link->config_index = 1;
 
-    dev->netdev_ops = &el3_netdev_ops;
-    dev->watchdog_timeo = TX_TIMEOUT;
+       dev->netdev_ops = &el3_netdev_ops;
+       dev->watchdog_timeo = TX_TIMEOUT;
 
-    SET_ETHTOOL_OPS(dev, &netdev_ethtool_ops);
+       SET_ETHTOOL_OPS(dev, &netdev_ethtool_ops);
 
-    return tc589_config(link);
+       return tc589_config(link);
 }
 
 static void tc589_detach(struct pcmcia_device *link)
 {
-    struct net_device *dev = link->priv;
+       struct net_device *dev = link->priv;
 
-    dev_dbg(&link->dev, "3c589_detach\n");
+       dev_dbg(&link->dev, "3c589_detach\n");
 
-    unregister_netdev(dev);
+       unregister_netdev(dev);
 
-    tc589_release(link);
+       tc589_release(link);
 
-    free_netdev(dev);
+       free_netdev(dev);
 } /* tc589_detach */
 
 static int tc589_config(struct pcmcia_device *link)
 {
-    struct net_device *dev = link->priv;
-    __be16 *phys_addr;
-    int ret, i, j, multi = 0, fifo;
-    unsigned int ioaddr;
-    static const char * const ram_split[] = {"5:3", "3:1", "1:1", "3:5"};
-    u8 *buf;
-    size_t len;
-
-    dev_dbg(&link->dev, "3c589_config\n");
-
-    phys_addr = (__be16 *)dev->dev_addr;
-    /* Is this a 3c562? */
-    if (link->manf_id != MANFID_3COM)
-           dev_info(&link->dev, "hmmm, is this really a 3Com card??\n");
-    multi = (link->card_id == PRODID_3COM_3C562);
-
-    link->io_lines = 16;
-
-    /* For the 3c562, the base address must be xx00-xx7f */
-    for (i = j = 0; j < 0x400; j += 0x10) {
-       if (multi && (j & 0x80)) continue;
-       link->resource[0]->start = j ^ 0x300;
-       i = pcmcia_request_io(link);
-       if (i == 0)
-               break;
-    }
-    if (i != 0)
-       goto failed;
-
-    ret = pcmcia_request_irq(link, el3_interrupt);
-    if (ret)
-           goto failed;
-
-    ret = pcmcia_enable_device(link);
-    if (ret)
-           goto failed;
-
-    dev->irq = link->irq;
-    dev->base_addr = link->resource[0]->start;
-    ioaddr = dev->base_addr;
-    EL3WINDOW(0);
-
-    /* The 3c589 has an extra EEPROM for configuration info, including
-       the hardware address.  The 3c562 puts the address in the CIS. */
-    len = pcmcia_get_tuple(link, 0x88, &buf);
-    if (buf && len >= 6) {
-           for (i = 0; i < 3; i++)
-                   phys_addr[i] = htons(le16_to_cpu(buf[i*2]));
-           kfree(buf);
-    } else {
-       kfree(buf); /* 0 < len < 6 */
-       for (i = 0; i < 3; i++)
-           phys_addr[i] = htons(read_eeprom(ioaddr, i));
-       if (phys_addr[0] == htons(0x6060)) {
-           dev_err(&link->dev, "IO port conflict at 0x%03lx-0x%03lx\n",
-                   dev->base_addr, dev->base_addr+15);
-           goto failed;
+       struct net_device *dev = link->priv;
+       __be16 *phys_addr;
+       int ret, i, j, multi = 0, fifo;
+       unsigned int ioaddr;
+       static const char * const ram_split[] = {"5:3", "3:1", "1:1", "3:5"};
+       u8 *buf;
+       size_t len;
+
+       dev_dbg(&link->dev, "3c589_config\n");
+
+       phys_addr = (__be16 *)dev->dev_addr;
+       /* Is this a 3c562? */
+       if (link->manf_id != MANFID_3COM)
+               dev_info(&link->dev, "hmmm, is this really a 3Com card??\n");
+       multi = (link->card_id == PRODID_3COM_3C562);
+
+       link->io_lines = 16;
+
+       /* For the 3c562, the base address must be xx00-xx7f */
+       for (i = j = 0; j < 0x400; j += 0x10) {
+               if (multi && (j & 0x80))
+                       continue;
+               link->resource[0]->start = j ^ 0x300;
+               i = pcmcia_request_io(link);
+               if (i == 0)
+                       break;
        }
-    }
-
-    /* The address and resource configuration register aren't loaded from
-       the EEPROM and *must* be set to 0 and IRQ3 for the PCMCIA version. */
-    outw(0x3f00, ioaddr + 8);
-    fifo = inl(ioaddr);
-
-    /* The if_port symbol can be set when the module is loaded */
-    if ((if_port >= 0) && (if_port <= 3))
-       dev->if_port = if_port;
-    else
-       dev_err(&link->dev, "invalid if_port requested\n");
-
-    SET_NETDEV_DEV(dev, &link->dev);
-
-    if (register_netdev(dev) != 0) {
-           dev_err(&link->dev, "register_netdev() failed\n");
-       goto failed;
-    }
-
-    netdev_info(dev, "3Com 3c%s, io %#3lx, irq %d, hw_addr %pM\n",
-               (multi ? "562" : "589"), dev->base_addr, dev->irq,
-               dev->dev_addr);
-    netdev_info(dev, "  %dK FIFO split %s Rx:Tx, %s xcvr\n",
-               (fifo & 7) ? 32 : 8, ram_split[(fifo >> 16) & 3],
-               if_names[dev->if_port]);
-    return 0;
+       if (i != 0)
+               goto failed;
+
+       ret = pcmcia_request_irq(link, el3_interrupt);
+       if (ret)
+               goto failed;
+
+       ret = pcmcia_enable_device(link);
+       if (ret)
+               goto failed;
+
+       dev->irq = link->irq;
+       dev->base_addr = link->resource[0]->start;
+       ioaddr = dev->base_addr;
+       EL3WINDOW(0);
+
+       /* The 3c589 has an extra EEPROM for configuration info, including
+        * the hardware address.  The 3c562 puts the address in the CIS.
+        */
+       len = pcmcia_get_tuple(link, 0x88, &buf);
+       if (buf && len >= 6) {
+               for (i = 0; i < 3; i++)
+                       phys_addr[i] = htons(le16_to_cpu(buf[i*2]));
+               kfree(buf);
+       } else {
+               kfree(buf); /* 0 < len < 6 */
+               for (i = 0; i < 3; i++)
+                       phys_addr[i] = htons(read_eeprom(ioaddr, i));
+               if (phys_addr[0] == htons(0x6060)) {
+                       dev_err(&link->dev, "IO port conflict at 0x%03lx-0x%03lx\n",
+                                       dev->base_addr, dev->base_addr+15);
+                       goto failed;
+               }
+       }
+
+       /* The address and resource configuration register aren't loaded from
+        * the EEPROM and *must* be set to 0 and IRQ3 for the PCMCIA version.
+        */
+
+       outw(0x3f00, ioaddr + 8);
+       fifo = inl(ioaddr);
+
+       /* The if_port symbol can be set when the module is loaded */
+       if ((if_port >= 0) && (if_port <= 3))
+               dev->if_port = if_port;
+       else
+               dev_err(&link->dev, "invalid if_port requested\n");
+
+       SET_NETDEV_DEV(dev, &link->dev);
+
+       if (register_netdev(dev) != 0) {
+               dev_err(&link->dev, "register_netdev() failed\n");
+               goto failed;
+       }
+
+       netdev_info(dev, "3Com 3c%s, io %#3lx, irq %d, hw_addr %pM\n",
+                       (multi ? "562" : "589"), dev->base_addr, dev->irq,
+                       dev->dev_addr);
+       netdev_info(dev, "  %dK FIFO split %s Rx:Tx, %s xcvr\n",
+                       (fifo & 7) ? 32 : 8, ram_split[(fifo >> 16) & 3],
+                       if_names[dev->if_port]);
+       return 0;
 
 failed:
-    tc589_release(link);
-    return -ENODEV;
+       tc589_release(link);
+       return -ENODEV;
 } /* tc589_config */
 
 static void tc589_release(struct pcmcia_device *link)
@@ -353,113 +362,120 @@ static int tc589_resume(struct pcmcia_device *link)
 
 /*====================================================================*/
 
-/*
-  Use this for commands that may take time to finish
-*/
+/* Use this for commands that may take time to finish */
+
 static void tc589_wait_for_completion(struct net_device *dev, int cmd)
 {
-    int i = 100;
-    outw(cmd, dev->base_addr + EL3_CMD);
-    while (--i > 0)
-       if (!(inw(dev->base_addr + EL3_STATUS) & 0x1000)) break;
-    if (i == 0)
-       netdev_warn(dev, "command 0x%04x did not complete!\n", cmd);
+       int i = 100;
+       outw(cmd, dev->base_addr + EL3_CMD);
+       while (--i > 0)
+               if (!(inw(dev->base_addr + EL3_STATUS) & 0x1000))
+                       break;
+       if (i == 0)
+               netdev_warn(dev, "command 0x%04x did not complete!\n", cmd);
 }
 
-/*
 Read a word from the EEPROM using the regular EEPROM access register.
-  Assume that we are in register window zero.
-*/
+/* Read a word from the EEPROM using the regular EEPROM access register.
* Assume that we are in register window zero.
+ */
+
 static u16 read_eeprom(unsigned int ioaddr, int index)
 {
-    int i;
-    outw(EEPROM_READ + index, ioaddr + 10);
-    /* Reading the eeprom takes 162 us */
-    for (i = 1620; i >= 0; i--)
-       if ((inw(ioaddr + 10) & EEPROM_BUSY) == 0)
-           break;
-    return inw(ioaddr + 12);
+       int i;
+       outw(EEPROM_READ + index, ioaddr + 10);
+       /* Reading the eeprom takes 162 us */
+       for (i = 1620; i >= 0; i--)
+               if ((inw(ioaddr + 10) & EEPROM_BUSY) == 0)
+                       break;
+       return inw(ioaddr + 12);
 }
 
-/*
-  Set transceiver type, perhaps to something other than what the user
-  specified in dev->if_port.
-*/
+/* Set transceiver type, perhaps to something other than what the user
+ * specified in dev->if_port.
+ */
+
 static void tc589_set_xcvr(struct net_device *dev, int if_port)
 {
-    struct el3_private *lp = netdev_priv(dev);
-    unsigned int ioaddr = dev->base_addr;
-
-    EL3WINDOW(0);
-    switch (if_port) {
-    case 0: case 1: outw(0, ioaddr + 6); break;
-    case 2: outw(3<<14, ioaddr + 6); break;
-    case 3: outw(1<<14, ioaddr + 6); break;
-    }
-    /* On PCMCIA, this just turns on the LED */
-    outw((if_port == 2) ? StartCoax : StopCoax, ioaddr + EL3_CMD);
-    /* 10baseT interface, enable link beat and jabber check. */
-    EL3WINDOW(4);
-    outw(MEDIA_LED | ((if_port < 2) ? MEDIA_TP : 0), ioaddr + WN4_MEDIA);
-    EL3WINDOW(1);
-    if (if_port == 2)
-       lp->media_status = ((dev->if_port == 0) ? 0x8000 : 0x4000);
-    else
-       lp->media_status = ((dev->if_port == 0) ? 0x4010 : 0x8800);
+       struct el3_private *lp = netdev_priv(dev);
+       unsigned int ioaddr = dev->base_addr;
+
+       EL3WINDOW(0);
+       switch (if_port) {
+       case 0:
+       case 1:
+               outw(0, ioaddr + 6);
+               break;
+       case 2:
+               outw(3<<14, ioaddr + 6);
+               break;
+       case 3:
+               outw(1<<14, ioaddr + 6);
+               break;
+       }
+       /* On PCMCIA, this just turns on the LED */
+       outw((if_port == 2) ? StartCoax : StopCoax, ioaddr + EL3_CMD);
+       /* 10baseT interface, enable link beat and jabber check. */
+       EL3WINDOW(4);
+       outw(MEDIA_LED | ((if_port < 2) ? MEDIA_TP : 0), ioaddr + WN4_MEDIA);
+       EL3WINDOW(1);
+       if (if_port == 2)
+               lp->media_status = ((dev->if_port == 0) ? 0x8000 : 0x4000);
+       else
+               lp->media_status = ((dev->if_port == 0) ? 0x4010 : 0x8800);
 }
 
 static void dump_status(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    EL3WINDOW(1);
-    netdev_info(dev, "  irq status %04x, rx status %04x, tx status %02x  tx free %04x\n",
-               inw(ioaddr+EL3_STATUS), inw(ioaddr+RX_STATUS),
-               inb(ioaddr+TX_STATUS), inw(ioaddr+TX_FREE));
-    EL3WINDOW(4);
-    netdev_info(dev, "  diagnostics: fifo %04x net %04x ethernet %04x media %04x\n",
-               inw(ioaddr+0x04), inw(ioaddr+0x06), inw(ioaddr+0x08),
-               inw(ioaddr+0x0a));
-    EL3WINDOW(1);
+       unsigned int ioaddr = dev->base_addr;
+       EL3WINDOW(1);
+       netdev_info(dev, "  irq status %04x, rx status %04x, tx status %02x  tx free %04x\n",
+                       inw(ioaddr+EL3_STATUS), inw(ioaddr+RX_STATUS),
+                       inb(ioaddr+TX_STATUS), inw(ioaddr+TX_FREE));
+       EL3WINDOW(4);
+       netdev_info(dev, "  diagnostics: fifo %04x net %04x ethernet %04x media %04x\n",
+                       inw(ioaddr+0x04), inw(ioaddr+0x06), inw(ioaddr+0x08),
+                       inw(ioaddr+0x0a));
+       EL3WINDOW(1);
 }
 
 /* Reset and restore all of the 3c589 registers. */
 static void tc589_reset(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    int i;
-
-    EL3WINDOW(0);
-    outw(0x0001, ioaddr + 4);                  /* Activate board. */
-    outw(0x3f00, ioaddr + 8);                  /* Set the IRQ line. */
-
-    /* Set the station address in window 2. */
-    EL3WINDOW(2);
-    for (i = 0; i < 6; i++)
-       outb(dev->dev_addr[i], ioaddr + i);
-
-    tc589_set_xcvr(dev, dev->if_port);
-
-    /* Switch to the stats window, and clear all stats by reading. */
-    outw(StatsDisable, ioaddr + EL3_CMD);
-    EL3WINDOW(6);
-    for (i = 0; i < 9; i++)
-       inb(ioaddr+i);
-    inw(ioaddr + 10);
-    inw(ioaddr + 12);
-
-    /* Switch to register set 1 for normal use. */
-    EL3WINDOW(1);
-
-    set_rx_mode(dev);
-    outw(StatsEnable, ioaddr + EL3_CMD); /* Turn on statistics. */
-    outw(RxEnable, ioaddr + EL3_CMD); /* Enable the receiver. */
-    outw(TxEnable, ioaddr + EL3_CMD); /* Enable transmitter. */
-    /* Allow status bits to be seen. */
-    outw(SetStatusEnb | 0xff, ioaddr + EL3_CMD);
-    /* Ack all pending events, and set active indicator mask. */
-    outw(AckIntr | IntLatch | TxAvailable | RxEarly | IntReq,
+       unsigned int ioaddr = dev->base_addr;
+       int i;
+
+       EL3WINDOW(0);
+       outw(0x0001, ioaddr + 4);                       /* Activate board. */
+       outw(0x3f00, ioaddr + 8);                       /* Set the IRQ line. */
+
+       /* Set the station address in window 2. */
+       EL3WINDOW(2);
+       for (i = 0; i < 6; i++)
+               outb(dev->dev_addr[i], ioaddr + i);
+
+       tc589_set_xcvr(dev, dev->if_port);
+
+       /* Switch to the stats window, and clear all stats by reading. */
+       outw(StatsDisable, ioaddr + EL3_CMD);
+       EL3WINDOW(6);
+       for (i = 0; i < 9; i++)
+               inb(ioaddr+i);
+       inw(ioaddr + 10);
+       inw(ioaddr + 12);
+
+       /* Switch to register set 1 for normal use. */
+       EL3WINDOW(1);
+
+       set_rx_mode(dev);
+       outw(StatsEnable, ioaddr + EL3_CMD); /* Turn on statistics. */
+       outw(RxEnable, ioaddr + EL3_CMD); /* Enable the receiver. */
+       outw(TxEnable, ioaddr + EL3_CMD); /* Enable transmitter. */
+       /* Allow status bits to be seen. */
+       outw(SetStatusEnb | 0xff, ioaddr + EL3_CMD);
+       /* Ack all pending events, and set active indicator mask. */
+       outw(AckIntr | IntLatch | TxAvailable | RxEarly | IntReq,
         ioaddr + EL3_CMD);
-    outw(SetIntrEnb | IntLatch | TxAvailable | RxComplete | StatsFull
+       outw(SetIntrEnb | IntLatch | TxAvailable | RxComplete | StatsFull
         | AdapterFailure, ioaddr + EL3_CMD);
 }
 
@@ -478,381 +494,406 @@ static const struct ethtool_ops netdev_ethtool_ops = {
 
 static int el3_config(struct net_device *dev, struct ifmap *map)
 {
-    if ((map->port != (u_char)(-1)) && (map->port != dev->if_port)) {
-       if (map->port <= 3) {
-           dev->if_port = map->port;
-           netdev_info(dev, "switched to %s port\n", if_names[dev->if_port]);
-           tc589_set_xcvr(dev, dev->if_port);
-       } else
-           return -EINVAL;
-    }
-    return 0;
+       if ((map->port != (u_char)(-1)) && (map->port != dev->if_port)) {
+               if (map->port <= 3) {
+                       dev->if_port = map->port;
+                       netdev_info(dev, "switched to %s port\n", if_names[dev->if_port]);
+                       tc589_set_xcvr(dev, dev->if_port);
+               } else {
+                       return -EINVAL;
+               }
+       }
+       return 0;
 }
 
 static int el3_open(struct net_device *dev)
 {
-    struct el3_private *lp = netdev_priv(dev);
-    struct pcmcia_device *link = lp->p_dev;
+       struct el3_private *lp = netdev_priv(dev);
+       struct pcmcia_device *link = lp->p_dev;
 
-    if (!pcmcia_dev_present(link))
-       return -ENODEV;
+       if (!pcmcia_dev_present(link))
+               return -ENODEV;
 
-    link->open++;
-    netif_start_queue(dev);
+       link->open++;
+       netif_start_queue(dev);
 
-    tc589_reset(dev);
-    init_timer(&lp->media);
-    lp->media.function = media_check;
-    lp->media.data = (unsigned long) dev;
-    lp->media.expires = jiffies + HZ;
-    add_timer(&lp->media);
+       tc589_reset(dev);
+       init_timer(&lp->media);
+       lp->media.function = media_check;
+       lp->media.data = (unsigned long) dev;
+       lp->media.expires = jiffies + HZ;
+       add_timer(&lp->media);
 
-    dev_dbg(&link->dev, "%s: opened, status %4.4x.\n",
+       dev_dbg(&link->dev, "%s: opened, status %4.4x.\n",
          dev->name, inw(dev->base_addr + EL3_STATUS));
 
-    return 0;
+       return 0;
 }
 
 static void el3_tx_timeout(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-
-    netdev_warn(dev, "Transmit timed out!\n");
-    dump_status(dev);
-    dev->stats.tx_errors++;
-    dev->trans_start = jiffies; /* prevent tx timeout */
-    /* Issue TX_RESET and TX_START commands. */
-    tc589_wait_for_completion(dev, TxReset);
-    outw(TxEnable, ioaddr + EL3_CMD);
-    netif_wake_queue(dev);
+       unsigned int ioaddr = dev->base_addr;
+
+       netdev_warn(dev, "Transmit timed out!\n");
+       dump_status(dev);
+       dev->stats.tx_errors++;
+       dev->trans_start = jiffies; /* prevent tx timeout */
+       /* Issue TX_RESET and TX_START commands. */
+       tc589_wait_for_completion(dev, TxReset);
+       outw(TxEnable, ioaddr + EL3_CMD);
+       netif_wake_queue(dev);
 }
 
 static void pop_tx_status(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    int i;
-
-    /* Clear the Tx status stack. */
-    for (i = 32; i > 0; i--) {
-       u_char tx_status = inb(ioaddr + TX_STATUS);
-       if (!(tx_status & 0x84)) break;
-       /* reset transmitter on jabber error or underrun */
-       if (tx_status & 0x30)
-               tc589_wait_for_completion(dev, TxReset);
-       if (tx_status & 0x38) {
-               netdev_dbg(dev, "transmit error: status 0x%02x\n", tx_status);
-               outw(TxEnable, ioaddr + EL3_CMD);
-               dev->stats.tx_aborted_errors++;
+       unsigned int ioaddr = dev->base_addr;
+       int i;
+
+       /* Clear the Tx status stack. */
+       for (i = 32; i > 0; i--) {
+               u_char tx_status = inb(ioaddr + TX_STATUS);
+               if (!(tx_status & 0x84))
+                       break;
+               /* reset transmitter on jabber error or underrun */
+               if (tx_status & 0x30)
+                       tc589_wait_for_completion(dev, TxReset);
+               if (tx_status & 0x38) {
+                       netdev_dbg(dev, "transmit error: status 0x%02x\n", tx_status);
+                       outw(TxEnable, ioaddr + EL3_CMD);
+                       dev->stats.tx_aborted_errors++;
+               }
+               outb(0x00, ioaddr + TX_STATUS); /* Pop the status stack. */
        }
-       outb(0x00, ioaddr + TX_STATUS); /* Pop the status stack. */
-    }
 }
 
 static netdev_tx_t el3_start_xmit(struct sk_buff *skb,
                                        struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    struct el3_private *priv = netdev_priv(dev);
-    unsigned long flags;
+       unsigned int ioaddr = dev->base_addr;
+       struct el3_private *priv = netdev_priv(dev);
+       unsigned long flags;
 
-    netdev_dbg(dev, "el3_start_xmit(length = %ld) called, status %4.4x.\n",
+       netdev_dbg(dev, "el3_start_xmit(length = %ld) called, status %4.4x.\n",
               (long)skb->len, inw(ioaddr + EL3_STATUS));
 
-    spin_lock_irqsave(&priv->lock, flags);
+       spin_lock_irqsave(&priv->lock, flags);
 
-    dev->stats.tx_bytes += skb->len;
+       dev->stats.tx_bytes += skb->len;
 
-    /* Put out the doubleword header... */
-    outw(skb->len, ioaddr + TX_FIFO);
-    outw(0x00, ioaddr + TX_FIFO);
-    /* ... and the packet rounded to a doubleword. */
-    outsl(ioaddr + TX_FIFO, skb->data, (skb->len + 3) >> 2);
+       /* Put out the doubleword header... */
+       outw(skb->len, ioaddr + TX_FIFO);
+       outw(0x00, ioaddr + TX_FIFO);
+       /* ... and the packet rounded to a doubleword. */
+       outsl(ioaddr + TX_FIFO, skb->data, (skb->len + 3) >> 2);
 
-    if (inw(ioaddr + TX_FREE) <= 1536) {
-       netif_stop_queue(dev);
-       /* Interrupt us when the FIFO has room for max-sized packet. */
-       outw(SetTxThreshold + 1536, ioaddr + EL3_CMD);
-    }
+       if (inw(ioaddr + TX_FREE) <= 1536) {
+               netif_stop_queue(dev);
+               /* Interrupt us when the FIFO has room for max-sized packet. */
+               outw(SetTxThreshold + 1536, ioaddr + EL3_CMD);
+       }
 
-    pop_tx_status(dev);
-    spin_unlock_irqrestore(&priv->lock, flags);
-    dev_kfree_skb(skb);
+       pop_tx_status(dev);
+       spin_unlock_irqrestore(&priv->lock, flags);
+       dev_kfree_skb(skb);
 
-    return NETDEV_TX_OK;
+       return NETDEV_TX_OK;
 }
 
 /* The EL3 interrupt handler. */
 static irqreturn_t el3_interrupt(int irq, void *dev_id)
 {
-    struct net_device *dev = (struct net_device *) dev_id;
-    struct el3_private *lp = netdev_priv(dev);
-    unsigned int ioaddr;
-    __u16 status;
-    int i = 0, handled = 1;
+       struct net_device *dev = (struct net_device *) dev_id;
+       struct el3_private *lp = netdev_priv(dev);
+       unsigned int ioaddr;
+       __u16 status;
+       int i = 0, handled = 1;
 
-    if (!netif_device_present(dev))
-       return IRQ_NONE;
+       if (!netif_device_present(dev))
+               return IRQ_NONE;
 
-    ioaddr = dev->base_addr;
+       ioaddr = dev->base_addr;
 
-    netdev_dbg(dev, "interrupt, status %4.4x.\n", inw(ioaddr + EL3_STATUS));
+       netdev_dbg(dev, "interrupt, status %4.4x.\n", inw(ioaddr + EL3_STATUS));
 
-    spin_lock(&lp->lock);
-    while ((status = inw(ioaddr + EL3_STATUS)) &
+       spin_lock(&lp->lock);
+       while ((status = inw(ioaddr + EL3_STATUS)) &
        (IntLatch | RxComplete | StatsFull)) {
-       if ((status & 0xe000) != 0x2000) {
-               netdev_dbg(dev, "interrupt from dead card\n");
-               handled = 0;
-               break;
-       }
-       if (status & RxComplete)
-               el3_rx(dev);
-       if (status & TxAvailable) {
-               netdev_dbg(dev, "    TX room bit was handled.\n");
-               /* There's room in the FIFO for a full-sized packet. */
-               outw(AckIntr | TxAvailable, ioaddr + EL3_CMD);
-               netif_wake_queue(dev);
-       }
-       if (status & TxComplete)
-               pop_tx_status(dev);
-       if (status & (AdapterFailure | RxEarly | StatsFull)) {
-           /* Handle all uncommon interrupts. */
-           if (status & StatsFull)             /* Empty statistics. */
-               update_stats(dev);
-           if (status & RxEarly) {             /* Rx early is unused. */
-               el3_rx(dev);
-               outw(AckIntr | RxEarly, ioaddr + EL3_CMD);
-           }
-           if (status & AdapterFailure) {
-               u16 fifo_diag;
-               EL3WINDOW(4);
-               fifo_diag = inw(ioaddr + 4);
-               EL3WINDOW(1);
-               netdev_warn(dev, "adapter failure, FIFO diagnostic register %04x.\n",
+               if ((status & 0xe000) != 0x2000) {
+                       netdev_dbg(dev, "interrupt from dead card\n");
+                       handled = 0;
+                       break;
+               }
+               if (status & RxComplete)
+                       el3_rx(dev);
+               if (status & TxAvailable) {
+                       netdev_dbg(dev, "    TX room bit was handled.\n");
+                       /* There's room in the FIFO for a full-sized packet. */
+                       outw(AckIntr | TxAvailable, ioaddr + EL3_CMD);
+                       netif_wake_queue(dev);
+               }
+               if (status & TxComplete)
+                       pop_tx_status(dev);
+               if (status & (AdapterFailure | RxEarly | StatsFull)) {
+                       /* Handle all uncommon interrupts. */
+                       if (status & StatsFull)         /* Empty statistics. */
+                               update_stats(dev);
+                       if (status & RxEarly) {
+                               /* Rx early is unused. */
+                               el3_rx(dev);
+                               outw(AckIntr | RxEarly, ioaddr + EL3_CMD);
+                       }
+                       if (status & AdapterFailure) {
+                               u16 fifo_diag;
+                               EL3WINDOW(4);
+                               fifo_diag = inw(ioaddr + 4);
+                               EL3WINDOW(1);
+                               netdev_warn(dev, "adapter failure, FIFO diagnostic register %04x.\n",
                            fifo_diag);
-               if (fifo_diag & 0x0400) {
-                   /* Tx overrun */
-                   tc589_wait_for_completion(dev, TxReset);
-                   outw(TxEnable, ioaddr + EL3_CMD);
+                               if (fifo_diag & 0x0400) {
+                                       /* Tx overrun */
+                                       tc589_wait_for_completion(dev, TxReset);
+                                       outw(TxEnable, ioaddr + EL3_CMD);
+                               }
+                               if (fifo_diag & 0x2000) {
+                                       /* Rx underrun */
+                                       tc589_wait_for_completion(dev, RxReset);
+                                       set_rx_mode(dev);
+                                       outw(RxEnable, ioaddr + EL3_CMD);
+                               }
+                               outw(AckIntr | AdapterFailure, ioaddr + EL3_CMD);
+                       }
                }
-               if (fifo_diag & 0x2000) {
-                   /* Rx underrun */
-                   tc589_wait_for_completion(dev, RxReset);
-                   set_rx_mode(dev);
-                   outw(RxEnable, ioaddr + EL3_CMD);
+               if (++i > 10) {
+                       netdev_err(dev, "infinite loop in interrupt, status %4.4x.\n",
+                                       status);
+                       /* Clear all interrupts */
+                       outw(AckIntr | 0xFF, ioaddr + EL3_CMD);
+                       break;
                }
-               outw(AckIntr | AdapterFailure, ioaddr + EL3_CMD);
-           }
+               /* Acknowledge the IRQ. */
+               outw(AckIntr | IntReq | IntLatch, ioaddr + EL3_CMD);
        }
-       if (++i > 10) {
-               netdev_err(dev, "infinite loop in interrupt, status %4.4x.\n",
-                          status);
-               /* Clear all interrupts */
-               outw(AckIntr | 0xFF, ioaddr + EL3_CMD);
-               break;
-       }
-       /* Acknowledge the IRQ. */
-       outw(AckIntr | IntReq | IntLatch, ioaddr + EL3_CMD);
-    }
-    lp->last_irq = jiffies;
-    spin_unlock(&lp->lock);
-    netdev_dbg(dev, "exiting interrupt, status %4.4x.\n",
-              inw(ioaddr + EL3_STATUS));
-    return IRQ_RETVAL(handled);
+       lp->last_irq = jiffies;
+       spin_unlock(&lp->lock);
+       netdev_dbg(dev, "exiting interrupt, status %4.4x.\n",
+                       inw(ioaddr + EL3_STATUS));
+       return IRQ_RETVAL(handled);
 }
 
 static void media_check(unsigned long arg)
 {
-    struct net_device *dev = (struct net_device *)(arg);
-    struct el3_private *lp = netdev_priv(dev);
-    unsigned int ioaddr = dev->base_addr;
-    u16 media, errs;
-    unsigned long flags;
+       struct net_device *dev = (struct net_device *)(arg);
+       struct el3_private *lp = netdev_priv(dev);
+       unsigned int ioaddr = dev->base_addr;
+       u16 media, errs;
+       unsigned long flags;
 
-    if (!netif_device_present(dev)) goto reschedule;
+       if (!netif_device_present(dev))
+               goto reschedule;
 
-    /* Check for pending interrupt with expired latency timer: with
-       this, we can limp along even if the interrupt is blocked */
-    if ((inw(ioaddr + EL3_STATUS) & IntLatch) &&
+       /* Check for pending interrupt with expired latency timer: with
+        * this, we can limp along even if the interrupt is blocked
+        */
+       if ((inw(ioaddr + EL3_STATUS) & IntLatch) &&
        (inb(ioaddr + EL3_TIMER) == 0xff)) {
-       if (!lp->fast_poll)
-               netdev_warn(dev, "interrupt(s) dropped!\n");
-
-       local_irq_save(flags);
-       el3_interrupt(dev->irq, dev);
-       local_irq_restore(flags);
-
-       lp->fast_poll = HZ;
-    }
-    if (lp->fast_poll) {
-       lp->fast_poll--;
-       lp->media.expires = jiffies + HZ/100;
-       add_timer(&lp->media);
-       return;
-    }
-
-    /* lp->lock guards the EL3 window. Window should always be 1 except
-       when the lock is held */
-    spin_lock_irqsave(&lp->lock, flags);
-    EL3WINDOW(4);
-    media = inw(ioaddr+WN4_MEDIA) & 0xc810;
-
-    /* Ignore collisions unless we've had no irq's recently */
-    if (time_before(jiffies, lp->last_irq + HZ)) {
-       media &= ~0x0010;
-    } else {
-       /* Try harder to detect carrier errors */
-       EL3WINDOW(6);
-       outw(StatsDisable, ioaddr + EL3_CMD);
-       errs = inb(ioaddr + 0);
-       outw(StatsEnable, ioaddr + EL3_CMD);
-       dev->stats.tx_carrier_errors += errs;
-       if (errs || (lp->media_status & 0x0010)) media |= 0x0010;
-    }
+               if (!lp->fast_poll)
+                       netdev_warn(dev, "interrupt(s) dropped!\n");
+
+               local_irq_save(flags);
+               el3_interrupt(dev->irq, dev);
+               local_irq_restore(flags);
+
+               lp->fast_poll = HZ;
+       }
+       if (lp->fast_poll) {
+               lp->fast_poll--;
+               lp->media.expires = jiffies + HZ/100;
+               add_timer(&lp->media);
+               return;
+       }
+
+       /* lp->lock guards the EL3 window. Window should always be 1 except
+        * when the lock is held
+        */
+
+       spin_lock_irqsave(&lp->lock, flags);
+       EL3WINDOW(4);
+       media = inw(ioaddr+WN4_MEDIA) & 0xc810;
+
+       /* Ignore collisions unless we've had no irq's recently */
+       if (time_before(jiffies, lp->last_irq + HZ)) {
+               media &= ~0x0010;
+       } else {
+               /* Try harder to detect carrier errors */
+               EL3WINDOW(6);
+               outw(StatsDisable, ioaddr + EL3_CMD);
+               errs = inb(ioaddr + 0);
+               outw(StatsEnable, ioaddr + EL3_CMD);
+               dev->stats.tx_carrier_errors += errs;
+               if (errs || (lp->media_status & 0x0010))
+                       media |= 0x0010;
+       }
 
-    if (media != lp->media_status) {
-       if ((media & lp->media_status & 0x8000) &&
-           ((lp->media_status ^ media) & 0x0800))
+       if (media != lp->media_status) {
+               if ((media & lp->media_status & 0x8000) &&
+                               ((lp->media_status ^ media) & 0x0800))
                netdev_info(dev, "%s link beat\n",
-                           (lp->media_status & 0x0800 ? "lost" : "found"));
-       else if ((media & lp->media_status & 0x4000) &&
+                               (lp->media_status & 0x0800 ? "lost" : "found"));
+               else if ((media & lp->media_status & 0x4000) &&
                 ((lp->media_status ^ media) & 0x0010))
                netdev_info(dev, "coax cable %s\n",
-                           (lp->media_status & 0x0010 ? "ok" : "problem"));
-       if (dev->if_port == 0) {
-           if (media & 0x8000) {
-               if (media & 0x0800)
-                       netdev_info(dev, "flipped to 10baseT\n");
-               else
+                               (lp->media_status & 0x0010 ? "ok" : "problem"));
+               if (dev->if_port == 0) {
+                       if (media & 0x8000) {
+                               if (media & 0x0800)
+                                       netdev_info(dev, "flipped to 10baseT\n");
+                               else
                        tc589_set_xcvr(dev, 2);
-           } else if (media & 0x4000) {
-               if (media & 0x0010)
-                   tc589_set_xcvr(dev, 1);
-               else
-                   netdev_info(dev, "flipped to 10base2\n");
-           }
+                       } else if (media & 0x4000) {
+                               if (media & 0x0010)
+                                       tc589_set_xcvr(dev, 1);
+                               else
+                                       netdev_info(dev, "flipped to 10base2\n");
+                       }
+               }
+               lp->media_status = media;
        }
-       lp->media_status = media;
-    }
 
-    EL3WINDOW(1);
-    spin_unlock_irqrestore(&lp->lock, flags);
+       EL3WINDOW(1);
+       spin_unlock_irqrestore(&lp->lock, flags);
 
 reschedule:
-    lp->media.expires = jiffies + HZ;
-    add_timer(&lp->media);
+       lp->media.expires = jiffies + HZ;
+       add_timer(&lp->media);
 }
 
 static struct net_device_stats *el3_get_stats(struct net_device *dev)
 {
-    struct el3_private *lp = netdev_priv(dev);
-    unsigned long flags;
-    struct pcmcia_device *link = lp->p_dev;
+       struct el3_private *lp = netdev_priv(dev);
+       unsigned long flags;
+       struct pcmcia_device *link = lp->p_dev;
 
-    if (pcmcia_dev_present(link)) {
-       spin_lock_irqsave(&lp->lock, flags);
-       update_stats(dev);
-       spin_unlock_irqrestore(&lp->lock, flags);
-    }
-    return &dev->stats;
+       if (pcmcia_dev_present(link)) {
+               spin_lock_irqsave(&lp->lock, flags);
+               update_stats(dev);
+               spin_unlock_irqrestore(&lp->lock, flags);
+       }
+       return &dev->stats;
 }
 
-/*
-  Update statistics.  We change to register window 6, so this should be run
-  single-threaded if the device is active. This is expected to be a rare
-  operation, and it's simpler for the rest of the driver to assume that
-  window 1 is always valid rather than use a special window-state variable.
-
-  Caller must hold the lock for this
+/* Update statistics.  We change to register window 6, so this should be run
+* single-threaded if the device is active. This is expected to be a rare
+* operation, and it's simpler for the rest of the driver to assume that
+* window 1 is always valid rather than use a special window-state variable.
+*
+* Caller must hold the lock for this
 */
+
 static void update_stats(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-
-    netdev_dbg(dev, "updating the statistics.\n");
-    /* Turn off statistics updates while reading. */
-    outw(StatsDisable, ioaddr + EL3_CMD);
-    /* Switch to the stats window, and read everything. */
-    EL3WINDOW(6);
-    dev->stats.tx_carrier_errors       += inb(ioaddr + 0);
-    dev->stats.tx_heartbeat_errors     += inb(ioaddr + 1);
-    /* Multiple collisions. */         inb(ioaddr + 2);
-    dev->stats.collisions              += inb(ioaddr + 3);
-    dev->stats.tx_window_errors                += inb(ioaddr + 4);
-    dev->stats.rx_fifo_errors          += inb(ioaddr + 5);
-    dev->stats.tx_packets              += inb(ioaddr + 6);
-    /* Rx packets   */                 inb(ioaddr + 7);
-    /* Tx deferrals */                 inb(ioaddr + 8);
-    /* Rx octets */                    inw(ioaddr + 10);
-    /* Tx octets */                    inw(ioaddr + 12);
-
-    /* Back to window 1, and turn statistics back on. */
-    EL3WINDOW(1);
-    outw(StatsEnable, ioaddr + EL3_CMD);
+       unsigned int ioaddr = dev->base_addr;
+
+       netdev_dbg(dev, "updating the statistics.\n");
+       /* Turn off statistics updates while reading. */
+       outw(StatsDisable, ioaddr + EL3_CMD);
+       /* Switch to the stats window, and read everything. */
+       EL3WINDOW(6);
+       dev->stats.tx_carrier_errors    += inb(ioaddr + 0);
+       dev->stats.tx_heartbeat_errors  += inb(ioaddr + 1);
+       /* Multiple collisions. */
+       inb(ioaddr + 2);
+       dev->stats.collisions           += inb(ioaddr + 3);
+       dev->stats.tx_window_errors             += inb(ioaddr + 4);
+       dev->stats.rx_fifo_errors               += inb(ioaddr + 5);
+       dev->stats.tx_packets           += inb(ioaddr + 6);
+       /* Rx packets   */
+       inb(ioaddr + 7);
+       /* Tx deferrals */
+       inb(ioaddr + 8);
+       /* Rx octets */
+       inw(ioaddr + 10);
+       /* Tx octets */
+       inw(ioaddr + 12);
+
+       /* Back to window 1, and turn statistics back on. */
+       EL3WINDOW(1);
+       outw(StatsEnable, ioaddr + EL3_CMD);
 }
 
 static int el3_rx(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    int worklimit = 32;
-    short rx_status;
+       unsigned int ioaddr = dev->base_addr;
+       int worklimit = 32;
+       short rx_status;
 
-    netdev_dbg(dev, "in rx_packet(), status %4.4x, rx_status %4.4x.\n",
+       netdev_dbg(dev, "in rx_packet(), status %4.4x, rx_status %4.4x.\n",
               inw(ioaddr+EL3_STATUS), inw(ioaddr+RX_STATUS));
-    while (!((rx_status = inw(ioaddr + RX_STATUS)) & 0x8000) &&
+       while (!((rx_status = inw(ioaddr + RX_STATUS)) & 0x8000) &&
                    worklimit > 0) {
-       worklimit--;
-       if (rx_status & 0x4000) { /* Error, update stats. */
-           short error = rx_status & 0x3800;
-           dev->stats.rx_errors++;
-           switch (error) {
-           case 0x0000:        dev->stats.rx_over_errors++; break;
-           case 0x0800:        dev->stats.rx_length_errors++; break;
-           case 0x1000:        dev->stats.rx_frame_errors++; break;
-           case 0x1800:        dev->stats.rx_length_errors++; break;
-           case 0x2000:        dev->stats.rx_frame_errors++; break;
-           case 0x2800:        dev->stats.rx_crc_errors++; break;
-           }
-       } else {
-           short pkt_len = rx_status & 0x7ff;
-           struct sk_buff *skb;
-
-           skb = netdev_alloc_skb(dev, pkt_len + 5);
-
-           netdev_dbg(dev, "    Receiving packet size %d status %4.4x.\n",
+               worklimit--;
+               if (rx_status & 0x4000) { /* Error, update stats. */
+                       short error = rx_status & 0x3800;
+                       dev->stats.rx_errors++;
+                       switch (error) {
+                       case 0x0000:
+                               dev->stats.rx_over_errors++;
+                               break;
+                       case 0x0800:
+                               dev->stats.rx_length_errors++;
+                               break;
+                       case 0x1000:
+                               dev->stats.rx_frame_errors++;
+                               break;
+                       case 0x1800:
+                               dev->stats.rx_length_errors++;
+                               break;
+                       case 0x2000:
+                               dev->stats.rx_frame_errors++;
+                               break;
+                       case 0x2800:
+                               dev->stats.rx_crc_errors++;
+                               break;
+                       }
+               } else {
+                       short pkt_len = rx_status & 0x7ff;
+                       struct sk_buff *skb;
+
+                       skb = netdev_alloc_skb(dev, pkt_len + 5);
+
+                       netdev_dbg(dev, "    Receiving packet size %d status %4.4x.\n",
                       pkt_len, rx_status);
-           if (skb != NULL) {
-               skb_reserve(skb, 2);
-               insl(ioaddr+RX_FIFO, skb_put(skb, pkt_len),
+                       if (skb != NULL) {
+                               skb_reserve(skb, 2);
+                               insl(ioaddr+RX_FIFO, skb_put(skb, pkt_len),
                        (pkt_len+3)>>2);
-               skb->protocol = eth_type_trans(skb, dev);
-               netif_rx(skb);
-               dev->stats.rx_packets++;
-               dev->stats.rx_bytes += pkt_len;
-           } else {
-               netdev_dbg(dev, "couldn't allocate a sk_buff of size %d.\n",
+                               skb->protocol = eth_type_trans(skb, dev);
+                               netif_rx(skb);
+                               dev->stats.rx_packets++;
+                               dev->stats.rx_bytes += pkt_len;
+                       } else {
+                               netdev_dbg(dev, "couldn't allocate a sk_buff of size %d.\n",
                           pkt_len);
-               dev->stats.rx_dropped++;
-           }
+                               dev->stats.rx_dropped++;
+                       }
+               }
+               /* Pop the top of the Rx FIFO */
+               tc589_wait_for_completion(dev, RxDiscard);
        }
-       /* Pop the top of the Rx FIFO */
-       tc589_wait_for_completion(dev, RxDiscard);
-    }
-    if (worklimit == 0)
-       netdev_warn(dev, "too much work in el3_rx!\n");
-    return 0;
+       if (worklimit == 0)
+               netdev_warn(dev, "too much work in el3_rx!\n");
+       return 0;
 }
 
 static void set_rx_mode(struct net_device *dev)
 {
-    unsigned int ioaddr = dev->base_addr;
-    u16 opts = SetRxFilter | RxStation | RxBroadcast;
-
-    if (dev->flags & IFF_PROMISC)
-       opts |= RxMulticast | RxProm;
-    else if (!netdev_mc_empty(dev) || (dev->flags & IFF_ALLMULTI))
-       opts |= RxMulticast;
-    outw(opts, ioaddr + EL3_CMD);
+       unsigned int ioaddr = dev->base_addr;
+       u16 opts = SetRxFilter | RxStation | RxBroadcast;
+
+       if (dev->flags & IFF_PROMISC)
+               opts |= RxMulticast | RxProm;
+       else if (!netdev_mc_empty(dev) || (dev->flags & IFF_ALLMULTI))
+               opts |= RxMulticast;
+       outw(opts, ioaddr + EL3_CMD);
 }
 
 static void set_multicast_list(struct net_device *dev)
@@ -867,44 +908,44 @@ static void set_multicast_list(struct net_device *dev)
 
 static int el3_close(struct net_device *dev)
 {
-    struct el3_private *lp = netdev_priv(dev);
-    struct pcmcia_device *link = lp->p_dev;
-    unsigned int ioaddr = dev->base_addr;
-
-    dev_dbg(&link->dev, "%s: shutting down ethercard.\n", dev->name);
+       struct el3_private *lp = netdev_priv(dev);
+       struct pcmcia_device *link = lp->p_dev;
+       unsigned int ioaddr = dev->base_addr;
+
+       dev_dbg(&link->dev, "%s: shutting down ethercard.\n", dev->name);
+
+       if (pcmcia_dev_present(link)) {
+               /* Turn off statistics ASAP.  We update dev->stats below. */
+               outw(StatsDisable, ioaddr + EL3_CMD);
+
+               /* Disable the receiver and transmitter. */
+               outw(RxDisable, ioaddr + EL3_CMD);
+               outw(TxDisable, ioaddr + EL3_CMD);
+
+               if (dev->if_port == 2)
+                       /* Turn off thinnet power.  Green! */
+                       outw(StopCoax, ioaddr + EL3_CMD);
+               else if (dev->if_port == 1) {
+                       /* Disable link beat and jabber */
+                       EL3WINDOW(4);
+                       outw(0, ioaddr + WN4_MEDIA);
+               }
 
-    if (pcmcia_dev_present(link)) {
-       /* Turn off statistics ASAP.  We update dev->stats below. */
-       outw(StatsDisable, ioaddr + EL3_CMD);
+               /* Switching back to window 0 disables the IRQ. */
+               EL3WINDOW(0);
+               /* But we explicitly zero the IRQ line select anyway. */
+               outw(0x0f00, ioaddr + WN0_IRQ);
 
-       /* Disable the receiver and transmitter. */
-       outw(RxDisable, ioaddr + EL3_CMD);
-       outw(TxDisable, ioaddr + EL3_CMD);
-
-       if (dev->if_port == 2)
-           /* Turn off thinnet power.  Green! */
-           outw(StopCoax, ioaddr + EL3_CMD);
-       else if (dev->if_port == 1) {
-           /* Disable link beat and jabber */
-           EL3WINDOW(4);
-           outw(0, ioaddr + WN4_MEDIA);
+               /* Check if the card still exists */
+               if ((inw(ioaddr+EL3_STATUS) & 0xe000) == 0x2000)
+                       update_stats(dev);
        }
 
-       /* Switching back to window 0 disables the IRQ. */
-       EL3WINDOW(0);
-       /* But we explicitly zero the IRQ line select anyway. */
-       outw(0x0f00, ioaddr + WN0_IRQ);
-
-       /* Check if the card still exists */
-       if ((inw(ioaddr+EL3_STATUS) & 0xe000) == 0x2000)
-           update_stats(dev);
-    }
-
-    link->open--;
-    netif_stop_queue(dev);
-    del_timer_sync(&lp->media);
+       link->open--;
+       netif_stop_queue(dev);
+       del_timer_sync(&lp->media);
 
-    return 0;
+       return 0;
 }
 
 static const struct pcmcia_device_id tc589_ids[] = {