vserver 2.0 rc7
[linux-2.6.git] / drivers / net / sis900.c
index aa045b6..3107aed 100644 (file)
@@ -1,6 +1,6 @@
 /* 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
+   Revision:   1.08.08 Jan. 22 2005
    
    Modified from the driver which is originally written by Donald Becker.
    
@@ -16,8 +16,8 @@
    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.08 Jan. 22 2005 Daniele Venzano use netif_msg for debugging messages
    Rev 1.08.07 Nov.  2 2003 Daniele Venzano <webvenza@libero.it> 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
@@ -48,6 +48,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 <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.08 Jan. 22 2005"
 
 static char version[] __devinitdata =
 KERN_INFO "sis900.c: " SIS900_DRV_VERSION "\n";
@@ -81,8 +83,13 @@ 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)
@@ -116,16 +123,19 @@ static struct mii_chip_info {
 #define        HOME    0x0001
 #define LAN    0x0002
 #define MIX    0x0003
+#define UNKNOWN        0x0
 } mii_chip_table[] = {
        { "SiS 900 Internal MII PHY",           0x001d, 0x8000, LAN },
        { "SiS 7014 Physical Layer Solution",   0x0016, 0xf830, LAN },
+       { "Altimata AC101LF PHY",               0x0022, 0x5520, LAN },
        { "AMD 79C901 10BASE-T PHY",            0x0000, 0x6B70, LAN },
        { "AMD 79C901 HomePNA PHY",             0x0000, 0x6B90, HOME},
        { "ICS LAN PHY",                        0x0015, 0xF440, LAN },
        { "NS 83851 PHY",                       0x2000, 0x5C20, MIX },
+       { "NS 83847 PHY",                       0x2000, 0x5C30, MIX },
        { "Realtek RTL8201 PHY",                0x0000, 0x8200, LAN },
        { "VIA 6103 PHY",                       0x0101, 0x8f20, LAN },
-       {0,},
+       {NULL,},
 };
 
 struct mii_phy {
@@ -138,9 +148,9 @@ struct mii_phy {
 };
 
 typedef struct _BufferDesc {
-       u32     link;
-       u32     cmdsts;
-       u32     bufptr;
+       u32 link;
+       u32 cmdsts;
+       u32 bufptr;
 } BufferDesc;
 
 struct sis900_private {
@@ -152,9 +162,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;
@@ -168,27 +181,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);
@@ -232,8 +248,8 @@ static int __devinit sis900_get_mac_addr(struct pci_dev * pci_dev, struct net_de
        /* check to see if we have sane EEPROM */
        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;
        }
 
@@ -254,19 +270,20 @@ static int __devinit sis900_get_mac_addr(struct pci_dev * pci_dev, struct net_de
  *     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);
@@ -276,6 +293,7 @@ static int __devinit sis630e_get_mac_addr(struct pci_dev * pci_dev, struct net_d
                ((u8 *)(net_dev->dev_addr))[i] = inb(0x71); 
        }
        pci_write_config_byte(isa_bridge, 0x48, reg & ~0x40);
+       pci_dev_put(isa_bridge);
 
        return 1;
 }
@@ -291,7 +309,8 @@ static int __devinit sis630e_get_mac_addr(struct pci_dev * pci_dev, struct net_d
  *     @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;
@@ -333,7 +352,8 @@ static int __devinit sis635_get_mac_addr(struct pci_dev * pci_dev, struct net_de
  *     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;
@@ -370,7 +390,8 @@ static int __devinit sis96x_get_mac_addr(struct pci_dev * pci_dev, struct net_de
  *     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;
@@ -379,8 +400,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 *dev_name = pci_name(pci_dev);
 
 /* when built into the kernel, we only print version if device is found */
 #ifndef MODULE
@@ -449,43 +470,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);
+       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);
+       
        ret = 0;
-
-       if (revision == SIS630E_900_REV)
+       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,8 +541,6 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
 
        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);
@@ -521,13 +564,13 @@ static int __devinit sis900_probe (struct pci_dev *pci_dev, const struct pci_dev
  *     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;
 
@@ -541,12 +584,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;
@@ -571,20 +618,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 )
+               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;
        }
 
@@ -608,14 +659,14 @@ 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);
@@ -644,36 +695,45 @@ 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;
+       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 deafalut PHY */
-                if ( (status & MII_STAT_LINK) && !(default_phy) )
+               /* Link ON & Not select default PHY & not ghost PHY */
+                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)
+                               phy_lan = phy;
                 }
        }
 
-       if( (!default_phy) && phy_home )
+       if (!default_phy && phy_home)
                default_phy = phy_home;
-       else if(!default_phy)
+       else if (!default_phy && phy_lan)
+               default_phy = phy_lan;
+       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);
 
@@ -694,7 +754,7 @@ static u16 sis900_default_phy(struct net_device * net_dev)
  *     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;
@@ -801,7 +861,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);
@@ -844,7 +904,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);
@@ -909,6 +970,20 @@ static u16 sis900_reset_phy(struct net_device *net_dev, int phy_addr)
        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, NULL);
