net: arc: trivial: cleanup the emac driver
authorCaesar Wang <wxt@rock-chips.com>
Mon, 14 Mar 2016 08:01:55 +0000 (16:01 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Mar 2016 23:28:01 +0000 (19:28 -0400)
This patch will make the driver more readability

The emac has the error and warnings if you run
'scripts/checkpatch.pl -f --subjective xxx' to check.

Let's clean up such trivial details.

Signed-off-by: Caesar Wang <wxt@rock-chips.com>
Cc: Jiri Kosina <trivial@kernel.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Alexander Kochetkov <al.kochet@gmail.com>
Cc: netdev@vger.kernel.org
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/arc/emac.h
drivers/net/ethernet/arc/emac_main.c
drivers/net/ethernet/arc/emac_mdio.c
drivers/net/ethernet/arc/emac_rockchip.c

index 1a4040397a4bc1b213a37b46744751303750e6b4..ca562bc034c390f3f7f0be66db2c3c245d2cf390 100644 (file)
 #include <linux/clk.h>
 
 /* STATUS and ENABLE Register bit masks */
-#define TXINT_MASK     (1<<0)  /* Transmit interrupt */
-#define RXINT_MASK     (1<<1)  /* Receive interrupt */
-#define ERR_MASK       (1<<2)  /* Error interrupt */
-#define TXCH_MASK      (1<<3)  /* Transmit chaining error interrupt */
-#define MSER_MASK      (1<<4)  /* Missed packet counter error */
-#define RXCR_MASK      (1<<8)  /* RXCRCERR counter rolled over  */
-#define RXFR_MASK      (1<<9)  /* RXFRAMEERR counter rolled over */
-#define RXFL_MASK      (1<<10) /* RXOFLOWERR counter rolled over */
-#define MDIO_MASK      (1<<12) /* MDIO complete interrupt */
-#define TXPL_MASK      (1<<31) /* Force polling of BD by EMAC */
+#define TXINT_MASK     (1 << 0)        /* Transmit interrupt */
+#define RXINT_MASK     (1 << 1)        /* Receive interrupt */
+#define ERR_MASK       (1 << 2)        /* Error interrupt */
+#define TXCH_MASK      (1 << 3)        /* Transmit chaining error interrupt */
+#define MSER_MASK      (1 << 4)        /* Missed packet counter error */
+#define RXCR_MASK      (1 << 8)        /* RXCRCERR counter rolled over  */
+#define RXFR_MASK      (1 << 9)        /* RXFRAMEERR counter rolled over */
+#define RXFL_MASK      (1 << 10)       /* RXOFLOWERR counter rolled over */
+#define MDIO_MASK      (1 << 12)       /* MDIO complete interrupt */
+#define TXPL_MASK      (1 << 31)       /* Force polling of BD by EMAC */
 
 /* CONTROL Register bit masks */
-#define EN_MASK                (1<<0)  /* VMAC enable */
-#define TXRN_MASK      (1<<3)  /* TX enable */
-#define RXRN_MASK      (1<<4)  /* RX enable */
-#define DSBC_MASK      (1<<8)  /* Disable receive broadcast */
-#define ENFL_MASK      (1<<10) /* Enable Full-duplex */
-#define PROM_MASK      (1<<11) /* Promiscuous mode */
+#define EN_MASK                (1 << 0)        /* VMAC enable */
+#define TXRN_MASK      (1 << 3)        /* TX enable */
+#define RXRN_MASK      (1 << 4)        /* RX enable */
+#define DSBC_MASK      (1 << 8)        /* Disable receive broadcast */
+#define ENFL_MASK      (1 << 10)       /* Enable Full-duplex */
+#define PROM_MASK      (1 << 11)       /* Promiscuous mode */
 
 /* Buffer descriptor INFO bit masks */
-#define OWN_MASK       (1<<31) /* 0-CPU owns buffer, 1-EMAC owns buffer */
-#define FIRST_MASK     (1<<16) /* First buffer in chain */
-#define LAST_MASK      (1<<17) /* Last buffer in chain */
+#define OWN_MASK       (1 << 31)       /* 0-CPU or 1-EMAC owns buffer */
+#define FIRST_MASK     (1 << 16)       /* First buffer in chain */
+#define LAST_MASK      (1 << 17)       /* Last buffer in chain */
 #define LEN_MASK       0x000007FF      /* last 11 bits */
-#define CRLS           (1<<21)
-#define DEFR           (1<<22)
-#define DROP           (1<<23)
-#define RTRY           (1<<24)
-#define LTCL           (1<<28)
-#define UFLO           (1<<29)
+#define CRLS           (1 << 21)
+#define DEFR           (1 << 22)
+#define DROP           (1 << 23)
+#define RTRY           (1 << 24)
+#define LTCL           (1 << 28)
+#define UFLO           (1 << 29)
 
 #define FOR_EMAC       OWN_MASK
 #define FOR_CPU                0
@@ -66,7 +66,7 @@ enum {
        R_MDIO,
 };
 
-#define TX_TIMEOUT             (400*HZ/1000)   /* Transmission timeout */
+#define TX_TIMEOUT             (400 * HZ / 1000) /* Transmission timeout */
 
 #define ARC_EMAC_NAPI_WEIGHT   40              /* Workload for NAPI */
 
@@ -196,6 +196,7 @@ static inline unsigned int arc_reg_get(struct arc_emac_priv *priv, int reg)
 static inline void arc_reg_or(struct arc_emac_priv *priv, int reg, int mask)
 {
        unsigned int value = arc_reg_get(priv, reg);
+
        arc_reg_set(priv, reg, value | mask);
 }
 
@@ -211,6 +212,7 @@ static inline void arc_reg_or(struct arc_emac_priv *priv, int reg, int mask)
 static inline void arc_reg_clr(struct arc_emac_priv *priv, int reg, int mask)
 {
        unsigned int value = arc_reg_get(priv, reg);
+
        arc_reg_set(priv, reg, value & ~mask);
 }
 
index 6446af1403f70be8101625ed9a8b03d3de17b792..a3a9392a4954397a13e0e882626e7dfe4d4da900 100644 (file)
@@ -26,7 +26,6 @@
 
 #include "emac.h"
 
-
 /**
  * arc_emac_tx_avail - Return the number of available slots in the tx ring.
  * @priv: Pointer to ARC EMAC private data structure.
@@ -66,7 +65,7 @@ static void arc_emac_adjust_link(struct net_device *ndev)
        if (priv->duplex != phy_dev->duplex) {
                reg = arc_reg_get(priv, R_CTRL);
 
-               if (DUPLEX_FULL == phy_dev->duplex)
+               if (phy_dev->duplex == DUPLEX_FULL)
                        reg |= ENFL_MASK;
                else
                        reg &= ~ENFL_MASK;
@@ -466,9 +465,9 @@ static int arc_emac_open(struct net_device *ndev)
 
        /* Set CONTROL */
        arc_reg_set(priv, R_CTRL,
-                    (RX_BD_NUM << 24) |        /* RX BD table length */
-                    (TX_BD_NUM << 16) |        /* TX BD table length */
-                    TXRN_MASK | RXRN_MASK);
+                   (RX_BD_NUM << 24) | /* RX BD table length */
+                   (TX_BD_NUM << 16) | /* TX BD table length */
+                   TXRN_MASK | RXRN_MASK);
 
        napi_enable(&priv->napi);
 
