netdev
[Top] [All Lists]

[PATCH] 2.6.3 pcnet32.c convert to use netif_msg_*

To: tsbogend@xxxxxxxxxxxxxxxx, jgarzik@xxxxxxxxx, netdev@xxxxxxxxxxx
Subject: [PATCH] 2.6.3 pcnet32.c convert to use netif_msg_*
From: Don Fry <brazilnut@xxxxxxxxxx>
Date: Thu, 19 Feb 2004 14:42:57 -0800 (PST)
Sender: netdev-bounce@xxxxxxxxxxx
This patch converts the driver to use netif_msg_* for printing messages.
Tested IA32.

--- linux-2.6.3/drivers/net/jon.pcnet32.c       Thu Feb 19 09:14:03 2004
+++ linux-2.6.3/drivers/net/pcnet32.c   Thu Feb 19 14:20:28 2004
@@ -79,7 +79,7 @@
 
 
 
-static int pcnet32_debug = 1;
+static int pcnet32_debug = 0;
 static int tx_start = 1; /* Mapping -- 0:20, 1:64, 2:128, 3:~220 (depends on 
chip vers) */
 static int pcnet32vlb;  /* check for VLB cards ? */
 
@@ -327,6 +327,7 @@
     struct net_device  *next;
     struct mii_if_info mii_if;
     struct timer_list  watchdog_timer;
+    u32                        msg_enable;     /* debug message level */
 };
 
 static void pcnet32_probe_vlbus(void);
@@ -552,7 +553,7 @@
     }
 
     chip_version = a->read_csr(ioaddr, 88) | (a->read_csr(ioaddr,89) << 16);
-    if (pcnet32_debug > 2)
+    if (pcnet32_debug & NETIF_MSG_PROBE)
        printk(KERN_INFO "  PCnet chip version is %#x.\n", chip_version);
     if ((chip_version & 0xfff) != 0x003)
            goto err_release_region;
@@ -606,7 +607,7 @@
        media &= ~3;
        media |= 1;
 #endif
-       if (pcnet32_debug > 2)
+       if (pcnet32_debug & NETIF_MSG_PROBE)
            printk(KERN_DEBUG PFX "media reset to %#x.\n",  media);
        a->write_bcr(ioaddr, 49, media);
        break;
@@ -731,6 +732,7 @@
     lp->dxsuflo = dxsuflo;
     lp->ltint = ltint;
     lp->mii = mii;
+    lp->msg_enable = pcnet32_debug;
     if ((cards_found >= MAX_UNITS) || (options[cards_found] > 
sizeof(options_mapping)))
        lp->options = PCNET32_PORT_ASEL;
     else
@@ -858,7 +860,7 @@
     /* switch pcnet32 to 32bit mode */
     lp->a.write_bcr (ioaddr, 20, 2);
 
-    if (pcnet32_debug > 1)
+    if (netif_msg_ifup(lp))
        printk(KERN_DEBUG "%s: pcnet32_open() irq %d tx/rx rings %#x/%#x init 
%#x.\n",
               dev->name, dev->irq,
               (u32) (lp->dma_addr + offsetof(struct pcnet32_private, tx_ring)),
@@ -952,7 +954,7 @@
      */
     lp->a.write_csr (ioaddr, 0, 0x0042);
 
-    if (pcnet32_debug > 2)
+    if (netif_msg_ifup(lp))
        printk(KERN_DEBUG "%s: pcnet32 open after %d ticks, init block %#x csr0 
%4.4x.\n",
               dev->name, i, (u32) (lp->dma_addr + offsetof(struct 
pcnet32_private, init_block)),
               lp->a.read_csr(ioaddr, 0));
@@ -1070,7 +1072,7 @@
               dev->name, lp->a.read_csr(ioaddr, 0));
        lp->a.write_csr (ioaddr, 0, 0x0004);
        lp->stats.tx_errors++;
-       if (pcnet32_debug > 2) {
+       if (netif_msg_tx_err(lp)) {
            int i;
            printk(KERN_DEBUG " Ring data dump: dirty_tx %d cur_tx %d%s cur_rx 
%d.",
               lp->dirty_tx, lp->cur_tx, lp->tx_full ? " (full)" : "",
@@ -1103,7 +1105,7 @@
     int entry;
     unsigned long flags;
 
-    if (pcnet32_debug > 3) {
+    if (netif_msg_tx_queued(lp)) {
        printk(KERN_DEBUG "%s: pcnet32_start_xmit() called, csr0 %4.4x.\n",
               dev->name, lp->a.read_csr(ioaddr, 0));
     }
@@ -1192,7 +1194,7 @@
 
        must_restart = 0;
 
-       if (pcnet32_debug > 5)
+       if (netif_msg_intr(lp))
            printk(KERN_DEBUG "%s: interrupt  csr0=%#2.2x new csr=%#2.2x.\n",
                   dev->name, csr0, lp->a.read_csr (ioaddr, 0));
 
@@ -1307,7 +1309,7 @@
     lp->a.write_csr (ioaddr, 0, 0x7940);
     lp->a.write_rap (ioaddr,rap);
     
-    if (pcnet32_debug > 4)
+    if (netif_msg_intr(lp))
        printk(KERN_DEBUG "%s: exiting interrupt, csr0=%#4.4x.\n",
               dev->name, lp->a.read_csr (ioaddr, 0));
 
@@ -1431,7 +1433,7 @@
 
     lp->stats.rx_missed_errors = lp->a.read_csr (ioaddr, 112);
 
-    if (pcnet32_debug > 1)
+    if (netif_msg_ifdown(lp))
        printk(KERN_DEBUG "%s: Shutting down ethercard, status was %2.2x.\n",
               dev->name, lp->a.read_csr (ioaddr, 0));
 
@@ -1660,7 +1662,7 @@
        /* get message-level */
        case ETHTOOL_GMSGLVL: {
                struct ethtool_value edata = {ETHTOOL_GMSGLVL};
-               edata.data = pcnet32_debug;
+               edata.data = lp->msg_enable;
                if (copy_to_user(useraddr, &edata, sizeof(edata)))
                        return -EFAULT;
                return 0;
@@ -1670,7 +1672,7 @@
                struct ethtool_value edata;
                if (copy_from_user(&edata, useraddr, sizeof(edata)))
                        return -EFAULT;
-               pcnet32_debug = edata.data;
+               lp->msg_enable = edata.data;
                return 0;
        }
        default:

-- 
Don Fry
brazilnut@xxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH] 2.6.3 pcnet32.c convert to use netif_msg_*, Don Fry <=