]> nv-tegra.nvidia Code Review - linux-2.6.git/blobdiff - drivers/net/epic100.c
drivers/net: Convert unbounded kzalloc calls to kcalloc
[linux-2.6.git] / drivers / net / epic100.c
index 7a567201e8295f8c17763c2f350e96f3b8495a0c..57c8ac0ef3f1232d41e4f00399eb27b18170dcf6 100644 (file)
@@ -87,6 +87,7 @@ static int rx_copybreak;
 #include <linux/bitops.h>
 #include <asm/io.h>
 #include <asm/uaccess.h>
+#include <asm/byteorder.h>
 
 /* These identify the driver base version and may not be removed. */
 static char version[] __devinitdata =
@@ -230,7 +231,7 @@ static const u16 media2miictl[16] = {
  * The EPIC100 Rx and Tx buffer descriptors.  Note that these
  * really ARE host-endian; it's not a misannotation.  We tell
  * the card to byteswap them internally on big-endian hosts -
- * look for #ifdef CONFIG_BIG_ENDIAN in epic_open().
+ * look for #ifdef __BIG_ENDIAN in epic_open().
  */
 
 struct epic_tx_desc {
@@ -277,7 +278,6 @@ struct epic_private {
        struct pci_dev *pci_dev;                        /* PCI bus location. */
        int chip_id, chip_flags;
 
-       struct net_device_stats stats;
        struct timer_list timer;                        /* Media selection timer. */
        int tx_threshold;
        unsigned char mc_filter[8];
@@ -652,7 +652,6 @@ static void mdio_write(struct net_device *dev, int phy_id, int loc, int value)
                if ((inl(ioaddr + MIICtrl) & MII_WRITEOP) == 0)
                        break;
        }
-       return;
 }
 
 
@@ -691,7 +690,7 @@ static int epic_open(struct net_device *dev)
                outl((inl(ioaddr + NVCTL) & ~0x003C) | 0x4800, ioaddr + NVCTL);
 
        /* Tell the chip to byteswap descriptors on big-endian hosts */
-#ifdef CONFIG_BIG_ENDIAN
+#ifdef __BIG_ENDIAN
        outl(0x4432 | (RX_FIFO_THRESH<<8), ioaddr + GENCTL);
        inl(ioaddr + GENCTL);
        outl(0x0432 | (RX_FIFO_THRESH<<8), ioaddr + GENCTL);
@@ -770,7 +769,6 @@ static int epic_open(struct net_device *dev)
 static void epic_pause(struct net_device *dev)
 {
        long ioaddr = dev->base_addr;
-       struct epic_private *ep = netdev_priv(dev);
 
        netif_stop_queue (dev);
 
@@ -781,9 +779,9 @@ static void epic_pause(struct net_device *dev)
 
        /* Update the error counts. */
        if (inw(ioaddr + COMMAND) != 0xffff) {
-               ep->stats.rx_missed_errors += inb(ioaddr + MPCNT);
-               ep->stats.rx_frame_errors += inb(ioaddr + ALICNT);
-               ep->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
+               dev->stats.rx_missed_errors += inb(ioaddr + MPCNT);
+               dev->stats.rx_frame_errors += inb(ioaddr + ALICNT);
+               dev->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
        }
 
        /* Remove the packets on the Rx queue. */
@@ -807,7 +805,7 @@ static void epic_restart(struct net_device *dev)
        for (i = 16; i > 0; i--)
                outl(0x0008, ioaddr + TEST1);
 
-#ifdef CONFIG_BIG_ENDIAN
+#ifdef __BIG_ENDIAN
        outl(0x0432 | (RX_FIFO_THRESH<<8), ioaddr + GENCTL);
 #else
        outl(0x0412 | (RX_FIFO_THRESH<<8), ioaddr + GENCTL);
@@ -840,7 +838,6 @@ static void epic_restart(struct net_device *dev)
                   " interrupt %4.4x.\n",
                   dev->name, (int)inl(ioaddr + COMMAND), (int)inl(ioaddr + GENCTL),
                   (int)inl(ioaddr + INTSTAT));
-       return;
 }
 
 static void check_media(struct net_device *dev)
@@ -901,15 +898,15 @@ static void epic_tx_timeout(struct net_device *dev)
                }
        }
        if (inw(ioaddr + TxSTAT) & 0x10) {              /* Tx FIFO underflow. */
-               ep->stats.tx_fifo_errors++;
+               dev->stats.tx_fifo_errors++;
                outl(RestartTx, ioaddr + COMMAND);
        } else {
                epic_restart(dev);
                outl(TxQueued, dev->base_addr + COMMAND);
        }
 
-       dev->trans_start = jiffies;
-       ep->stats.tx_errors++;
+       dev->trans_start = jiffies; /* prevent tx timeout */
+       dev->stats.tx_errors++;
        if (!ep->tx_full)
                netif_wake_queue(dev);
 }
@@ -958,7 +955,6 @@ static void epic_init_ring(struct net_device *dev)
                        (i+1)*sizeof(struct epic_tx_desc);
        }
        ep->tx_ring[i-1].next = ep->tx_ring_dma;
-       return;
 }
 
 static netdev_tx_t epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
@@ -1006,7 +1002,6 @@ static netdev_tx_t epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
        /* Trigger an immediate transmit demand. */
        outl(TxQueued, dev->base_addr + COMMAND);
 