@@ -533,8 +532,10 @@ static void arc_free_tx_queue(struct net_device *ndev)
                struct buffer_state *tx_buff = &priv->tx_buff[i];
 
                if (tx_buff->skb) {
-                       dma_unmap_single(&ndev->dev, dma_unmap_addr(tx_buff, addr),
-                                        dma_unmap_len(tx_buff, len), DMA_TO_DEVICE);
+                       dma_unmap_single(&ndev->dev,
+                                        dma_unmap_addr(tx_buff, addr),
+                                        dma_unmap_len(tx_buff, len),
+                                        DMA_TO_DEVICE);
 
                        /* return the sk_buff to system */
                        dev_kfree_skb_irq(tx_buff->skb);
@@ -562,8 +563,10 @@ static void arc_free_rx_queue(struct net_device *ndev)
                struct buffer_state *rx_buff = &priv->rx_buff[i];
 
                if (rx_buff->skb) {
-                       dma_unmap_single(&ndev->dev, dma_unmap_addr(rx_buff, addr),
-                                       dma_unmap_len(rx_buff, len), DMA_FROM_DEVICE);
+                       dma_unmap_single(&ndev->dev,
+                                        dma_unmap_addr(rx_buff, addr),
+                                        dma_unmap_len(rx_buff, len),
+                                        DMA_FROM_DEVICE);
 
                        /* return the sk_buff to system */
                        dev_kfree_skb_irq(rx_buff->skb);
@@ -717,8 +720,8 @@ static void arc_emac_set_address_internal(struct net_device *ndev)
        struct arc_emac_priv *priv = netdev_priv(ndev);
        unsigned int addr_low, addr_hi;
 
-       addr_low = le32_to_cpu(*(__le32 *) &ndev->dev_addr[0]);
-       addr_hi = le16_to_cpu(*(__le16 *) &ndev->dev_addr[4]);
+       addr_low = le32_to_cpu(*(__le32 *)&ndev->dev_addr[0]);
+       addr_hi = le16_to_cpu(*(__le16 *)&ndev->dev_addr[4]);
 
        arc_reg_set(priv, R_ADDRL, addr_low);
        arc_reg_set(priv, R_ADDRH, addr_hi);
@@ -774,7 +777,6 @@ int arc_emac_probe(struct net_device *ndev, int interface)
        unsigned int id, clock_frequency, irq;
        int err;
 
-
        /* Get PHY from device tree */
        phy_node = of_parse_phandle(dev->of_node, "phy", 0);
        if (!phy_node) {
@@ -796,7 +798,6 @@ int arc_emac_probe(struct net_device *ndev, int interface)
                return -ENODEV;
        }
 
-
        ndev->netdev_ops = &arc_emac_netdev_ops;
        ndev->ethtool_ops = &arc_emac_ethtool_ops;
        ndev->watchdog_timeo = TX_TIMEOUT;
@@ -807,9 +808,9 @@ int arc_emac_probe(struct net_device *ndev, int interface)
        priv->dev = dev;
 
        priv->regs = devm_ioremap_resource(dev, &res_regs);
-       if (IS_ERR(priv->regs)) {
+       if (IS_ERR(priv->regs))
                return PTR_ERR(priv->regs);
-       }
+
        dev_dbg(dev, "Registers base address is 0x%p\n", priv->regs);
 
        if (priv->clk) {
@@ -930,10 +931,8 @@ int arc_emac_remove(struct net_device *ndev)
        unregister_netdev(ndev);
        netif_napi_del(&priv->napi);
 
-       if (!IS_ERR(priv->clk)) {
+       if (!IS_ERR(priv->clk))
                clk_disable_unprepare(priv->clk);
-       }
-
 
        return 0;
 }
index caf704264eba906db6e5376bda4369f7ed8d5281..16419f550eff0643665fc6e9b24b96a22eb2ea45 100644 (file)
@@ -94,7 +94,7 @@ static int arc_mdio_write(struct mii_bus *bus, int phy_addr,
                phy_addr, reg_num, value);
 
        arc_reg_set(priv, R_MDIO,
-                    0x50020000 | (phy_addr << 23) | (reg_num << 18) | value);
+                   0x50020000 | (phy_addr << 23) | (reg_num << 18) | value);
 
        return arc_mdio_complete_wait(priv);
 }
