fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git] / drivers / net / sis900.c
index 1a87a82..fb2b530 100644 (file)
@@ -1,14 +1,14 @@
 /* sis900.c: A SiS 900/7016 PCI Fast Ethernet driver for Linux.
-   Copyright 1999 Silicon Integrated System Corporation 
-   Revision:   1.08.06 Sep. 24 2002
-   
+   Copyright 1999 Silicon Integrated System Corporation
+   Revision:   1.08.10 Apr. 2 2006
+
    Modified from the driver which is originally written by Donald Becker.
-   
+
    This software may be used and distributed according to the terms
    of the GNU General Public License (GPL), incorporated herein by reference.
    Drivers based on this skeleton fall under the GPL and must retain
    the authorship (implicit copyright) notice.
-   
+
    References:
    SiS 7016 Fast Ethernet PCI Bus 10/100 Mbps LAN Controller with OnNow Support,
    preliminary Rev. 1.0 Jan. 14, 1998
    preliminary Rev. 1.0 Nov. 10, 1998
    SiS 7014 Single Chip 100BASE-TX/10BASE-T Physical Layer Solution,
    preliminary Rev. 1.0 Jan. 18, 1998
-   http://www.sis.com.tw/support/databook.htm
 
-   Rev 1.08.07 Nov.  2 2003 Daniele Venzano <webvenza@libero.it> add suspend/resume support
+   Rev 1.08.10 Apr.  2 2006 Daniele Venzano add vlan (jumbo packets) support
+   Rev 1.08.09 Sep. 19 2005 Daniele Venzano add Wake on LAN support
+   Rev 1.08.08 Jan. 22 2005 Daniele Venzano use netif_msg for debugging messages
+   Rev 1.08.07 Nov.  2 2003 Daniele Venzano <venza@brownhat.org> add suspend/resume support
    Rev 1.08.06 Sep. 24 2002 Mufasa Yang bug fix for Tx timeout & add SiS963 support
    Rev 1.08.05 Jun.  6 2002 Mufasa Yang bug fix for read_eeprom & Tx descriptor over-boundary
    Rev 1.08.04 Apr. 25 2002 Mufasa Yang <mufasa@sis.com.tw> added SiS962 support
@@ -27,7 +29,7 @@
    Rev 1.08.01 Aug. 25 2001 Hui-Fen Hsu update for 630ET & workaround for ICS1893 PHY
    Rev 1.08.00 Jun. 11 2001 Hui-Fen Hsu workaround for RTL8201 PHY and some bug fix
    Rev 1.07.11 Apr.  2 2001 Hui-Fen Hsu updates PCI drivers to use the new pci_set_dma_mask for kernel 2.4.3
-   Rev 1.07.10 Mar.  1 2001 Hui-Fen Hsu <hfhsu@sis.com.tw> some bug fix & 635M/B support 
+   Rev 1.07.10 Mar.  1 2001 Hui-Fen Hsu <hfhsu@sis.com.tw> some bug fix & 635M/B support
    Rev 1.07.09 Feb.  9 2001 Dave Jones <davej@suse.de> PCI enable cleanup
    Rev 1.07.08 Jan.  8 2001 Lei-Chun Chang added RTL8201 PHY support
    Rev 1.07.07 Nov. 29 2000 Lei-Chun Chang added kernel-doc extractable documentation and 630 workaround fix
@@ -48,6 +50,7 @@
 */
 
 #include <linux/module.h>
+#include <linux/moduleparam.h>
 #include <linux/kernel.h>
 #include <linux/string.h>
 #include <linux/timer.h>
 #include <linux/delay.h>
 #include <linux/ethtool.h>
 #include <linux/crc32.h>
+#include <linux/bitops.h>
+#include <linux/dma-mapping.h>
 
 #include <asm/processor.h>      /* Processor type for cache alignment. */
-#include <asm/bitops.h>
 #include <asm/io.h>
+#include <asm/irq.h>
 #include <asm/uaccess.h>       /* User space memory access functions */
 
 #include "sis900.h"
 
 #define SIS900_MODULE_NAME "sis900"
-#define SIS900_DRV_VERSION "v1.08.07 11/02/2003"
+#define SIS900_DRV_VERSION "v1.08.10 Apr. 2 2006"
 
 static char version[] __devinitdata =
 KERN_INFO "sis900.c: " SIS900_DRV_VERSION "\n";
@@ -81,19 +86,22 @@ KERN_INFO "sis900.c: " SIS900_DRV_VERSION "\n";
 static int max_interrupt_work = 40;
 static int multicast_filter_limit = 128;
 
-#define sis900_debug debug
-static int sis900_debug;
+static int sis900_debug = -1; /* Use SIS900_DEF_MSG as value */
+
+#define SIS900_DEF_MSG \
+       (NETIF_MSG_DRV          | \
+        NETIF_MSG_LINK         | \
+        NETIF_MSG_RX_ERR       | \
+        NETIF_MSG_TX_ERR)
 
 /* Time in jiffies before concluding the transmitter is hung. */
 #define TX_TIMEOUT  (4*HZ)
-/* SiS 900 is capable of 32 bits BM DMA */
-#define SIS900_DMA_MASK 0xffffffff
 
 enum {
        SIS_900 = 0,
        SIS_7016
 };