+       enable_irq(dev->irq);
+}
+#endif
+
 /**
  *     sis900_open - open sis900 device
  *     @net_dev: the net device to open
@@ -922,17 +997,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, SA_SHIRQ,
+                                               net_dev->name, net_dev);
        if (ret)
                return ret;
 
@@ -977,6 +1051,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;
@@ -994,8 +1069,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));
                }
        }
@@ -1032,8 +1107,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));
 }
 
@@ -1086,8 +1161,8 @@ sis900_init_rx_ring(struct net_device *net_dev)
 
        /* 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));
 }
 
@@ -1129,48 +1204,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 || 
                     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);
+       } 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);
+                       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;
 }
@@ -1190,7 +1272,6 @@ static void sis900_timer(unsigned long data)
        struct mii_phy *mii_phy = sis_priv->mii;
        static int next_tick = 5*HZ;
        u16 status;
-       u8 revision;
 
        if (!sis_priv->autong_complete){
                int speed, duplex = 0;
@@ -1198,8 +1279,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);
                }
 
@@ -1222,20 +1302,19 @@ 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) && 
                            ((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;
                 }
@@ -1257,18 +1336,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);
@@ -1293,20 +1372,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);
        }
@@ -1341,7 +1420,8 @@ static void sis900_auto_negotiate(struct net_device *net_dev, int phy_addr)
                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;
@@ -1396,19 +1476,20 @@ static void sis900_read_mode(struct net_device *net_dev, int *speed, int *duplex
        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");
 }
 
 /**
@@ -1426,8 +1507,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);
@@ -1445,7 +1527,7 @@ static void sis900_tx_timeout(struct net_device *net_dev)
                                sis_priv->tx_ring[i].bufptr, skb->len,
                                PCI_DMA_TODEVICE);
                        dev_kfree_skb_irq(skb);
-                       sis_priv->tx_skbuff[i] = 0;
+                       sis_priv->tx_skbuff[i] = NULL;
                        sis_priv->tx_ring[i].cmdsts = 0;
                        sis_priv->tx_ring[i].bufptr = 0;
                        sis_priv->stats.tx_dropped++;
@@ -1528,8 +1610,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);
 
@@ -1576,20 +1658,22 @@ 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));
        
@@ -1614,8 +1698,8 @@ static int sis900_rx(struct net_device *net_dev)
        unsigned int entry = sis_priv->cur_rx % NUM_RX_DESC;
        u32 rx_status = sis_priv->rx_ring[entry].cmdsts;
 
-       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);
 
@@ -1626,8 +1710,8 @@ static int sis900_rx(struct net_device *net_dev)
 
                if (rx_status & (ABORT|OVERRUN|TOOLONG|RUNT|RXISERR|CRCERR|FAERR)) {
                        /* corrupted packet received */
-                       if (sis900_debug > 3)
-                               printk(KERN_INFO "%s: Corrupted packet "
+                       if (netif_msg_rx_err(sis_priv))
+                               printk(KERN_DEBUG "%s: Corrupted packet "
                                       "received, buffer status = 0x%8.8x.\n",
                                       net_dev->name, rx_status);
                        sis_priv->stats.rx_errors++;
@@ -1648,9 +1732,10 @@ 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_INFO "%s: NULL pointer " 
+                                               "encountered in Rx ring, skipping\n",
+                                               net_dev->name);
                                break;
                        }
 
@@ -1670,16 +1755,17 @@ 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;
@@ -1700,8 +1786,8 @@ 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 ?? */
+       /* 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++) {
                struct sk_buff *skb;
 
@@ -1709,13 +1795,14 @@ static int sis900_rx(struct net_device *net_dev)
 
                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;
                        }
@@ -1757,15 +1844,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++;
@@ -1796,8 +1883,8 @@ 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);
        }
@@ -1811,8 +1898,7 @@ static void sis900_finish_xmit (struct net_device *net_dev)
  *     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;
@@ -1840,7 +1926,7 @@ sis900_close(struct net_device *net_dev)
                                sis_priv->rx_ring[i].bufptr,
                                RX_BUF_SIZE, PCI_DMA_FROMDEVICE);
                        dev_kfree_skb(skb);
-                       sis_priv->rx_skbuff[i] = 0;
+                       sis_priv->rx_skbuff[i] = NULL;
                }
        }
        for (i = 0; i < NUM_TX_DESC; i++) {
@@ -1850,7 +1936,7 @@ sis900_close(struct net_device *net_dev)
                                sis_priv->tx_ring[i].bufptr, skb->len,
                                PCI_DMA_TODEVICE);
                        dev_kfree_skb(skb);
-                       sis_priv->tx_skbuff[i] = 0;
+                       sis_priv->tx_skbuff[i] = NULL;
                }
        }
 
@@ -1877,8 +1963,59 @@ static void sis900_get_drvinfo(struct net_device *net_dev,
        strcpy (info->bus_info, pci_name(sis_priv->pci_dev));
 }
 
+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);
+}
+
 static struct ethtool_ops sis900_ethtool_ops = {
-       .get_drvinfo =          sis900_get_drvinfo,
+       .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,
 };
 
 /**
@@ -1948,27 +2085,28 @@ static int sis900_set_config(struct net_device *dev, struct ifmap *map)
 
        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 */   
                        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);
 
@@ -1977,10 +2115,11 @@ static int sis900_set_config(struct net_device *dev, struct ifmap *map)
                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 */
@@ -1989,24 +2128,27 @@ static int sis900_set_config(struct net_device *dev, struct ifmap *map)
                 
                        /* 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 */ 
                        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;
             
@@ -2014,12 +2156,10 @@ static int sis900_set_config(struct net_device *dev, struct ifmap *map)
                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;
                }
        }
@@ -2065,11 +2205,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))
+       if((sis_priv->chipset_rev >= SIS635A_900_REV) ||
+                       (sis_priv->chipset_rev == SIS900B_900_REV))
                table_entries = 16;
        else
                table_entries = 8;
@@ -2086,14 +2225,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));
                }
        }
@@ -2107,7 +2248,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 */
@@ -2138,7 +2280,6 @@ 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);
@@ -2151,8 +2292,8 @@ static void sis900_reset(struct net_device *net_dev)
                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);
@@ -2189,10 +2330,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))
@@ -2204,8 +2344,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;
 }
@@ -2218,8 +2358,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);