fbdev: section cleanup in w100fb
[safe/jmp/linux-2.6] / drivers / net / tehuti.c
index 82e522a..20ab161 100644 (file)
@@ -66,7 +66,7 @@
 
 #include "tehuti.h"
 
-static struct pci_device_id __devinitdata bdx_pci_tbl[] = {
+static DEFINE_PCI_DEVICE_TABLE(bdx_pci_tbl) = {
        {0x1FC9, 0x3009, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
        {0x1FC9, 0x3010, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
        {0x1FC9, 0x3014, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
@@ -419,9 +419,11 @@ static int bdx_hw_start(struct bdx_priv *priv)
        WRITE_REG(priv, regGMAC_RXF_A, GMAC_RX_FILTER_OSEN |
                  GMAC_RX_FILTER_AM | GMAC_RX_FILTER_AB);
 
-#define BDX_IRQ_TYPE   ((priv->nic->irq_type == IRQ_MSI)?0:IRQF_SHARED)
-       if ((rc = request_irq(priv->pdev->irq, bdx_isr_napi, BDX_IRQ_TYPE,
-                        ndev->name, ndev)))
+#define BDX_IRQ_TYPE   ((priv->nic->irq_type == IRQ_MSI) ? 0 : IRQF_SHARED)
+
+       rc = request_irq(priv->pdev->irq, bdx_isr_napi, BDX_IRQ_TYPE,
+                        ndev->name, ndev);
+       if (rc)
                goto err_irq;
        bdx_enable_interrupts(priv);
 
@@ -603,18 +605,15 @@ static int bdx_open(struct net_device *ndev)
        if (netif_running(ndev))
                netif_stop_queue(priv->ndev);
 
-       if ((rc = bdx_tx_init(priv)))
-               goto err;
-
-       if ((rc = bdx_rx_init(priv)))
-               goto err;
-
-       if ((rc = bdx_fw_load(priv)))
+       if ((rc = bdx_tx_init(priv)) ||
+           (rc = bdx_rx_init(priv)) ||
+           (rc = bdx_fw_load(priv)))
                goto err;
 
        bdx_rx_alloc_skbs(priv, &priv->rxf_fifo0);
 
-       if ((rc = bdx_hw_start(priv)))
+       rc = bdx_hw_start(priv);
+       if (rc)
                goto err;
 
        napi_enable(&priv->napi);
@@ -809,7 +808,7 @@ static void bdx_setmulti(struct net_device *ndev)
                        WRITE_REG(priv, regRX_MCST_HASH0 + i * 4, ~0);
        } else if (!netdev_mc_empty(ndev)) {
                u8 hash;
-               struct dev_mc_list *mclist;
+               struct netdev_hw_addr *ha;
                u32 reg, val;
 
                /* set IMF to deny all multicast frames */
@@ -825,13 +824,11 @@ static void bdx_setmulti(struct net_device *ndev)
                /* TBD: sort addreses and write them in ascending order
                 * into RX_MAC_MCST regs. we skip this phase now and accept ALL
                 * multicast frames throu IMF */
-               mclist = ndev->mc_list;
-
                /* accept the rest of addresses throu IMF */
-               for (; mclist; mclist = mclist->next) {
+               netdev_for_each_mc_addr(ha, ndev) {
                        hash = 0;
                        for (i = 0; i < ETH_ALEN; i++)
-                               hash ^= mclist->dmi_addr[i];
+                               hash ^= ha->addr[i];
                        reg = regRX_MCST_HASH0 + ((hash >> 5) << 2);
                        val = READ_REG(priv, reg);
                        val |= (1 << (hash % 32));
@@ -1027,10 +1024,9 @@ static int bdx_rx_init(struct bdx_priv *priv)
                          regRXF_CFG0_0, regRXF_CFG1_0,
                          regRXF_RPTR_0, regRXF_WPTR_0))
                goto err_mem;
-       if (!
-           (priv->rxdb =
-            bdx_rxdb_create(priv->rxf_fifo0.m.memsz /
-                            sizeof(struct rxf_desc))))
+       priv->rxdb = bdx_rxdb_create(priv->rxf_fifo0.m.memsz /
+                                    sizeof(struct rxf_desc));
+       if (!priv->rxdb)
                goto err_mem;
 
        priv->rxf_fifo0.m.pktsz = priv->ndev->mtu + VLAN_ETH_HLEN;
@@ -1114,7 +1110,8 @@ static void bdx_rx_alloc_skbs(struct bdx_priv *priv, struct rxf_fifo *f)
        ENTER;
        dno = bdx_rxdb_available(db) - 1;
        while (dno > 0) {
-               if (!(skb = dev_alloc_skb(f->m.pktsz + NET_IP_ALIGN))) {
+               skb = dev_alloc_skb(f->m.pktsz + NET_IP_ALIGN);
+               if (!skb) {
                        pr_err("NO MEM: dev_alloc_skb failed\n");
                        break;
                }
@@ -1306,7 +1303,6 @@ static int bdx_rx_receive(struct bdx_priv *priv, struct rxd_fifo *f, int budget)
                priv->net_stats.rx_bytes += len;
 
                skb_put(skb, len);
-               skb->dev = priv->ndev;
                skb->ip_summed = CHECKSUM_UNNECESSARY;
                skb->protocol = eth_type_trans(skb, priv->ndev);
 
@@ -1512,7 +1508,7 @@ bdx_tx_map_skb(struct bdx_priv *priv, struct sk_buff *skb,
        int nr_frags = skb_shinfo(skb)->nr_frags;
        int i;
 
-       db->wptr->len = skb->len - skb->data_len;
+       db->wptr->len = skb_headlen(skb);
        db->wptr->addr.dma = pci_map_single(priv->pdev, skb->data,
                                            db->wptr->len, PCI_DMA_TODEVICE);
        pbl->len = CPU_CHIP_SWAP32(db->wptr->len);
@@ -1606,7 +1602,7 @@ static inline int bdx_tx_space(struct bdx_priv *priv)
        fsize = f->m.rptr - f->m.wptr;
        if (fsize <= 0)
                fsize = f->m.memsz + fsize;
-       return (fsize);
+       return fsize;
 }
 
 /* bdx_tx_transmit - send packet to NIC
@@ -1854,7 +1850,7 @@ static void bdx_tx_push_desc(struct bdx_priv *priv, void *data, int size)
  * @data - desc's data
  * @size - desc's size
  *
- * NOTE: this func does check for available space and, if neccessary, waits for
+ * NOTE: this func does check for available space and, if necessary, waits for
  *   NIC to read existing data before writing new one.
  */
 static void bdx_tx_push_desc_safe(struct bdx_priv *priv, void *data, int size)
@@ -1934,8 +1930,9 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                RET(-ENOMEM);
 
     /************** pci *****************/
-       if ((err = pci_enable_device(pdev)))    /* it trigers interrupt, dunno why. */
-               goto err_pci;                   /* it's not a problem though */
+       err = pci_enable_device(pdev);
+       if (err)                        /* it triggers interrupt, dunno why. */
+               goto err_pci;           /* it's not a problem though */
 
        if (!(err = pci_set_dma_mask(pdev, DMA_BIT_MASK(64))) &&
            !(err = pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(64)))) {
@@ -1949,7 +1946,8 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                pci_using_dac = 0;
        }
 
-       if ((err = pci_request_regions(pdev, BDX_DRV_NAME)))
+       err = pci_request_regions(pdev, BDX_DRV_NAME);
+       if (err)
                goto err_dma;
 
        pci_set_master(pdev);
@@ -1960,7 +1958,8 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                pr_err("no MMIO resource\n");
                goto err_out_res;
        }
-       if ((regionSize = pci_resource_len(pdev, 0)) < BDX_REGS_SIZE) {
+       regionSize = pci_resource_len(pdev, 0);
+       if (regionSize < BDX_REGS_SIZE) {
                err = -EIO;
                pr_err("MMIO resource (%x) too small\n", regionSize);
                goto err_out_res;
@@ -1992,7 +1991,8 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        nic->irq_type = IRQ_INTX;
 #ifdef BDX_MSI
        if ((readl(nic->regs + FPGA_VER) & 0xFFF) >= 378) {
-               if ((err = pci_enable_msi(pdev)))
+               err = pci_enable_msi(pdev);
+               if (err)
                        pr_err("Can't eneble msi. error is %d\n", err);
                else
                        nic->irq_type = IRQ_MSI;
@@ -2002,7 +2002,8 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 
     /************** netdev **************/
        for (port = 0; port < nic->port_num; port++) {
-               if (!(ndev = alloc_etherdev(sizeof(struct bdx_priv)))) {
+               ndev = alloc_etherdev(sizeof(struct bdx_priv));
+               if (!ndev) {
                        err = -ENOMEM;
                        pr_err("alloc_etherdev failed\n");
                        goto err_out_iomap;
@@ -2032,7 +2033,6 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        /************** priv ****************/
                priv = nic->priv[port] = netdev_priv(ndev);
 
-               memset(priv, 0, sizeof(struct bdx_priv));
                priv->pBdxRegs = nic->regs + port * 0x8000;
                priv->port = port;
                priv->pdev = pdev;
@@ -2075,7 +2075,8 @@ bdx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                        goto err_out_iomap;
                }
                SET_NETDEV_DEV(ndev, &pdev->dev);
-               if ((err = register_netdev(ndev))) {
+               err = register_netdev(ndev);
+               if (err) {
                        pr_err("register_netdev failed\n");
                        goto err_out_free;
                }
@@ -2290,13 +2291,13 @@ bdx_set_coalesce(struct net_device *netdev, struct ethtool_coalesce *ecoal)
 /* Convert RX fifo size to number of pending packets */
 static inline int bdx_rx_fifo_size_to_packets(int rx_size)
 {
-       return ((FIFO_SIZE * (1 << rx_size)) / sizeof(struct rxf_desc));
+       return (FIFO_SIZE * (1 << rx_size)) / sizeof(struct rxf_desc);
 }
 
 /* Convert TX fifo size to number of pending packets */
 static inline int bdx_tx_fifo_size_to_packets(int tx_size)
 {
-       return ((FIFO_SIZE * (1 << tx_size)) / BDX_TXF_DESC_SZ);
+       return (FIFO_SIZE * (1 << tx_size)) / BDX_TXF_DESC_SZ;
 }
 
 /*
@@ -2388,10 +2389,10 @@ static int bdx_get_sset_count(struct net_device *netdev, int stringset)
        case ETH_SS_STATS:
                BDX_ASSERT(ARRAY_SIZE(bdx_stat_names)
                           != sizeof(struct bdx_stats) / sizeof(u64));
-               return ((priv->stats_flag) ? ARRAY_SIZE(bdx_stat_names) : 0);
-       default:
-               return -EINVAL;
+               return (priv->stats_flag) ? ARRAY_SIZE(bdx_stat_names)  : 0;
        }
+
+       return -EINVAL;
 }
 
 /*