index 85e821ccfcd2994ec4ad30297b4b9571980aa8cb..e278e3d96ee010235ec4d21696a996b68cb3ef18 100644 (file)
@@ -50,7 +50,7 @@ static void emac_rockchip_set_mac_speed(void *priv, unsigned int speed)
        u32 data;
        int err = 0;
 
-       switch(speed) {
+       switch (speed) {
        case 10:
                data = (1 << (speed_offset + 16)) | (0 << speed_offset);
                break;
@@ -83,9 +83,18 @@ static const struct emac_rockchip_soc_data emac_rk3188_emac_data = {
 };
 
 static const struct of_device_id emac_rockchip_dt_ids[] = {
-       { .compatible = "rockchip,rk3036-emac", .data = &emac_rk3036_emac_data },
-       { .compatible = "rockchip,rk3066-emac", .data = &emac_rk3066_emac_data },
-       { .compatible = "rockchip,rk3188-emac", .data = &emac_rk3188_emac_data },
+       {
+               .compatible = "rockchip,rk3036-emac",
+               .data = &emac_rk3036_emac_data,
+       },
+       {
+               .compatible = "rockchip,rk3066-emac",
+               .data = &emac_rk3066_emac_data,
+       },
+       {
+               .compatible = "rockchip,rk3188-emac",
+               .data = &emac_rk3188_emac_data,
+       },
        { /* Sentinel */ }
 };
 
@@ -123,9 +132,11 @@ static int emac_rockchip_probe(struct platform_device *pdev)
                goto out_netdev;
        }
 
-       priv->grf = syscon_regmap_lookup_by_phandle(dev->of_node, "rockchip,grf");
+       priv->grf = syscon_regmap_lookup_by_phandle(dev->of_node,
+                                                   "rockchip,grf");
        if (IS_ERR(priv->grf)) {
-               dev_err(dev, "failed to retrieve global register file (%ld)\n", PTR_ERR(priv->grf));
+               dev_err(dev, "failed to retrieve global register file (%ld)\n",
+                       PTR_ERR(priv->grf));
                err = PTR_ERR(priv->grf);
                goto out_netdev;
        }
@@ -135,14 +146,16 @@ static int emac_rockchip_probe(struct platform_device *pdev)
 
        priv->emac.clk = devm_clk_get(dev, "hclk");
        if (IS_ERR(priv->emac.clk)) {
-               dev_err(dev, "failed to retrieve host clock (%ld)\n", PTR_ERR(priv->emac.clk));
+               dev_err(dev, "failed to retrieve host clock (%ld)\n",
+                       PTR_ERR(priv->emac.clk));
                err = PTR_ERR(priv->emac.clk);
                goto out_netdev;
        }
 
        priv->refclk = devm_clk_get(dev, "macref");
        if (IS_ERR(priv->refclk)) {
-               dev_err(dev, "failed to retrieve reference clock (%ld)\n", PTR_ERR(priv->refclk));
+               dev_err(dev, "failed to retrieve reference clock (%ld)\n",
+                       PTR_ERR(priv->refclk));
                err = PTR_ERR(priv->refclk);
                goto out_netdev;
        }
@@ -179,19 +192,22 @@ static int emac_rockchip_probe(struct platform_device *pdev)
 
        err = regmap_write(priv->grf, priv->soc_data->grf_offset, data);
        if (err) {
-               dev_err(dev, "unable to apply initial settings to grf (%d)\n", err);
+               dev_err(dev, "unable to apply initial settings to grf (%d)\n",
+                       err);
                goto out_regulator_disable;
        }
 
        /* RMII interface needs always a rate of 50MHz */
        err = clk_set_rate(priv->refclk, 50000000);
        if (err)
-               dev_err(dev, "failed to change reference clock rate (%d)\n", err);
+               dev_err(dev,
+                       "failed to change reference clock rate (%d)\n", err);
 
        if (priv->soc_data->need_div_macclk) {
                priv->macclk = devm_clk_get(dev, "macclk");
                if (IS_ERR(priv->macclk)) {
-                       dev_err(dev, "failed to retrieve mac clock (%ld)\n", PTR_ERR(priv->macclk));
+                       dev_err(dev, "failed to retrieve mac clock (%ld)\n",
+                               PTR_ERR(priv->macclk));
                        err = PTR_ERR(priv->macclk);
                        goto out_regulator_disable;
                }
@@ -205,7 +221,8 @@ static int emac_rockchip_probe(struct platform_device *pdev)
                /* RMII TX/RX needs always a rate of 25MHz */
                err = clk_set_rate(priv->macclk, 25000000);
                if (err)
-                       dev_err(dev, "failed to change mac clock rate (%d)\n", err);
+                       dev_err(dev,
+                               "failed to change mac clock rate (%d)\n", err);
        }
 
        err = arc_emac_probe(ndev, interface);