-       dev->trans_start = jiffies;
        if (debug > 4)
                printk(KERN_DEBUG "%s: Queued Tx packet size %d to slot %d, "
                           "flag %2.2x Tx status %8.8x.\n",
@@ -1019,7 +1014,7 @@ static netdev_tx_t epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
 static void epic_tx_error(struct net_device *dev, struct epic_private *ep,
                          int status)
 {
-       struct net_device_stats *stats = &ep->stats;
+       struct net_device_stats *stats = &dev->stats;
 
 #ifndef final_version
        /* There was an major error, log it. */
@@ -1056,9 +1051,9 @@ static void epic_tx(struct net_device *dev, struct epic_private *ep)
                        break;  /* It still hasn't been Txed */
 
                if (likely(txstatus & 0x0001)) {
-                       ep->stats.collisions += (txstatus >> 8) & 15;
-                       ep->stats.tx_packets++;
-                       ep->stats.tx_bytes += ep->tx_skbuff[entry]->len;
+                       dev->stats.collisions += (txstatus >> 8) & 15;
+                       dev->stats.tx_packets++;
+                       dev->stats.tx_bytes += ep->tx_skbuff[entry]->len;
                } else
                        epic_tx_error(dev, ep, txstatus);
 
@@ -1128,12 +1123,12 @@ static irqreturn_t epic_interrupt(int irq, void *dev_instance)
                        goto out;
 
                /* Always update the error counts to avoid overhead later. */
-               ep->stats.rx_missed_errors += inb(ioaddr + MPCNT);
-               ep->stats.rx_frame_errors += inb(ioaddr + ALICNT);
-               ep->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
+               dev->stats.rx_missed_errors += inb(ioaddr + MPCNT);
+               dev->stats.rx_frame_errors += inb(ioaddr + ALICNT);
+               dev->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
 
                if (status & TxUnderrun) { /* Tx FIFO underflow. */
-                       ep->stats.tx_fifo_errors++;
+                       dev->stats.tx_fifo_errors++;
                        outl(ep->tx_threshold += 128, ioaddr + TxThresh);
                        /* Restart the transmit process. */
                        outl(RestartTx, ioaddr + COMMAND);
@@ -1186,10 +1181,10 @@ static int epic_rx(struct net_device *dev, int budget)
                        if (status & 0x2000) {
                                printk(KERN_WARNING "%s: Oversized Ethernet frame spanned "
                                           "multiple buffers, status %4.4x!\n", dev->name, status);
-                               ep->stats.rx_length_errors++;
+                               dev->stats.rx_length_errors++;
                        } else if (status & 0x0006)
                                /* Rx Frame errors are counted in hardware. */
-                               ep->stats.rx_errors++;
+                               dev->stats.rx_errors++;
                } else {
                        /* Malloc up new buffer, compatible with net-2e. */
                        /* Omit the four octet CRC from the length. */
@@ -1226,8 +1221,8 @@ static int epic_rx(struct net_device *dev, int budget)
                        }
                        skb->protocol = eth_type_trans(skb, dev);
                        netif_receive_skb(skb);
-                       ep->stats.rx_packets++;
-                       ep->stats.rx_bytes += pkt_len;
+                       dev->stats.rx_packets++;
+                       dev->stats.rx_bytes += pkt_len;
                }
                work_done++;
                entry = (++ep->cur_rx) % RX_RING_SIZE;
@@ -1262,7 +1257,7 @@ static void epic_rx_err(struct net_device *dev, struct epic_private *ep)
        if (status == EpicRemoved)
                return;
        if (status & RxOverflow)        /* Missed a Rx frame. */
-               ep->stats.rx_errors++;
+               dev->stats.rx_errors++;
        if (status & (RxOverflow | RxFull))
                outw(RxQueued, ioaddr + COMMAND);
 }
@@ -1360,17 +1355,16 @@ static int epic_close(struct net_device *dev)
 
 static struct net_device_stats *epic_get_stats(struct net_device *dev)
 {
-       struct epic_private *ep = netdev_priv(dev);
        long ioaddr = dev->base_addr;
 
        if (netif_running(dev)) {
                /* Update the error counts. */
-               ep->stats.rx_missed_errors += inb(ioaddr + MPCNT);
-               ep->stats.rx_frame_errors += inb(ioaddr + ALICNT);
-               ep->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
+               dev->stats.rx_missed_errors += inb(ioaddr + MPCNT);
+               dev->stats.rx_frame_errors += inb(ioaddr + ALICNT);
+               dev->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
        }
 
-       return &ep->stats;
+       return &dev->stats;
 }
 
 /* Set or clear the multicast filter for this adaptor.
@@ -1399,12 +1393,12 @@ static void set_rx_mode(struct net_device *dev)
                outl(0x0004, ioaddr + RxCtrl);
                return;
        } else {                                        /* Never executed, for now. */
-               struct dev_mc_list *mclist;
+               struct netdev_hw_addr *ha;
 
                memset(mc_filter, 0, sizeof(mc_filter));
-               netdev_for_each_mc_addr(mclist, dev) {
+               netdev_for_each_mc_addr(ha, dev) {
                        unsigned int bit_nr =
-                               ether_crc_le(ETH_ALEN, mclist->dmi_addr) & 0x3f;
+                               ether_crc_le(ETH_ALEN, ha->addr) & 0x3f;
                        mc_filter[bit_nr >> 3] |= (1 << bit_nr);
                }
        }
@@ -1414,7 +1408,6 @@ static void set_rx_mode(struct net_device *dev)
                        outw(((u16 *)mc_filter)[i], ioaddr + MC0 + i*4);
                memcpy(ep->mc_filter, mc_filter, sizeof(mc_filter));
        }
-       return;
 }
 
 static void netdev_get_drvinfo (struct net_device *dev, struct ethtool_drvinfo *info)