-static char * card_names[] = {
+static const char * card_names[] = {
        "SiS 900 PCI Fast Ethernet",
        "SiS 7016 PCI Fast Ethernet"
 };
@@ -108,7 +116,7 @@ MODULE_DEVICE_TABLE (pci, sis900_pci_tbl);
 
 static void sis900_read_mode(struct net_device *net_dev, int *speed, int *duplex);
 
-static struct mii_chip_info {
+static const struct mii_chip_info {
        const char * name;
        u16 phy_id0;
        u16 phy_id1;
@@ -120,9 +128,13 @@ static struct mii_chip_info {
 } mii_chip_table[] = {
        { "SiS 900 Internal MII PHY",           0x001d, 0x8000, LAN },
        { "SiS 7014 Physical Layer Solution",   0x0016, 0xf830, LAN },
+       { "SiS 900 on Foxconn 661 7MI",         0x0143, 0xBC70, LAN },
+       { "Altimata AC101LF PHY",               0x0022, 0x5520, LAN },
+       { "ADM 7001 LAN PHY",                   0x002e, 0xcc60, LAN },
        { "AMD 79C901 10BASE-T PHY",            0x0000, 0x6B70, LAN },
        { "AMD 79C901 HomePNA PHY",             0x0000, 0x6B90, HOME},
        { "ICS LAN PHY",                        0x0015, 0xF440, LAN },
+       { "ICS LAN PHY",                        0x0143, 0xBC70, LAN },
        { "NS 83851 PHY",                       0x2000, 0x5C20, MIX },
        { "NS 83847 PHY",                       0x2000, 0x5C30, MIX },
        { "Realtek RTL8201 PHY",                0x0000, 0x8200, LAN },
@@ -140,9 +152,9 @@ struct mii_phy {
 };
 
 typedef struct _BufferDesc {
-       u32     link;
-       u32     cmdsts;
-       u32     bufptr;
+       u32 link;
+       u32 cmdsts;
+       u32 bufptr;
 } BufferDesc;
 
 struct sis900_private {
@@ -154,9 +166,12 @@ struct sis900_private {
        struct mii_phy * mii;
        struct mii_phy * first_mii; /* record the first mii structure */
        unsigned int cur_phy;
+       struct mii_if_info mii_info;
 
        struct timer_list timer; /* Link status detection timer. */
-       u8     autong_complete; /* 1: auto-negotiate complete  */
+       u8 autong_complete; /* 1: auto-negotiate complete  */
+
+       u32 msg_enable;
 
        unsigned int cur_rx, dirty_rx; /* producer/comsumer pointers for Tx/Rx ring */
        unsigned int cur_tx, dirty_tx;
@@ -170,27 +185,30 @@ struct sis900_private {
        dma_addr_t tx_ring_dma;
        dma_addr_t rx_ring_dma;
 
-       unsigned int tx_full;                   /* The Tx queue is full.    */
+       unsigned int tx_full; /* The Tx queue is full. */
        u8 host_bridge_rev;
-       u32 pci_state[16];
+       u8 chipset_rev;
 };
 
 MODULE_AUTHOR("Jim Huang <cmhuang@sis.com.tw>, Ollie Lho <ollie@sis.com.tw>");
 MODULE_DESCRIPTION("SiS 900 PCI Fast Ethernet driver");
 MODULE_LICENSE("GPL");
 
-MODULE_PARM(multicast_filter_limit, "i");
-MODULE_PARM(max_interrupt_work, "i");
-MODULE_PARM(debug, "i");
+module_param(multicast_filter_limit, int, 0444);
+module_param(max_interrupt_work, int, 0444);
+module_param(sis900_debug, int, 0444);
 MODULE_PARM_DESC(multicast_filter_limit, "SiS 900/7016 maximum number of filtered multicast addresses");
 MODULE_PARM_DESC(max_interrupt_work, "SiS 900/7016 maximum events handled per interrupt");
-MODULE_PARM_DESC(debug, "SiS 900/7016 debug level (2-4)");
+MODULE_PARM_DESC(sis900_debug, "SiS 900/7016 bitmapped debugging message level");
 
+#ifdef CONFIG_NET_POLL_CONTROLLER
+static void sis900_poll(struct net_device *dev);
+#endif
 static int sis900_open(struct net_device *net_dev);
 static int sis900_mii_probe (struct net_device * net_dev);
 static void sis900_init_rxfilter (struct net_device * net_dev);
 static u16 read_eeprom(long ioaddr, int location);
-static u16 mdio_read(struct net_device *net_dev, int phy_id, int location);
+static int mdio_read(struct net_device *net_dev, int phy_id, int location);
 static void mdio_write(struct net_device *net_dev, int phy_id, int location, int val);
 static void sis900_timer(unsigned long data);
 static void sis900_check_mode (struct net_device *net_dev, struct mii_phy *mii_phy);
@@ -200,7 +218,7 @@ static void sis900_init_rx_ring(struct net_device *net_dev);
 static int sis900_start_xmit(struct sk_buff *skb, struct net_device *net_dev);
 static int sis900_rx(struct net_device *net_dev);
 static void sis900_finish_xmit (struct net_device *net_dev);
-static irqreturn_t sis900_interrupt(int irq, void *dev_instance, struct pt_regs *regs);
+static irqreturn_t sis900_interrupt(int irq, void *dev_instance);
 static int sis900_close(struct net_device *net_dev);
 static int mii_ioctl(struct net_device *net_dev, struct ifreq *rq, int cmd);
 static struct net_device_stats *sis900_get_stats(struct net_device *net_dev);
@@ -214,12 +232,12 @@ static void sis900_set_capability( struct net_device *net_dev ,struct mii_phy *p
 static u16 sis900_reset_phy(struct net_device *net_dev, int phy_addr);
 static void sis900_auto_negotiate(struct net_device *net_dev, int phy_addr);
 static void sis900_set_mode (long ioaddr, int speed, int duplex);
-static struct ethtool_ops sis900_ethtool_ops;
+static const struct ethtool_ops sis900_ethtool_ops;
 
 /**
  *     sis900_get_mac_addr - Get MAC address for stand alone SiS900 model
  *     @pci_dev: the sis900 pci device
- *     @net_dev: the net device to get address for 
+ *     @net_dev: the net device to get address for
  *
  *     Older SiS900 and friends, use EEPROM to store MAC address.
  *     MAC address is read from read_eeprom() into @net_dev->dev_addr.
@@ -232,10 +250,10 @@ static int __devinit sis900_get_mac_addr(struct pci_dev * pci_dev, struct net_de
        int i;
 
        /* check to see if we have sane EEPROM */
-       signature = (u16) read_eeprom(ioaddr, EEPROMSignature);    
+       signature = (u16) read_eeprom(ioaddr, EEPROMSignature);
        if (signature == 0xffff || signature == 0x0000) {
-               printk (KERN_INFO "%s: Error EERPOM read %x\n", 
-                       net_dev->name, signature);
+               printk (KERN_WARNING "%s: Error EERPOM read %x\n",
+                       pci_name(pci_dev), signature);
                return 0;
        }
 
@@ -249,35 +267,37 @@ static int __devinit sis900_get_mac_addr(struct pci_dev * pci_dev, struct net_de
 /**
  *     sis630e_get_mac_addr - Get MAC address for SiS630E model
  *     @pci_dev: the sis900 pci device
- *     @net_dev: the net device to get address for 
+ *     @net_dev: the net device to get address for
  *
  *     SiS630E model, use APC CMOS RAM to store MAC address.
  *     APC CMOS RAM is accessed through ISA bridge.
  *     MAC address is read into @net_dev->dev_addr.
  */
 
-static int __devinit sis630e_get_mac_addr(struct pci_dev * pci_dev, struct net_device *net_dev)
+static int __devinit sis630e_get_mac_addr(struct pci_dev * pci_dev,
+                                       struct net_device *net_dev)
 {
        struct pci_dev *isa_bridge = NULL;
        u8 reg;
        int i;
 
-       isa_bridge = pci_find_device(PCI_VENDOR_ID_SI, 0x0008, isa_bridge);
+       isa_bridge = pci_get_device(PCI_VENDOR_ID_SI, 0x0008, isa_bridge);
+       if (!isa_bridge)
+               isa_bridge = pci_get_device(PCI_VENDOR_ID_SI, 0x0018, isa_bridge);
        if (!isa_bridge) {
-               isa_bridge = pci_find_device(PCI_VENDOR_ID_SI, 0x0018, isa_bridge);
-               if (!isa_bridge) {
-                       printk("%s: Can not find ISA bridge\n", net_dev->name);
-                       return 0;
-               }
+               printk(KERN_WARNING "%s: Can not find ISA bridge\n",
+                      pci_name(pci_dev));
+               return 0;
        }
        pci_read_config_byte(isa_bridge, 0x48, &reg);
        pci_write_config_byte(isa_bridge, 0x48, reg | 0x40);
 
        for (i = 0; i < 6; i++) {
                outb(0x09 + i, 0x70);
-               ((u8 *)(net_dev->dev_addr))[i] = inb(0x71); 
+               ((u8 *)(net_dev->dev_addr))[i] = inb(0x71);
        }
        pci_write_config_byte(isa_bridge, 0x48, reg & ~0x40);
+       pci_dev_put(isa_bridge);
 
        return 1;
 }
@@ -286,14 +306,15 @@ static int __devinit sis630e_get_mac_addr(struct pci_dev * pci_dev, struct net_d
 /**
  *     sis635_get_mac_addr - Get MAC address for SIS635 model
  *     @pci_dev: the sis900 pci device
- *     @net_dev: the net device to get address for 
+ *     @net_dev: the net device to get address for
  *
  *     SiS635 model, set MAC Reload Bit to load Mac address from APC
- *     to rfdr. rfdr is accessed through rfcr. MAC address is read into 
+ *     to rfdr. rfdr is accessed through rfcr. MAC address is read into
  *     @net_dev->dev_addr.
  */
 
-static int __devinit sis635_get_mac_addr(struct pci_dev * pci_dev, struct net_device *net_dev)
+static int __devinit sis635_get_mac_addr(struct pci_dev * pci_dev,
+                                       struct net_device *net_dev)
 {
        long ioaddr = net_dev->base_addr;
        u32 rfcrSave;
@@ -322,26 +343,27 @@ static int __devinit sis635_get_mac_addr(struct pci_dev * pci_dev, struct net_de
 /**
  *     sis96x_get_mac_addr - Get MAC address for SiS962 or SiS963 model
  *     @pci_dev: the sis900 pci device
- *     @net_dev: the net device to get address for 
+ *     @net_dev: the net device to get address for
  *
- *     SiS962 or SiS963 model, use EEPROM to store MAC address. And EEPROM 
+ *     SiS962 or SiS963 model, use EEPROM to store MAC address. And EEPROM
  *     is shared by
- *     LAN and 1394. When access EEPROM, send EEREQ signal to hardware first 
- *     and wait for EEGNT. If EEGNT is ON, EEPROM is permitted to be access 
+ *     LAN and 1394. When access EEPROM, send EEREQ signal to hardware first
+ *     and wait for EEGNT. If EEGNT is ON, EEPROM is permitted to be access
  *     by LAN, otherwise is not. After MAC address is read from EEPROM, send
- *     EEDONE signal to refuse EEPROM access by LAN. 
- *     The EEPROM map of SiS962 or SiS963 is different to SiS900. 
- *     The signature field in SiS962 or SiS963 spec is meaningless. 
+ *     EEDONE signal to refuse EEPROM access by LAN.
+ *     The EEPROM map of SiS962 or SiS963 is different to SiS900.
+ *     The signature field in SiS962 or SiS963 spec is meaningless.
  *     MAC address is read into @net_dev->dev_addr.
  */
 
-static int __devinit sis96x_get_mac_addr(struct pci_dev * pci_dev, struct net_device *net_dev)
+static int __devinit sis96x_get_mac_addr(struct pci_dev * pci_dev,
+                                       struct net_device *net_dev)
 {
        long ioaddr = net_dev->base_addr;
        long ee_addr = ioaddr + mear;
        u32 waittime = 0;
        int i;
-       
+
        outl(EEREQ, ee_addr);
        while(waittime < 2000) {
                if(inl(ee_addr) & EEGNT) {
@@ -353,7 +375,7 @@ static int __devinit sis96x_get_mac_addr(struct pci_dev * pci_dev, struct net_de
                        outl(EEDONE, ee_addr);
                        return 1;
                } else {
-                       udelay(1);      
+                       udelay(1);
                        waittime ++;
                }
        }
@@ -367,12 +389,13 @@ static int __devinit sis96x_get_mac_addr(struct pci_dev * pci_dev, struct net_de
  *     @pci_id: the pci device ID
  *
  *     Check and probe sis900 net device for @pci_dev.
- *     Get mac address according to the chip revision, 
+ *     Get mac address according to the chip revision,
  *     and assign SiS900-specific entries in the device structure.
  *     ie: sis900_open(), sis900_start_xmit(), sis900_close(), etc.
  */
 
-static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_device_id *pci_id)
+static int __devinit sis900_probe(struct pci_dev *pci_dev,
+                               const struct pci_device_id *pci_id)
 {
        struct sis900_private *sis_priv;
        struct net_device *net_dev;
@@ -381,8 +404,8 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
        void *ring_space;
        long ioaddr;
        int i, ret;
-       u8 revision;
-       char *card_name = card_names[pci_id->driver_data];
+       const char *card_name = card_names[pci_id->driver_data];
+       const char *dev_name = pci_name(pci_dev);
 
 /* when built into the kernel, we only print version if device is found */
 #ifndef MODULE
@@ -394,16 +417,16 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
        /* setup various bits in PCI command register */
        ret = pci_enable_device(pci_dev);
        if(ret) return ret;
-       
-       i = pci_set_dma_mask(pci_dev, SIS900_DMA_MASK);
+
+       i = pci_set_dma_mask(pci_dev, DMA_32BIT_MASK);
        if(i){
                printk(KERN_ERR "sis900.c: architecture does not support"
                        "32bit PCI busmaster DMA\n");
                return i;
        }
-       
+
        pci_set_master(pci_dev);
-       
+
        net_dev = alloc_etherdev(sizeof(struct sis900_private));
        if (!net_dev)
                return -ENOMEM;
@@ -411,7 +434,7 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
        SET_NETDEV_DEV(net_dev, &pci_dev->dev);
 
        /* We do a request_region() to register /proc/ioports info. */
-       ioaddr = pci_resource_start(pci_dev, 0);        
+       ioaddr = pci_resource_start(pci_dev, 0);
        ret = pci_request_regions(pci_dev, "sis900");
        if (ret)
                goto err_out;
@@ -439,7 +462,7 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
        }
        sis_priv->rx_ring = (BufferDesc *)ring_space;
        sis_priv->rx_ring_dma = ring_dma;
-               
+
        /* The SiS900-specific entries in the device structure. */
        net_dev->open = &sis900_open;
        net_dev->hard_start_xmit = &sis900_start_xmit;
@@ -451,43 +474,67 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
        net_dev->tx_timeout = sis900_tx_timeout;
        net_dev->watchdog_timeo = TX_TIMEOUT;
        net_dev->ethtool_ops = &sis900_ethtool_ops;
-       
-       ret = register_netdev(net_dev);
-       if (ret)
-               goto err_unmap_rx;
-               
+
+#ifdef CONFIG_NET_POLL_CONTROLLER
+        net_dev->poll_controller = &sis900_poll;
+#endif
+
+       if (sis900_debug > 0)
+               sis_priv->msg_enable = sis900_debug;
+       else
+               sis_priv->msg_enable = SIS900_DEF_MSG;
+
+       sis_priv->mii_info.dev = net_dev;
+       sis_priv->mii_info.mdio_read = mdio_read;
+       sis_priv->mii_info.mdio_write = mdio_write;
+       sis_priv->mii_info.phy_id_mask = 0x1f;
+       sis_priv->mii_info.reg_num_mask = 0x1f;
+
        /* Get Mac address according to the chip revision */
-       pci_read_config_byte(pci_dev, PCI_CLASS_REVISION, &revision);
-       ret = 0;
+       pci_read_config_byte(pci_dev, PCI_CLASS_REVISION, &(sis_priv->chipset_rev));
+       if(netif_msg_probe(sis_priv))
+               printk(KERN_DEBUG "%s: detected revision %2.2x, "
+                               "trying to get MAC address...\n",
+                               dev_name, sis_priv->chipset_rev);
 
-       if (revision == SIS630E_900_REV)
+       ret = 0;
+       if (sis_priv->chipset_rev == SIS630E_900_REV)
                ret = sis630e_get_mac_addr(pci_dev, net_dev);
-       else if ((revision > 0x81) && (revision <= 0x90) )
+       else if ((sis_priv->chipset_rev > 0x81) && (sis_priv->chipset_rev <= 0x90) )
                ret = sis635_get_mac_addr(pci_dev, net_dev);
-       else if (revision == SIS96x_900_REV)
+       else if (sis_priv->chipset_rev == SIS96x_900_REV)
                ret = sis96x_get_mac_addr(pci_dev, net_dev);
        else
                ret = sis900_get_mac_addr(pci_dev, net_dev);
 
        if (ret == 0) {
+               printk(KERN_WARNING "%s: Cannot read MAC address.\n", dev_name);
                ret = -ENODEV;
-               goto err_out_unregister;
+               goto err_unmap_rx;
        }
-       
+
        /* 630ET : set the mii access mode as software-mode */
-       if (revision == SIS630ET_900_REV)
+       if (sis_priv->chipset_rev == SIS630ET_900_REV)
                outl(ACCESSMODE | inl(ioaddr + cr), ioaddr + cr);
 
        /* probe for mii transceiver */
        if (sis900_mii_probe(net_dev) == 0) {
+               printk(KERN_WARNING "%s: Error probing MII device.\n",
+                      dev_name);
                ret = -ENODEV;
-               goto err_out_unregister;
+               goto err_unmap_rx;
        }
 
        /* save our host bridge revision */
-       dev = pci_find_device(PCI_VENDOR_ID_SI, PCI_DEVICE_ID_SI_630, NULL);
-       if (dev)
+       dev = pci_get_device(PCI_VENDOR_ID_SI, PCI_DEVICE_ID_SI_630, NULL);
+       if (dev) {
                pci_read_config_byte(dev, PCI_CLASS_REVISION, &sis_priv->host_bridge_rev);
+               pci_dev_put(dev);
+       }
+
+       ret = register_netdev(net_dev);
+       if (ret)
+               goto err_unmap_rx;
 
        /* print some information about our NIC */
        printk(KERN_INFO "%s: %s at %#lx, IRQ %d, ", net_dev->name,
@@ -496,10 +543,13 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
                printk("%2.2x:", (u8)net_dev->dev_addr[i]);
        printk("%2.2x.\n", net_dev->dev_addr[i]);
 
+       /* Detect Wake on Lan support */
+       ret = (inl(net_dev->base_addr + CFGPMC) & PMESP) >> 27;
+       if (netif_msg_probe(sis_priv) && (ret & PME_D3C) == 0)
+               printk(KERN_INFO "%s: Wake on LAN only available from suspend to RAM.", net_dev->name);
+
        return 0;
 
- err_out_unregister:
-       unregister_netdev(net_dev);
  err_unmap_rx:
        pci_free_consistent(pci_dev, RX_TOTAL_SIZE, sis_priv->rx_ring,
                sis_priv->rx_ring_dma);
@@ -517,24 +567,24 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
 /**
  *     sis900_mii_probe - Probe MII PHY for sis900
  *     @net_dev: the net device to probe for
- *     
+ *
  *     Search for total of 32 possible mii phy addresses.
  *     Identify and set current phy if found one,
  *     return error if it failed to found.
  */
 
-static int __init sis900_mii_probe (struct net_device * net_dev)
+static int __init sis900_mii_probe(struct net_device * net_dev)
 {
        struct sis900_private * sis_priv = net_dev->priv;
+       const char *dev_name = pci_name(sis_priv->pci_dev);
        u16 poll_bit = MII_STAT_LINK, status = 0;
        unsigned long timeout = jiffies + 5 * HZ;
        int phy_addr;
-       u8 revision;
 
        sis_priv->mii = NULL;
 
        /* search for total of 32 possible mii phy addresses */
-       for (phy_addr = 0; phy_addr < 32; phy_addr++) { 
+       for (phy_addr = 0; phy_addr < 32; phy_addr++) {
                struct mii_phy * mii_phy = NULL;
                u16 mii_status;
                int i;
@@ -543,12 +593,16 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
                for(i = 0; i < 2; i++)
                        mii_status = mdio_read(net_dev, phy_addr, MII_STATUS);
 
-               if (mii_status == 0xffff || mii_status == 0x0000)
-                       /* the mii is not accessible, try next one */
+               if (mii_status == 0xffff || mii_status == 0x0000) {
+                       if (netif_msg_probe(sis_priv))
+                               printk(KERN_DEBUG "%s: MII at address %d"
+                                               " not accessible\n",
+                                               dev_name, phy_addr);
                        continue;
-               
+               }
+
                if ((mii_phy = kmalloc(sizeof(struct mii_phy), GFP_KERNEL)) == NULL) {
-                       printk(KERN_INFO "Cannot allocate mem for struct mii_phy\n");
+                       printk(KERN_WARNING "Cannot allocate mem for struct mii_phy\n");
                        mii_phy = sis_priv->first_mii;
                        while (mii_phy) {
                                struct mii_phy *phy;
@@ -558,9 +612,9 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
                        }
                        return 0;
                }
-               
+
                mii_phy->phy_id0 = mdio_read(net_dev, phy_addr, MII_PHY_ID0);
-               mii_phy->phy_id1 = mdio_read(net_dev, phy_addr, MII_PHY_ID1);           
+               mii_phy->phy_id1 = mdio_read(net_dev, phy_addr, MII_PHY_ID1);
                mii_phy->phy_addr = phy_addr;
                mii_phy->status = mii_status;
                mii_phy->next = sis_priv->mii;
@@ -573,22 +627,24 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
                                mii_phy->phy_types = mii_chip_table[i].phy_types;
                                if (mii_chip_table[i].phy_types == MIX)
                                        mii_phy->phy_types =
-                                               (mii_status & (MII_STAT_CAN_TX_FDX | MII_STAT_CAN_TX)) ? LAN : HOME;
-                               printk(KERN_INFO "%s: %s transceiver found at address %d.\n",
-                                      net_dev->name, mii_chip_table[i].name, phy_addr);
+                                           (mii_status & (MII_STAT_CAN_TX_FDX | MII_STAT_CAN_TX)) ? LAN : HOME;
+                               printk(KERN_INFO "%s: %s transceiver found "
+                                                       "at address %d.\n",
+                                                       dev_name,
+                                                       mii_chip_table[i].name,
+                                                       phy_addr);
                                break;
                        }
-                       
+
                if( !mii_chip_table[i].phy_id1 ) {
                        printk(KERN_INFO "%s: Unknown PHY transceiver found at address %d.\n",
-                              net_dev->name, phy_addr);
+                              dev_name, phy_addr);
                        mii_phy->phy_types = UNKNOWN;
                }
        }
-       
+
        if (sis_priv->mii == NULL) {
-               printk(KERN_INFO "%s: No MII transceivers found!\n",
-                      net_dev->name);
+               printk(KERN_INFO "%s: No MII transceivers found!\n", dev_name);
                return 0;
        }
 
@@ -600,7 +656,7 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
         if ((sis_priv->mii->phy_id0 == 0x001D) &&
            ((sis_priv->mii->phy_id1&0xFFF0) == 0x8000))
                status = sis900_reset_phy(net_dev, sis_priv->cur_phy);
-        
+
         /* workaround for ICS1893 PHY */
         if ((sis_priv->mii->phy_id0 == 0x0015) &&
             ((sis_priv->mii->phy_id1&0xFFF0) == 0xF440))
@@ -612,20 +668,20 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
 
                        poll_bit ^= (mdio_read(net_dev, sis_priv->cur_phy, MII_STATUS) & poll_bit);
                        if (time_after_eq(jiffies, timeout)) {
-                               printk(KERN_WARNING "%s: reset phy and link down now\n", net_dev->name);
+                               printk(KERN_WARNING "%s: reset phy and link down now\n",
+                                      dev_name);
                                return -ETIME;
                        }
                }
        }
 
-       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-       if (revision == SIS630E_900_REV) {
+       if (sis_priv->chipset_rev == SIS630E_900_REV) {
                /* SiS 630E has some bugs on default value of PHY registers */
                mdio_write(net_dev, sis_priv->cur_phy, MII_ANADV, 0x05e1);
                mdio_write(net_dev, sis_priv->cur_phy, MII_CONFIG1, 0x22);
                mdio_write(net_dev, sis_priv->cur_phy, MII_CONFIG2, 0xff00);
                mdio_write(net_dev, sis_priv->cur_phy, MII_MASK, 0xffc0);
-               //mdio_write(net_dev, sis_priv->cur_phy, MII_CONTROL, 0x1000);  
+               //mdio_write(net_dev, sis_priv->cur_phy, MII_CONTROL, 0x1000);
        }
 
        if (sis_priv->mii->status & MII_STAT_LINK)
@@ -648,48 +704,53 @@ static int __init sis900_mii_probe (struct net_device * net_dev)
 static u16 sis900_default_phy(struct net_device * net_dev)
 {
        struct sis900_private * sis_priv = net_dev->priv;
-       struct mii_phy *phy = NULL, *phy_home = NULL, *default_phy = NULL, *phy_lan = NULL;
+       struct mii_phy *phy = NULL, *phy_home = NULL,
+               *default_phy = NULL, *phy_lan = NULL;
        u16 status;
 
-        for( phy=sis_priv->first_mii; phy; phy=phy->next ){
+        for (phy=sis_priv->first_mii; phy; phy=phy->next) {
                status = mdio_read(net_dev, phy->phy_addr, MII_STATUS);
                status = mdio_read(net_dev, phy->phy_addr, MII_STATUS);
 
                /* Link ON & Not select default PHY & not ghost PHY */
-                if ( (status & MII_STAT_LINK) && !default_phy && (phy->phy_types != UNKNOWN) )
+                if ((status & MII_STAT_LINK) && !default_phy &&
+                                       (phy->phy_types != UNKNOWN))
                        default_phy = phy;
-                else{
+                else {
                        status = mdio_read(net_dev, phy->phy_addr, MII_CONTROL);
                        mdio_write(net_dev, phy->phy_addr, MII_CONTROL,
                                status | MII_CNTL_AUTO | MII_CNTL_ISOLATE);
-                       if( phy->phy_types == HOME )
+                       if (phy->phy_types == HOME)
                                phy_home = phy;
-                       else if (phy->phy_types == LAN)
+                       else if(phy->phy_types == LAN)
                                phy_lan = phy;
                 }
        }
 
-       if( !default_phy && phy_home )
+       if (!default_phy && phy_home)
                default_phy = phy_home;
-       else if( !default_phy && phy_lan )
+       else if (!default_phy && phy_lan)
                default_phy = phy_lan;
-       else if ( !default_phy )
+       else if (!default_phy)
                default_phy = sis_priv->first_mii;
 
-       if( sis_priv->mii != default_phy ){
+       if (sis_priv->mii != default_phy) {
                sis_priv->mii = default_phy;
                sis_priv->cur_phy = default_phy->phy_addr;
-               printk(KERN_INFO "%s: Using transceiver found at address %d as default\n", net_dev->name,sis_priv->cur_phy);
+               printk(KERN_INFO "%s: Using transceiver found at address %d as default\n",
+                      pci_name(sis_priv->pci_dev), sis_priv->cur_phy);
        }
-       
+
+       sis_priv->mii_info.phy_id = sis_priv->cur_phy;
+
        status = mdio_read(net_dev, sis_priv->cur_phy, MII_CONTROL);
        status &= (~MII_CNTL_ISOLATE);
 
-       mdio_write(net_dev, sis_priv->cur_phy, MII_CONTROL, status);    
+       mdio_write(net_dev, sis_priv->cur_phy, MII_CONTROL, status);
        status = mdio_read(net_dev, sis_priv->cur_phy, MII_STATUS);
        status = mdio_read(net_dev, sis_priv->cur_phy, MII_STATUS);
 
-       return status;  
+       return status;
 }
 
 
@@ -701,15 +762,15 @@ static u16 sis900_default_phy(struct net_device * net_dev)
  *     Set the media capability of network adapter according to
  *     mii status register. It's necessary before auto-negotiate.
  */
-static void sis900_set_capability( struct net_device *net_dev , struct mii_phy *phy )
+
+static void sis900_set_capability(struct net_device *net_dev, struct mii_phy *phy)
 {
        u16 cap;
        u16 status;
-       
+
        status = mdio_read(net_dev, phy->phy_addr, MII_STATUS);
        status = mdio_read(net_dev, phy->phy_addr, MII_STATUS);
-       
+
        cap = MII_NWAY_CSMA_CD |
                ((phy->status & MII_STAT_CAN_TX_FDX)? MII_NWAY_TX_FDX:0) |
                ((phy->status & MII_STAT_CAN_TX)    ? MII_NWAY_TX:0) |
@@ -809,7 +870,7 @@ static void mdio_reset(long mdio_addr)
  *     Please see SiS7014 or ICS spec
  */
 
-static u16 mdio_read(struct net_device *net_dev, int phy_id, int location)
+static int mdio_read(struct net_device *net_dev, int phy_id, int location)
 {
        long mdio_addr = net_dev->base_addr + mear;
        int mii_cmd = MIIread|(phy_id<<MIIpmdShift)|(location<<MIIregShift);
@@ -852,7 +913,8 @@ static u16 mdio_read(struct net_device *net_dev, int phy_id, int location)
  *     please see SiS7014 or ICS spec
  */
 
-static void mdio_write(struct net_device *net_dev, int phy_id, int location, int value)
+static void mdio_write(struct net_device *net_dev, int phy_id, int location,
+                       int value)
 {
        long mdio_addr = net_dev->base_addr + mear;
        int mii_cmd = MIIwrite|(phy_id<<MIIpmdShift)|(location<<MIIregShift);
@@ -913,10 +975,24 @@ static u16 sis900_reset_phy(struct net_device *net_dev, int phy_addr)
                status = mdio_read(net_dev, phy_addr, MII_STATUS);
 
        mdio_write( net_dev, phy_addr, MII_CONTROL, MII_CNTL_RESET );
-       
+
        return status;
 }
 
+#ifdef CONFIG_NET_POLL_CONTROLLER
+/*
+ * Polling 'interrupt' - used by things like netconsole to send skbs
+ * without having to re-enable interrupts. It's not called while
+ * the interrupt routine is executing.
+*/
+static void sis900_poll(struct net_device *dev)
+{
+       disable_irq(dev->irq);
+       sis900_interrupt(dev->irq, dev);
+       enable_irq(dev->irq);
+}
+#endif
+
 /**
  *     sis900_open - open sis900 device
  *     @net_dev: the net device to open
@@ -930,17 +1006,16 @@ sis900_open(struct net_device *net_dev)
 {
        struct sis900_private *sis_priv = net_dev->priv;
        long ioaddr = net_dev->base_addr;
-       u8 revision;
        int ret;
 
        /* Soft reset the chip. */
        sis900_reset(net_dev);
 
        /* Equalizer workaround Rule */
-       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-       sis630_set_eq(net_dev, revision);
+       sis630_set_eq(net_dev, sis_priv->chipset_rev);
 
-       ret = request_irq(net_dev->irq, &sis900_interrupt, SA_SHIRQ, net_dev->name, net_dev);
+       ret = request_irq(net_dev->irq, &sis900_interrupt, IRQF_SHARED,
+                                               net_dev->name, net_dev);
        if (ret)
                return ret;
 
@@ -985,6 +1060,7 @@ sis900_open(struct net_device *net_dev)
 static void
 sis900_init_rxfilter (struct net_device * net_dev)
 {
+       struct sis900_private *sis_priv = net_dev->priv;
        long ioaddr = net_dev->base_addr;
        u32 rfcrSave;
        u32 i;
@@ -1002,8 +1078,8 @@ sis900_init_rxfilter (struct net_device * net_dev)
                outl((i << RFADDR_shift), ioaddr + rfcr);
                outl(w, ioaddr + rfdr);
 
-               if (sis900_debug > 2) {
-                       printk(KERN_INFO "%s: Receive Filter Addrss[%d]=%x\n",
+               if (netif_msg_hw(sis_priv)) {
+                       printk(KERN_DEBUG "%s: Receive Filter Addrss[%d]=%x\n",
                               net_dev->name, i, inl(ioaddr + rfdr));
                }
        }
@@ -1016,7 +1092,7 @@ sis900_init_rxfilter (struct net_device * net_dev)
  *     sis900_init_tx_ring - Initialize the Tx descriptor ring
  *     @net_dev: the net device to initialize for
  *
- *     Initialize the Tx descriptor ring, 
+ *     Initialize the Tx descriptor ring,
  */
 
 static void
@@ -1040,8 +1116,8 @@ sis900_init_tx_ring(struct net_device *net_dev)
 
        /* load Transmit Descriptor Register */
        outl(sis_priv->tx_ring_dma, ioaddr + txdp);
-       if (sis900_debug > 2)
-               printk(KERN_INFO "%s: TX descriptor register loaded with: %8.8x\n",
+       if (netif_msg_hw(sis_priv))
+               printk(KERN_DEBUG "%s: TX descriptor register loaded with: %8.8x\n",
                       net_dev->name, inl(ioaddr + txdp));
 }
 
@@ -1049,11 +1125,11 @@ sis900_init_tx_ring(struct net_device *net_dev)
  *     sis900_init_rx_ring - Initialize the Rx descriptor ring
  *     @net_dev: the net device to initialize for
  *
- *     Initialize the Rx descriptor ring, 
+ *     Initialize the Rx descriptor ring,
  *     and pre-allocate recevie buffers (socket buffer)
  */
 
-static void 
+static void
 sis900_init_rx_ring(struct net_device *net_dev)
 {
        struct sis900_private *sis_priv = net_dev->priv;
@@ -1088,14 +1164,14 @@ sis900_init_rx_ring(struct net_device *net_dev)
                sis_priv->rx_skbuff[i] = skb;
                sis_priv->rx_ring[i].cmdsts = RX_BUF_SIZE;
                 sis_priv->rx_ring[i].bufptr = pci_map_single(sis_priv->pci_dev,
-                        skb->tail, RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
+                        skb->data, RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
        }
        sis_priv->dirty_rx = (unsigned int) (i - NUM_RX_DESC);
 
        /* load Receive Descriptor Register */
        outl(sis_priv->rx_ring_dma, ioaddr + rxdp);
-       if (sis900_debug > 2)
-               printk(KERN_INFO "%s: RX descriptor register loaded with: %8.8x\n",
+       if (netif_msg_hw(sis_priv))
+               printk(KERN_DEBUG "%s: RX descriptor register loaded with: %8.8x\n",
                       net_dev->name, inl(ioaddr + rxdp));
 }
 
@@ -1137,48 +1213,55 @@ static void sis630_set_eq(struct net_device *net_dev, u8 revision)
                return;
 
        if (netif_carrier_ok(net_dev)) {
-               reg14h=mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
-               mdio_write(net_dev, sis_priv->cur_phy, MII_RESV, (0x2200 | reg14h) & 0xBFFF);
+               reg14h = mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
+               mdio_write(net_dev, sis_priv->cur_phy, MII_RESV,
+                                       (0x2200 | reg14h) & 0xBFFF);
                for (i=0; i < maxcount; i++) {
-                       eq_value=(0x00F8 & mdio_read(net_dev, sis_priv->cur_phy, MII_RESV)) >> 3;
+                       eq_value = (0x00F8 & mdio_read(net_dev,
+                                       sis_priv->cur_phy, MII_RESV)) >> 3;
                        if (i == 0)
                                max_value=min_value=eq_value;
-                       max_value=(eq_value > max_value) ? eq_value : max_value;
-                       min_value=(eq_value < min_value) ? eq_value : min_value;
+                       max_value = (eq_value > max_value) ?
+                                               eq_value : max_value;
+                       min_value = (eq_value < min_value) ?
+                                               eq_value : min_value;
                }
                /* 630E rule to determine the equalizer value */
                if (revision == SIS630E_900_REV || revision == SIS630EA1_900_REV ||
                    revision == SIS630ET_900_REV) {
                        if (max_value < 5)
-                               eq_value=max_value;
+                               eq_value = max_value;
                        else if (max_value >= 5 && max_value < 15)
-                               eq_value=(max_value == min_value) ? max_value+2 : max_value+1;
+                               eq_value = (max_value == min_value) ?
+                                               max_value+2 : max_value+1;
                        else if (max_value >= 15)
-                               eq_value=(max_value == min_value) ? max_value+6 : max_value+5;
+                               eq_value=(max_value == min_value) ?
+                                               max_value+6 : max_value+5;
                }
                /* 630B0&B1 rule to determine the equalizer value */
-               if (revision == SIS630A_900_REV && 
-                   (sis_priv->host_bridge_rev == SIS630B0 || 
+               if (revision == SIS630A_900_REV &&
+                   (sis_priv->host_bridge_rev == SIS630B0 ||
                     sis_priv->host_bridge_rev == SIS630B1)) {
                        if (max_value == 0)
-                               eq_value=3;
+                               eq_value = 3;
                        else
-                               eq_value=(max_value+min_value+1)/2;
+                               eq_value = (max_value + min_value + 1)/2;
                }
                /* write equalizer value and setting */
-               reg14h=mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
-               reg14h=(reg14h & 0xFF07) | ((eq_value << 3) & 0x00F8);
-               reg14h=(reg14h | 0x6000) & 0xFDFF;
+               reg14h = mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
+               reg14h = (reg14h & 0xFF07) | ((eq_value << 3) & 0x00F8);
+               reg14h = (reg14h | 0x6000) & 0xFDFF;
                mdio_write(net_dev, sis_priv->cur_phy, MII_RESV, reg14h);
-       }
-       else {
-               reg14h=mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
-               if (revision == SIS630A_900_REV && 
-                   (sis_priv->host_bridge_rev == SIS630B0 || 
-                    sis_priv->host_bridge_rev == SIS630B1)) 
-                       mdio_write(net_dev, sis_priv->cur_phy, MII_RESV, (reg14h | 0x2200) & 0xBFFF);
+       } else {
+               reg14h = mdio_read(net_dev, sis_priv->cur_phy, MII_RESV);
+               if (revision == SIS630A_900_REV &&
+                   (sis_priv->host_bridge_rev == SIS630B0 ||
+                    sis_priv->host_bridge_rev == SIS630B1))
+                       mdio_write(net_dev, sis_priv->cur_phy, MII_RESV,
+                                               (reg14h | 0x2200) & 0xBFFF);
                else
-                       mdio_write(net_dev, sis_priv->cur_phy, MII_RESV, (reg14h | 0x2000) & 0xBFFF);
+                       mdio_write(net_dev, sis_priv->cur_phy, MII_RESV,
+                                               (reg14h | 0x2000) & 0xBFFF);
        }
        return;
 }
@@ -1187,7 +1270,7 @@ static void sis630_set_eq(struct net_device *net_dev, u8 revision)
  *     sis900_timer - sis900 timer routine
  *     @data: pointer to sis900 net device
  *
- *     On each timer ticks we check two things, 
+ *     On each timer ticks we check two things,
  *     link status (ON/OFF) and link mode (10/100/Full/Half)
  */
 
@@ -1196,9 +1279,8 @@ static void sis900_timer(unsigned long data)
        struct net_device *net_dev = (struct net_device *)data;
        struct sis900_private *sis_priv = net_dev->priv;
        struct mii_phy *mii_phy = sis_priv->mii;
-       static int next_tick = 5*HZ;
+       static const int next_tick = 5*HZ;
        u16 status;
-       u8 revision;
 
        if (!sis_priv->autong_complete){
                int speed, duplex = 0;
@@ -1206,8 +1288,7 @@ static void sis900_timer(unsigned long data)
                sis900_read_mode(net_dev, &speed, &duplex);
                if (duplex){
                        sis900_set_mode(net_dev->base_addr, speed, duplex);
-                       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-                       sis630_set_eq(net_dev, revision);
+                       sis630_set_eq(net_dev, sis_priv->chipset_rev);
                        netif_start_queue(net_dev);
                }
 
@@ -1230,21 +1311,20 @@ static void sis900_timer(unsigned long data)
                        sis900_check_mode(net_dev, mii_phy);
                        netif_carrier_on(net_dev);
                }
-       }
+       } else {
        /* Link ON -> OFF */
-       else {
                 if (!(status & MII_STAT_LINK)){
                        netif_carrier_off(net_dev);
-                       printk(KERN_INFO "%s: Media Link Off\n", net_dev->name);
+                       if(netif_msg_link(sis_priv))
+                               printk(KERN_INFO "%s: Media Link Off\n", net_dev->name);
 
                        /* Change mode issue */
-                       if ((mii_phy->phy_id0 == 0x001D) && 
+                       if ((mii_phy->phy_id0 == 0x001D) &&
                            ((mii_phy->phy_id1 & 0xFFF0) == 0x8000))
                                        sis900_reset_phy(net_dev,  sis_priv->cur_phy);
-  
-                       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-                       sis630_set_eq(net_dev, revision);
-  
+
+                       sis630_set_eq(net_dev, sis_priv->chipset_rev);
+
                        goto LookForLink;
                 }
        }
@@ -1265,18 +1345,18 @@ static void sis900_timer(unsigned long data)
  *     and autong_complete should be set to 1.
  */
 
-static void sis900_check_mode (struct net_device *net_dev, struct mii_phy *mii_phy)
+static void sis900_check_mode(struct net_device *net_dev, struct mii_phy *mii_phy)
 {
        struct sis900_private *sis_priv = net_dev->priv;
        long ioaddr = net_dev->base_addr;
        int speed, duplex;
 
-       if( mii_phy->phy_types == LAN  ){
-               outl( ~EXD & inl( ioaddr + cfg ), ioaddr + cfg);
+       if (mii_phy->phy_types == LAN) {
+               outl(~EXD & inl(ioaddr + cfg), ioaddr + cfg);
                sis900_set_capability(net_dev , mii_phy);
                sis900_auto_negotiate(net_dev, sis_priv->cur_phy);
-       }else{
-               outl(EXD | inl( ioaddr + cfg ), ioaddr + cfg);
+       } else {
+               outl(EXD | inl(ioaddr + cfg), ioaddr + cfg);
                speed = HW_SPEED_HOME;
                duplex = FDX_CAPABLE_HALF_SELECTED;
                sis900_set_mode(ioaddr, speed, duplex);
@@ -1301,20 +1381,20 @@ static void sis900_set_mode (long ioaddr, int speed, int duplex)
 {
        u32 tx_flags = 0, rx_flags = 0;
 
-       if( inl(ioaddr + cfg) & EDB_MASTER_EN ){
-               tx_flags = TxATP | (DMA_BURST_64 << TxMXDMA_shift) | (TX_FILL_THRESH << TxFILLT_shift);
+       if (inl(ioaddr + cfg) & EDB_MASTER_EN) {
+               tx_flags = TxATP | (DMA_BURST_64 << TxMXDMA_shift) |
+                                       (TX_FILL_THRESH << TxFILLT_shift);
                rx_flags = DMA_BURST_64 << RxMXDMA_shift;
-       }
-       else{
-               tx_flags = TxATP | (DMA_BURST_512 << TxMXDMA_shift) | (TX_FILL_THRESH << TxFILLT_shift);
+       } else {
+               tx_flags = TxATP | (DMA_BURST_512 << TxMXDMA_shift) |
+                                       (TX_FILL_THRESH << TxFILLT_shift);
                rx_flags = DMA_BURST_512 << RxMXDMA_shift;
        }
 
-       if (speed == HW_SPEED_HOME || speed == HW_SPEED_10_MBPS ) {
+       if (speed == HW_SPEED_HOME || speed == HW_SPEED_10_MBPS) {
                rx_flags |= (RxDRNT_10 << RxDRNT_shift);
                tx_flags |= (TxDRNT_10 << TxDRNT_shift);
-       }
-       else {
+       } else {
                rx_flags |= (RxDRNT_100 << RxDRNT_shift);
                tx_flags |= (TxDRNT_100 << TxDRNT_shift);
        }
@@ -1324,6 +1404,11 @@ static void sis900_set_mode (long ioaddr, int speed, int duplex)
                rx_flags |= RxATX;
        }
 
+#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE)
+       /* Can accept Jumbo packet */
+       rx_flags |= RxAJAB;
+#endif
+
        outl (tx_flags, ioaddr + txcfg);
        outl (rx_flags, ioaddr + rxcfg);
 }
@@ -1344,12 +1429,13 @@ static void sis900_auto_negotiate(struct net_device *net_dev, int phy_addr)
        struct sis900_private *sis_priv = net_dev->priv;
        int i = 0;
        u32 status;
-       
+
        while (i++ < 2)
                status = mdio_read(net_dev, phy_addr, MII_STATUS);
 
        if (!(status & MII_STAT_LINK)){
-               printk(KERN_INFO "%s: Media Link Off\n", net_dev->name);
+               if(netif_msg_link(sis_priv))
+                       printk(KERN_INFO "%s: Media Link Off\n", net_dev->name);
                sis_priv->autong_complete = 1;
                netif_carrier_off(net_dev);
                return;
@@ -1392,7 +1478,7 @@ static void sis900_read_mode(struct net_device *net_dev, int *speed, int *duplex
        autoadv = mdio_read(net_dev, phy_addr, MII_ANADV);
        autorec = mdio_read(net_dev, phy_addr, MII_ANLPAR);
        status = autoadv & autorec;
-       
+
        *speed = HW_SPEED_10_MBPS;
        *duplex = FDX_CAPABLE_HALF_SELECTED;
 
@@ -1400,23 +1486,24 @@ static void sis900_read_mode(struct net_device *net_dev, int *speed, int *duplex
                *speed = HW_SPEED_100_MBPS;
        if (status & ( MII_NWAY_TX_FDX | MII_NWAY_T_FDX))
                *duplex = FDX_CAPABLE_FULL_SELECTED;
-       
+
        sis_priv->autong_complete = 1;
 
        /* Workaround for Realtek RTL8201 PHY issue */
-       if((phy->phy_id0 == 0x0000) && ((phy->phy_id1 & 0xFFF0) == 0x8200)){
-               if(mdio_read(net_dev, phy_addr, MII_CONTROL) & MII_CNTL_FDX)
+       if ((phy->phy_id0 == 0x0000) && ((phy->phy_id1 & 0xFFF0) == 0x8200)) {
+               if (mdio_read(net_dev, phy_addr, MII_CONTROL) & MII_CNTL_FDX)
                        *duplex = FDX_CAPABLE_FULL_SELECTED;
-               if(mdio_read(net_dev, phy_addr, 0x0019) & 0x01)
+               if (mdio_read(net_dev, phy_addr, 0x0019) & 0x01)
                        *speed = HW_SPEED_100_MBPS;
        }
 
-       printk(KERN_INFO "%s: Media Link On %s %s-duplex \n",
-              net_dev->name,
-              *speed == HW_SPEED_100_MBPS ?
-              "100mbps" : "10mbps",
-              *duplex == FDX_CAPABLE_FULL_SELECTED ?
-              "full" : "half");
+       if(netif_msg_link(sis_priv))
+               printk(KERN_INFO "%s: Media Link On %s %s-duplex \n",
+                                       net_dev->name,
+                                       *speed == HW_SPEED_100_MBPS ?
+                                               "100mbps" : "10mbps",
+                                       *duplex == FDX_CAPABLE_FULL_SELECTED ?
+                                               "full" : "half");
 }
 
 /**
@@ -1434,8 +1521,9 @@ static void sis900_tx_timeout(struct net_device *net_dev)
        unsigned long flags;
        int i;
 
-       printk(KERN_INFO "%s: Transmit timeout, status %8.8x %8.8x \n",
-              net_dev->name, inl(ioaddr + cr), inl(ioaddr + isr));
+       if(netif_msg_tx_err(sis_priv))
+               printk(KERN_INFO "%s: Transmit timeout, status %8.8x %8.8x \n",
+                       net_dev->name, inl(ioaddr + cr), inl(ioaddr + isr));
 
        /* Disable interrupts by clearing the interrupt mask. */
        outl(0x0000, ioaddr + imr);
@@ -1449,7 +1537,7 @@ static void sis900_tx_timeout(struct net_device *net_dev)
                struct sk_buff *skb = sis_priv->tx_skbuff[i];
 
                if (skb) {
-                       pci_unmap_single(sis_priv->pci_dev, 
+                       pci_unmap_single(sis_priv->pci_dev,
                                sis_priv->tx_ring[i].bufptr, skb->len,
                                PCI_DMA_TODEVICE);
                        dev_kfree_skb_irq(skb);
@@ -1479,7 +1567,7 @@ static void sis900_tx_timeout(struct net_device *net_dev)
  *     @skb: socket buffer pointer to put the data being transmitted
  *     @net_dev: the net device to transmit with
  *
- *     Set the transmit buffer descriptor, 
+ *     Set the transmit buffer descriptor,
  *     and write TxENA to enable transmit state machine.
  *     tell upper layer if the buffer is full
  */
@@ -1523,7 +1611,7 @@ sis900_start_xmit(struct sk_buff *skb, struct net_device *net_dev)
                /* dirty_tx is met in the cycle of cur_tx, buffer full */
                sis_priv->tx_full = 1;
                netif_stop_queue(net_dev);
-       } else if (count_dirty_tx < NUM_TX_DESC) { 
+       } else if (count_dirty_tx < NUM_TX_DESC) {
                /* Typical path, tell upper layer that more transmission is possible */
                netif_start_queue(net_dev);
        } else {
@@ -1536,8 +1624,8 @@ sis900_start_xmit(struct sk_buff *skb, struct net_device *net_dev)
 
        net_dev->trans_start = jiffies;
 
-       if (sis900_debug > 3)
-               printk(KERN_INFO "%s: Queued Tx packet at %p size %d "
+       if (netif_msg_tx_queued(sis_priv))
+               printk(KERN_DEBUG "%s: Queued Tx packet at %p size %d "
                       "to slot %d.\n",
                       net_dev->name, skb->data, (int)skb->len, entry);
 
@@ -1550,11 +1638,11 @@ sis900_start_xmit(struct sk_buff *skb, struct net_device *net_dev)
  *     @dev_instance: the client data object
  *     @regs: snapshot of processor context
  *
- *     The interrupt handler does all of the Rx thread work, 
+ *     The interrupt handler does all of the Rx thread work,
  *     and cleans up after the Tx thread
  */
 
-static irqreturn_t sis900_interrupt(int irq, void *dev_instance, struct pt_regs *regs)
+static irqreturn_t sis900_interrupt(int irq, void *dev_instance)
 {
        struct net_device *net_dev = dev_instance;
        struct sis900_private *sis_priv = net_dev->priv;
@@ -1584,23 +1672,25 @@ static irqreturn_t sis900_interrupt(int irq, void *dev_instance, struct pt_regs
 
                /* something strange happened !!! */
                if (status & HIBERR) {
-                       printk(KERN_INFO "%s: Abnormal interrupt,"
-                              "status %#8.8x.\n", net_dev->name, status);
+                       if(netif_msg_intr(sis_priv))
+                               printk(KERN_INFO "%s: Abnormal interrupt,"
+                                       "status %#8.8x.\n", net_dev->name, status);
                        break;
                }
                if (--boguscnt < 0) {
-                       printk(KERN_INFO "%s: Too much work at interrupt, "
-                              "interrupt status = %#8.8x.\n",
-                              net_dev->name, status);
+                       if(netif_msg_intr(sis_priv))
+                               printk(KERN_INFO "%s: Too much work at interrupt, "
+                                       "interrupt status = %#8.8x.\n",
+                                       net_dev->name, status);
                        break;
                }
        } while (1);
 
-       if (sis900_debug > 3)
-               printk(KERN_INFO "%s: exiting interrupt, "
+       if(netif_msg_intr(sis_priv))
+               printk(KERN_DEBUG "%s: exiting interrupt, "
                       "interrupt status = 0x%#8.8x.\n",
                       net_dev->name, inl(ioaddr + isr));
-       
+
        spin_unlock (&sis_priv->lock);
        return IRQ_RETVAL(handled);
 }
@@ -1609,9 +1699,9 @@ static irqreturn_t sis900_interrupt(int irq, void *dev_instance, struct pt_regs
  *     sis900_rx - sis900 receive routine
  *     @net_dev: the net device which receives data
  *
- *     Process receive interrupt events, 
+ *     Process receive interrupt events,
  *     put buffer to higher layer and refill buffer pool
- *     Note: This fucntion is called by interrupt handler, 
+ *     Note: This function is called by interrupt handler,
  *     don't do "too much" work here
  */
 
@@ -1621,23 +1711,36 @@ static int sis900_rx(struct net_device *net_dev)
        long ioaddr = net_dev->base_addr;
        unsigned int entry = sis_priv->cur_rx % NUM_RX_DESC;
        u32 rx_status = sis_priv->rx_ring[entry].cmdsts;
+       int rx_work_limit;
 
-       if (sis900_debug > 3)
-               printk(KERN_INFO "sis900_rx, cur_rx:%4.4d, dirty_rx:%4.4d "
+       if (netif_msg_rx_status(sis_priv))
+               printk(KERN_DEBUG "sis900_rx, cur_rx:%4.4d, dirty_rx:%4.4d "
                       "status:0x%8.8x\n",
                       sis_priv->cur_rx, sis_priv->dirty_rx, rx_status);
+       rx_work_limit = sis_priv->dirty_rx + NUM_RX_DESC - sis_priv->cur_rx;
 
        while (rx_status & OWN) {
                unsigned int rx_size;
+               unsigned int data_size;
+
+               if (--rx_work_limit < 0)
+                       break;
 
-               rx_size = (rx_status & DSIZE) - CRC_SIZE;
+               data_size = rx_status & DSIZE;
+               rx_size = data_size - CRC_SIZE;
+
+#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE)
+               /* ``TOOLONG'' flag means jumbo packet recived. */
+               if ((rx_status & TOOLONG) && data_size <= MAX_FRAME_SIZE)
+                       rx_status &= (~ ((unsigned int)TOOLONG));
+#endif
 
                if (rx_status & (ABORT|OVERRUN|TOOLONG|RUNT|RXISERR|CRCERR|FAERR)) {
                        /* corrupted packet received */
-                       if (sis900_debug > 3)
-                               printk(KERN_INFO "%s: Corrupted packet "
-                                      "received, buffer status = 0x%8.8x.\n",
-                                      net_dev->name, rx_status);
+                       if (netif_msg_rx_err(sis_priv))
+                               printk(KERN_DEBUG "%s: Corrupted packet "
+                                      "received, buffer status = 0x%8.8x/%d.\n",
+                                      net_dev->name, rx_status, data_size);
                        sis_priv->stats.rx_errors++;
                        if (rx_status & OVERRUN)
                                sis_priv->stats.rx_over_errors++;
@@ -1645,7 +1748,7 @@ static int sis900_rx(struct net_device *net_dev)
                                sis_priv->stats.rx_length_errors++;
                        if (rx_status & (RXISERR | FAERR))
                                sis_priv->stats.rx_frame_errors++;
-                       if (rx_status & CRCERR) 
+                       if (rx_status & CRCERR)
                                sis_priv->stats.rx_crc_errors++;
                        /* reset buffer descriptor state */
                        sis_priv->rx_ring[entry].cmdsts = RX_BUF_SIZE;
@@ -1656,14 +1759,17 @@ static int sis900_rx(struct net_device *net_dev)
                           some unknow bugs, it is possible that
                           we are working on NULL sk_buff :-( */
                        if (sis_priv->rx_skbuff[entry] == NULL) {
-                               printk(KERN_INFO "%s: NULL pointer " 
-                                      "encountered in Rx ring, skipping\n",
-                                      net_dev->name);
+                               if (netif_msg_rx_err(sis_priv))
+                                       printk(KERN_WARNING "%s: NULL pointer "
+                                             "encountered in Rx ring\n"
+                                             "cur_rx:%4.4d, dirty_rx:%4.4d\n",
+                                             net_dev->name, sis_priv->cur_rx,
+                                             sis_priv->dirty_rx);
                                break;
                        }
 
-                       pci_unmap_single(sis_priv->pci_dev, 
-                               sis_priv->rx_ring[entry].bufptr, RX_BUF_SIZE, 
+                       pci_unmap_single(sis_priv->pci_dev,
+                               sis_priv->rx_ring[entry].bufptr, RX_BUF_SIZE,
                                PCI_DMA_FROMDEVICE);
                        /* give the socket buffer to upper layers */
                        skb = sis_priv->rx_skbuff[entry];
@@ -1678,28 +1784,30 @@ static int sis900_rx(struct net_device *net_dev)
                        sis_priv->stats.rx_bytes += rx_size;
                        sis_priv->stats.rx_packets++;
 
-                       /* refill the Rx buffer, what if there is not enought memory for
-                          new socket buffer ?? */
+                       /* refill the Rx buffer, what if there is not enought
+                        * memory for new socket buffer ?? */
                        if ((skb = dev_alloc_skb(RX_BUF_SIZE)) == NULL) {
-                               /* not enough memory for skbuff, this makes a "hole"
-                                  on the buffer ring, it is not clear how the
-                                  hardware will react to this kind of degenerated
-                                  buffer */
-                               printk(KERN_INFO "%s: Memory squeeze,"
-                                      "deferring packet.\n",
-                                      net_dev->name);
+                               /* not enough memory for skbuff, this makes a
+                                * "hole" on the buffer ring, it is not clear
+                                * how the hardware will react to this kind
+                                * of degenerated buffer */
+                               if (netif_msg_rx_status(sis_priv))
+                                       printk(KERN_INFO "%s: Memory squeeze,"
+                                               "deferring packet.\n",
+                                               net_dev->name);
                                sis_priv->rx_skbuff[entry] = NULL;
                                /* reset buffer descriptor state */
                                sis_priv->rx_ring[entry].cmdsts = 0;
                                sis_priv->rx_ring[entry].bufptr = 0;
                                sis_priv->stats.rx_dropped++;
+                               sis_priv->cur_rx++;
                                break;
                        }
                        skb->dev = net_dev;
                        sis_priv->rx_skbuff[entry] = skb;
                        sis_priv->rx_ring[entry].cmdsts = RX_BUF_SIZE;
-                       sis_priv->rx_ring[entry].bufptr = 
-                               pci_map_single(sis_priv->pci_dev, skb->tail, 
+                       sis_priv->rx_ring[entry].bufptr =
+                               pci_map_single(sis_priv->pci_dev, skb->data,
                                        RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
                        sis_priv->dirty_rx++;
                }
@@ -1708,22 +1816,23 @@ static int sis900_rx(struct net_device *net_dev)
                rx_status = sis_priv->rx_ring[entry].cmdsts;
        } // while
 
-       /* refill the Rx buffer, what if the rate of refilling is slower than 
-          consuming ?? */
-       for (;sis_priv->cur_rx - sis_priv->dirty_rx > 0; sis_priv->dirty_rx++) {
+       /* refill the Rx buffer, what if the rate of refilling is slower
+        * than consuming ?? */
+       for (; sis_priv->cur_rx != sis_priv->dirty_rx; sis_priv->dirty_rx++) {
                struct sk_buff *skb;
 
                entry = sis_priv->dirty_rx % NUM_RX_DESC;
 
                if (sis_priv->rx_skbuff[entry] == NULL) {
                        if ((skb = dev_alloc_skb(RX_BUF_SIZE)) == NULL) {
-                               /* not enough memory for skbuff, this makes a "hole"
-                                  on the buffer ring, it is not clear how the 
-                                  hardware will react to this kind of degenerated 
-                                  buffer */
-                               printk(KERN_INFO "%s: Memory squeeze,"
-                                      "deferring packet.\n",
-                                      net_dev->name);
+                               /* not enough memory for skbuff, this makes a
+                                * "hole" on the buffer ring, it is not clear
+                                * how the hardware will react to this kind
+                                * of degenerated buffer */
+                               if (netif_msg_rx_err(sis_priv))
+                                       printk(KERN_INFO "%s: Memory squeeze,"
+                                               "deferring packet.\n",
+                                               net_dev->name);
                                sis_priv->stats.rx_dropped++;
                                break;
                        }
@@ -1731,7 +1840,7 @@ static int sis900_rx(struct net_device *net_dev)
                        sis_priv->rx_skbuff[entry] = skb;
                        sis_priv->rx_ring[entry].cmdsts = RX_BUF_SIZE;
                        sis_priv->rx_ring[entry].bufptr =
-                               pci_map_single(sis_priv->pci_dev, skb->tail,
+                               pci_map_single(sis_priv->pci_dev, skb->data,
                                        RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
                }
        }
@@ -1745,9 +1854,9 @@ static int sis900_rx(struct net_device *net_dev)
  *     sis900_finish_xmit - finish up transmission of packets
  *     @net_dev: the net device to be transmitted on
  *
- *     Check for error condition and free socket buffer etc 
+ *     Check for error condition and free socket buffer etc
  *     schedule for more transmission as needed
- *     Note: This fucntion is called by interrupt handler, 
+ *     Note: This function is called by interrupt handler,
  *     don't do "too much" work here
  */
 
@@ -1765,15 +1874,15 @@ static void sis900_finish_xmit (struct net_device *net_dev)
 
                if (tx_status & OWN) {
                        /* The packet is not transmitted yet (owned by hardware) !
-                          Note: the interrupt is generated only when Tx Machine
-                          is idle, so this is an almost impossible case */
+                        * Note: the interrupt is generated only when Tx Machine
+                        * is idle, so this is an almost impossible case */
                        break;
                }
 
                if (tx_status & (ABORT | UNDERRUN | OWCOLL)) {
                        /* packet unsuccessfully transmitted */
-                       if (sis900_debug > 3)
-                               printk(KERN_INFO "%s: Transmit "
+                       if (netif_msg_tx_err(sis_priv))
+                               printk(KERN_DEBUG "%s: Transmit "
                                       "error, Tx status %8.8x.\n",
                                       net_dev->name, tx_status);
                        sis_priv->stats.tx_errors++;
@@ -1793,7 +1902,7 @@ static void sis900_finish_xmit (struct net_device *net_dev)
                }
                /* Free the original skb. */
                skb = sis_priv->tx_skbuff[entry];
-               pci_unmap_single(sis_priv->pci_dev, 
+               pci_unmap_single(sis_priv->pci_dev,
                        sis_priv->tx_ring[entry].bufptr, skb->len,
                        PCI_DMA_TODEVICE);
                dev_kfree_skb_irq(skb);
@@ -1804,23 +1913,22 @@ static void sis900_finish_xmit (struct net_device *net_dev)
 
        if (sis_priv->tx_full && netif_queue_stopped(net_dev) &&
            sis_priv->cur_tx - sis_priv->dirty_tx < NUM_TX_DESC - 4) {
-               /* The ring is no longer full, clear tx_full and schedule more transmission
-                  by netif_wake_queue(net_dev) */
+               /* The ring is no longer full, clear tx_full and schedule
+                * more transmission by netif_wake_queue(net_dev) */
                sis_priv->tx_full = 0;
                netif_wake_queue (net_dev);
        }
 }
 
 /**
- *     sis900_close - close sis900 device 
+ *     sis900_close - close sis900 device
  *     @net_dev: the net device to be closed
  *
- *     Disable interrupts, stop the Tx and Rx Status Machine 
+ *     Disable interrupts, stop the Tx and Rx Status Machine
  *     free Tx and RX socket buffer
  */
 
-static int
-sis900_close(struct net_device *net_dev)
+static int sis900_close(struct net_device *net_dev)
 {
        long ioaddr = net_dev->base_addr;
        struct sis900_private *sis_priv = net_dev->priv;
@@ -1844,7 +1952,7 @@ sis900_close(struct net_device *net_dev)
        for (i = 0; i < NUM_RX_DESC; i++) {
                skb = sis_priv->rx_skbuff[i];
                if (skb) {
-                       pci_unmap_single(sis_priv->pci_dev, 
+                       pci_unmap_single(sis_priv->pci_dev,
                                sis_priv->rx_ring[i].bufptr,
                                RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
                        dev_kfree_skb(skb);
@@ -1854,7 +1962,7 @@ sis900_close(struct net_device *net_dev)
        for (i = 0; i < NUM_TX_DESC; i++) {
                skb = sis_priv->tx_skbuff[i];
                if (skb) {
-                       pci_unmap_single(sis_priv->pci_dev, 
+                       pci_unmap_single(sis_priv->pci_dev,
                                sis_priv->tx_ring[i].bufptr, skb->len,
                                PCI_DMA_TODEVICE);
                        dev_kfree_skb(skb);
@@ -1874,7 +1982,7 @@ sis900_close(struct net_device *net_dev)
  *
  *     Process ethtool command such as "ehtool -i" to show information
  */
+
 static void sis900_get_drvinfo(struct net_device *net_dev,
                               struct ethtool_drvinfo *info)
 {
@@ -1885,12 +1993,126 @@ static void sis900_get_drvinfo(struct net_device *net_dev,
        strcpy (info->bus_info, pci_name(sis_priv->pci_dev));
 }
 
-static struct ethtool_ops sis900_ethtool_ops = {
-       .get_drvinfo =          sis900_get_drvinfo,
+static u32 sis900_get_msglevel(struct net_device *net_dev)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       return sis_priv->msg_enable;
+}
+
+static void sis900_set_msglevel(struct net_device *net_dev, u32 value)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       sis_priv->msg_enable = value;
+}
+
+static u32 sis900_get_link(struct net_device *net_dev)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       return mii_link_ok(&sis_priv->mii_info);
+}
+
+static int sis900_get_settings(struct net_device *net_dev,
+                               struct ethtool_cmd *cmd)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       spin_lock_irq(&sis_priv->lock);
+       mii_ethtool_gset(&sis_priv->mii_info, cmd);
+       spin_unlock_irq(&sis_priv->lock);
+       return 0;
+}
+
+static int sis900_set_settings(struct net_device *net_dev,
+                               struct ethtool_cmd *cmd)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       int rt;
+       spin_lock_irq(&sis_priv->lock);
+       rt = mii_ethtool_sset(&sis_priv->mii_info, cmd);
+       spin_unlock_irq(&sis_priv->lock);
+       return rt;
+}
+
+static int sis900_nway_reset(struct net_device *net_dev)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       return mii_nway_restart(&sis_priv->mii_info);
+}
+
+/**
+ *     sis900_set_wol - Set up Wake on Lan registers
+ *     @net_dev: the net device to probe
+ *     @wol: container for info passed to the driver
+ *
+ *     Process ethtool command "wol" to setup wake on lan features.
+ *     SiS900 supports sending WoL events if a correct packet is received,
+ *     but there is no simple way to filter them to only a subset (broadcast,
+ *     multicast, unicast or arp).
+ */
+
+static int sis900_set_wol(struct net_device *net_dev, struct ethtool_wolinfo *wol)
+{
+       struct sis900_private *sis_priv = net_dev->priv;
+       long pmctrl_addr = net_dev->base_addr + pmctrl;
+       u32 cfgpmcsr = 0, pmctrl_bits = 0;
+
+       if (wol->wolopts == 0) {
+               pci_read_config_dword(sis_priv->pci_dev, CFGPMCSR, &cfgpmcsr);
+               cfgpmcsr &= ~PME_EN;
+               pci_write_config_dword(sis_priv->pci_dev, CFGPMCSR, cfgpmcsr);
+               outl(pmctrl_bits, pmctrl_addr);
+               if (netif_msg_wol(sis_priv))
+                       printk(KERN_DEBUG "%s: Wake on LAN disabled\n", net_dev->name);
+               return 0;
+       }
+
+       if (wol->wolopts & (WAKE_MAGICSECURE | WAKE_UCAST | WAKE_MCAST
+                               | WAKE_BCAST | WAKE_ARP))
+               return -EINVAL;
+
+       if (wol->wolopts & WAKE_MAGIC)
+               pmctrl_bits |= MAGICPKT;
+       if (wol->wolopts & WAKE_PHY)
+               pmctrl_bits |= LINKON;
+
+       outl(pmctrl_bits, pmctrl_addr);
+
+       pci_read_config_dword(sis_priv->pci_dev, CFGPMCSR, &cfgpmcsr);
+       cfgpmcsr |= PME_EN;
+       pci_write_config_dword(sis_priv->pci_dev, CFGPMCSR, cfgpmcsr);
+       if (netif_msg_wol(sis_priv))
+               printk(KERN_DEBUG "%s: Wake on LAN enabled\n", net_dev->name);
+
+       return 0;
+}
+
+static void sis900_get_wol(struct net_device *net_dev, struct ethtool_wolinfo *wol)
+{
+       long pmctrl_addr = net_dev->base_addr + pmctrl;
+       u32 pmctrl_bits;
+
+       pmctrl_bits = inl(pmctrl_addr);
+       if (pmctrl_bits & MAGICPKT)
+               wol->wolopts |= WAKE_MAGIC;
+       if (pmctrl_bits & LINKON)
+               wol->wolopts |= WAKE_PHY;
+
+       wol->supported = (WAKE_PHY | WAKE_MAGIC);
+}
+
+static const struct ethtool_ops sis900_ethtool_ops = {
+       .get_drvinfo    = sis900_get_drvinfo,
+       .get_msglevel   = sis900_get_msglevel,
+       .set_msglevel   = sis900_set_msglevel,
+       .get_link       = sis900_get_link,
+       .get_settings   = sis900_get_settings,
+       .set_settings   = sis900_set_settings,
+       .nway_reset     = sis900_nway_reset,
+       .get_wol        = sis900_get_wol,
+       .set_wol        = sis900_set_wol
 };
 
 /**
- *     mii_ioctl - process MII i/o control command 
+ *     mii_ioctl - process MII i/o control command
  *     @net_dev: the net device to command for
  *     @rq: parameter for command
  *     @cmd: the i/o command
@@ -1923,7 +2145,7 @@ static int mii_ioctl(struct net_device *net_dev, struct ifreq *rq, int cmd)
 }
 
 /**
- *     sis900_get_stats - Get sis900 read/write statistics 
+ *     sis900_get_stats - Get sis900 read/write statistics
  *     @net_dev: the net device to get statistics for
  *
  *     get tx/rx statistics for sis900
@@ -1938,7 +2160,7 @@ sis900_get_stats(struct net_device *net_dev)
 }
 
 /**
- *     sis900_set_config - Set media type by net_device.set_config 
+ *     sis900_set_config - Set media type by net_device.set_config
  *     @dev: the net device for media type change
  *     @map: ifmap passed by ifconfig
  *
@@ -1948,86 +2170,89 @@ sis900_get_stats(struct net_device *net_dev)
  */
 
 static int sis900_set_config(struct net_device *dev, struct ifmap *map)
-{    
+{
        struct sis900_private *sis_priv = dev->priv;
        struct mii_phy *mii_phy = sis_priv->mii;
-        
+
        u16 status;
 
        if ((map->port != (u_char)(-1)) && (map->port != dev->if_port)) {
                /* we switch on the ifmap->port field. I couldn't find anything
-                  like a definition or standard for the values of that field.
-                  I think the meaning of those values is device specific. But
-                  since I would like to change the media type via the ifconfig
-                  command I use the definition from linux/netdevice.h 
-                  (which seems to be different from the ifport(pcmcia) definition) 
-               */
+                * like a definition or standard for the values of that field.
+                * I think the meaning of those values is device specific. But
+                * since I would like to change the media type via the ifconfig
+                * command I use the definition from linux/netdevice.h
+                * (which seems to be different from the ifport(pcmcia) definition) */
                switch(map->port){
-               case IF_PORT_UNKNOWN: /* use auto here */   
+               case IF_PORT_UNKNOWN: /* use auto here */
                        dev->if_port = map->port;
-                       /* we are going to change the media type, so the Link will
-                          be temporary down and we need to reflect that here. When
-                          the Link comes up again, it will be sensed by the sis_timer
-                          procedure, which also does all the rest for us */
+                       /* we are going to change the media type, so the Link
+                        * will be temporary down and we need to reflect that
+                        * here. When the Link comes up again, it will be
+                        * sensed by the sis_timer procedure, which also does
+                        * all the rest for us */
                        netif_carrier_off(dev);
-                
+
                        /* read current state */
                        status = mdio_read(dev, mii_phy->phy_addr, MII_CONTROL);
-                
+
                        /* enable auto negotiation and reset the negotioation
-                          (I don't really know what the auto negatiotiation reset
-                          really means, but it sounds for me right to do one here)*/
+                        * (I don't really know what the auto negatiotiation
+                        * reset really means, but it sounds for me right to
+                        * do one here) */
                        mdio_write(dev, mii_phy->phy_addr,
                                   MII_CONTROL, status | MII_CNTL_AUTO | MII_CNTL_RST_AUTO);
 
                        break;
-            
-               case IF_PORT_10BASET: /* 10BaseT */         
+
+               case IF_PORT_10BASET: /* 10BaseT */
                        dev->if_port = map->port;
-                
-                       /* we are going to change the media type, so the Link will
-                          be temporary down and we need to reflect that here. When
-                          the Link comes up again, it will be sensed by the sis_timer
-                          procedure, which also does all the rest for us */
+
+                       /* we are going to change the media type, so the Link
+                        * will be temporary down and we need to reflect that
+                        * here. When the Link comes up again, it will be
+                        * sensed by the sis_timer procedure, which also does
+                        * all the rest for us */
                        netif_carrier_off(dev);
-        
+
                        /* set Speed to 10Mbps */
                        /* read current state */
                        status = mdio_read(dev, mii_phy->phy_addr, MII_CONTROL);
-                
+
                        /* disable auto negotiation and force 10MBit mode*/
                        mdio_write(dev, mii_phy->phy_addr,
-                                  MII_CONTROL, status & ~(MII_CNTL_SPEED | MII_CNTL_AUTO));
+                                  MII_CONTROL, status & ~(MII_CNTL_SPEED |
+                                       MII_CNTL_AUTO));
                        break;
-            
+
                case IF_PORT_100BASET: /* 100BaseT */
-               case IF_PORT_100BASETX: /* 100BaseTx */ 
+               case IF_PORT_100BASETX: /* 100BaseTx */
                        dev->if_port = map->port;
-                
-                       /* we are going to change the media type, so the Link will
-                          be temporary down and we need to reflect that here. When
-                          the Link comes up again, it will be sensed by the sis_timer
-                          procedure, which also does all the rest for us */
+
+                       /* we are going to change the media type, so the Link
+                        * will be temporary down and we need to reflect that
+                        * here. When the Link comes up again, it will be
+                        * sensed by the sis_timer procedure, which also does
+                        * all the rest for us */
                        netif_carrier_off(dev);
-                
+
                        /* set Speed to 100Mbps */
                        /* disable auto negotiation and enable 100MBit Mode */
                        status = mdio_read(dev, mii_phy->phy_addr, MII_CONTROL);
                        mdio_write(dev, mii_phy->phy_addr,
-                                  MII_CONTROL, (status & ~MII_CNTL_SPEED) | MII_CNTL_SPEED);
-                
+                                  MII_CONTROL, (status & ~MII_CNTL_SPEED) |
+                                  MII_CNTL_SPEED);
+
                        break;
-            
+
                case IF_PORT_10BASE2: /* 10Base2 */
                case IF_PORT_AUI: /* AUI */
                case IF_PORT_100BASEFX: /* 100BaseFx */
                        /* These Modes are not supported (are they?)*/
-                       printk(KERN_INFO "Not supported");
                        return -EOPNOTSUPP;
                        break;
-            
+
                default:
-                       printk(KERN_INFO "Invalid");
                        return -EINVAL;
                }
        }
@@ -2035,14 +2260,14 @@ static int sis900_set_config(struct net_device *dev, struct ifmap *map)
 }
 
 /**
- *     sis900_mcast_bitnr - compute hashtable index 
+ *     sis900_mcast_bitnr - compute hashtable index
  *     @addr: multicast address
  *     @revision: revision id of chip
  *
  *     SiS 900 uses the most sigificant 7 bits to index a 128 bits multicast
  *     hash table, which makes this function a little bit different from other drivers
  *     SiS 900 B0 & 635 M/B uses the most significat 8 bits to index 256 bits
- *     multicast hash table. 
+ *     multicast hash table.
  */
 
 static inline u16 sis900_mcast_bitnr(u8 *addr, u8 revision)
@@ -2058,7 +2283,7 @@ static inline u16 sis900_mcast_bitnr(u8 *addr, u8 revision)
 }
 
 /**
- *     set_rx_mode - Set SiS900 receive mode 
+ *     set_rx_mode - Set SiS900 receive mode
  *     @net_dev: the net device to be set
  *
  *     Set SiS900 receive mode for promiscuous, multicast, or broadcast mode.
@@ -2073,11 +2298,10 @@ static void set_rx_mode(struct net_device *net_dev)
        u16 mc_filter[16] = {0};        /* 256/128 bits multicast hash table */
        int i, table_entries;
        u32 rx_mode;
-       u8 revision;
 
-       /* 635 Hash Table entires = 256(2^16) */
-       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-       if((revision >= SIS635A_900_REV) || (revision == SIS900B_900_REV))
+       /* 635 Hash Table entries = 256(2^16) */
+       if((sis_priv->chipset_rev >= SIS635A_900_REV) ||
+                       (sis_priv->chipset_rev == SIS900B_900_REV))
                table_entries = 16;
        else
                table_entries = 8;
@@ -2094,14 +2318,16 @@ static void set_rx_mode(struct net_device *net_dev)
                for (i = 0; i < table_entries; i++)
                        mc_filter[i] = 0xffff;
        } else {
-               /* Accept Broadcast packet, destination address matchs our MAC address,
-                  use Receive Filter to reject unwanted MCAST packet */
+               /* Accept Broadcast packet, destination address matchs our
+                * MAC address, use Receive Filter to reject unwanted MCAST
+                * packets */
                struct dev_mc_list *mclist;
                rx_mode = RFAAB;
-               for (i = 0, mclist = net_dev->mc_list; mclist && i < net_dev->mc_count;
-                    i++, mclist = mclist->next) {
+               for (i = 0, mclist = net_dev->mc_list;
+                       mclist && i < net_dev->mc_count;
+                       i++, mclist = mclist->next) {
                        unsigned int bit_nr =
-                               sis900_mcast_bitnr(mclist->dmi_addr, revision);
+                               sis900_mcast_bitnr(mclist->dmi_addr, sis_priv->chipset_rev);
                        mc_filter[bit_nr >> 4] |= (1 << (bit_nr & 0xf));
                }
        }
@@ -2115,7 +2341,8 @@ static void set_rx_mode(struct net_device *net_dev)
 
        outl(RFEN | rx_mode, ioaddr + rfcr);
 
-       /* sis900 is capatable of looping back packet at MAC level for debugging purpose */
+       /* sis900 is capable of looping back packets at MAC level for
+        * debugging purpose */
        if (net_dev->flags & IFF_LOOPBACK) {
                u32 cr_saved;
                /* We must disable Tx/Rx before setting loopback mode */
@@ -2132,7 +2359,7 @@ static void set_rx_mode(struct net_device *net_dev)
 }
 
 /**
- *     sis900_reset - Reset sis900 MAC 
+ *     sis900_reset - Reset sis900 MAC
  *     @net_dev: the net device to reset
  *
  *     reset sis900 MAC and wait until finished
@@ -2146,28 +2373,27 @@ static void sis900_reset(struct net_device *net_dev)
        long ioaddr = net_dev->base_addr;
        int i = 0;
        u32 status = TxRCMP | RxRCMP;
-       u8  revision;
 
        outl(0, ioaddr + ier);
        outl(0, ioaddr + imr);
        outl(0, ioaddr + rfcr);
 
        outl(RxRESET | TxRESET | RESET | inl(ioaddr + cr), ioaddr + cr);
-       
+
        /* Check that the chip has finished the reset. */
        while (status && (i++ < 1000)) {
                status ^= (inl(isr + ioaddr) & status);
        }
 
-       pci_read_config_byte(sis_priv->pci_dev, PCI_CLASS_REVISION, &revision);
-       if( (revision >= SIS635A_900_REV) || (revision == SIS900B_900_REV) )
+       if( (sis_priv->chipset_rev >= SIS635A_900_REV) ||
+                       (sis_priv->chipset_rev == SIS900B_900_REV) )
                outl(PESEL | RND_CNT, ioaddr + cfg);
        else
                outl(PESEL, ioaddr + cfg);
 }
 
 /**
- *     sis900_remove - Remove sis900 device 
+ *     sis900_remove - Remove sis900 device
  *     @pci_dev: the pci device to be removed
  *
  *     remove and release SiS900 net device
@@ -2197,10 +2423,9 @@ static void __devexit sis900_remove(struct pci_dev *pci_dev)
 
 #ifdef CONFIG_PM
 
-static int sis900_suspend(struct pci_dev *pci_dev, u32 state)
+static int sis900_suspend(struct pci_dev *pci_dev, pm_message_t state)
 {
        struct net_device *net_dev = pci_get_drvdata(pci_dev);
-       struct sis900_private *sis_priv = net_dev->priv;
        long ioaddr = net_dev->base_addr;
 
        if(!netif_running(net_dev))
@@ -2212,8 +2437,8 @@ static int sis900_suspend(struct pci_dev *pci_dev, u32 state)
        /* Stop the chip's Tx and Rx Status Machine */
        outl(RxDIS | TxDIS | inl(ioaddr + cr), ioaddr + cr);
 
-       pci_set_power_state(pci_dev, 3);
-       pci_save_state(pci_dev, sis_priv->pci_state);
+       pci_set_power_state(pci_dev, PCI_D3hot);
+       pci_save_state(pci_dev);
 
        return 0;
 }
@@ -2226,8 +2451,8 @@ static int sis900_resume(struct pci_dev *pci_dev)
 
        if(!netif_running(net_dev))
                return 0;
-       pci_restore_state(pci_dev, sis_priv->pci_state);
-       pci_set_power_state(pci_dev, 0);
+       pci_restore_state(pci_dev);
+       pci_set_power_state(pci_dev, PCI_D0);
 
        sis900_init_rxfilter(net_dev);
 
@@ -2271,7 +2496,7 @@ static int __init sis900_init_module(void)
        printk(version);
 #endif
 
-       return pci_module_init(&sis900_pci_driver);
+       return pci_register_driver(&sis900_pci_driver);
 }
 
 static void __exit sis900_cleanup_module(void)