]> Pileus Git - ~andy/linux/commitdiff
ethoc: Cleanup driver format
authorBarry Grussling <barry@grussling.com>
Sun, 27 Jan 2013 18:44:36 +0000 (18:44 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Jan 2013 19:07:05 +0000 (14:07 -0500)
Cleanup the format of ethoc.c to meet network driver style as
per checkpatch.pl.

Signed-off-by: Barry Grussling <barry@grussling.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ethoc.c

index aa47ef9689a8fdf643d80f572acb3fdd23be364a..5722bc61fa582d0796a4ce37b2e8912c4840cc59 100644 (file)
@@ -206,7 +206,7 @@ struct ethoc {
        unsigned int num_rx;
        unsigned int cur_rx;
 
-       void** vma;
+       void **vma;
 
        struct net_device *netdev;
        struct napi_struct napi;
@@ -292,7 +292,7 @@ static int ethoc_init_ring(struct ethoc *dev, unsigned long mem_start)
 {
        struct ethoc_bd bd;
        int i;
-       voidvma;
+       void *vma;
 
        dev->cur_tx = 0;
        dev->dty_tx = 0;
@@ -447,8 +447,8 @@ static int ethoc_rx(struct net_device *dev, int limit)
                                netif_receive_skb(skb);
                        } else {
                                if (net_ratelimit())
-                                       dev_warn(&dev->dev, "low on memory - "
-                                                       "packet dropped\n");
+                                       dev_warn(&dev->dev,
+                                           "low on memory - packet dropped\n");
 
                                dev->stats.rx_dropped++;
                                break;
@@ -555,9 +555,8 @@ static irqreturn_t ethoc_interrupt(int irq, void *dev_id)
        pending = ethoc_read(priv, INT_SOURCE);
        pending &= mask;
 
-       if (unlikely(pending == 0)) {
+       if (unlikely(pending == 0))
                return IRQ_NONE;
-       }
 
        ethoc_ack_irq(priv, pending);
 
@@ -620,7 +619,7 @@ static int ethoc_mdio_read(struct mii_bus *bus, int phy, int reg)
        ethoc_write(priv, MIIADDRESS, MIIADDRESS_ADDR(phy, reg));
        ethoc_write(priv, MIICOMMAND, MIICOMMAND_READ);
 
-       for (i=0; i < 5; i++) {
+       for (i = 0; i < 5; i++) {
                u32 status = ethoc_read(priv, MIISTATUS);
                if (!(status & MIISTATUS_BUSY)) {
                        u32 data = ethoc_read(priv, MIIRX_DATA);
@@ -628,7 +627,7 @@ static int ethoc_mdio_read(struct mii_bus *bus, int phy, int reg)
                        ethoc_write(priv, MIICOMMAND, 0);
                        return data;
                }
-               usleep_range(100,200);
+               usleep_range(100, 200);
        }
 
        return -EBUSY;
@@ -643,14 +642,14 @@ static int ethoc_mdio_write(struct mii_bus *bus, int phy, int reg, u16 val)
        ethoc_write(priv, MIITX_DATA, val);
        ethoc_write(priv, MIICOMMAND, MIICOMMAND_WRITE);
 
-       for (i=0; i < 5; i++) {
+       for (i = 0; i < 5; i++) {
                u32 stat = ethoc_read(priv, MIISTATUS);
                if (!(stat & MIISTATUS_BUSY)) {
                        /* reset MII command register */
                        ethoc_write(priv, MIICOMMAND, 0);
                        return 0;
                }
-               usleep_range(100,200);
+               usleep_range(100, 200);
        }
 
        return -EBUSY;
@@ -671,11 +670,10 @@ static int ethoc_mdio_probe(struct net_device *dev)
        struct phy_device *phy;
        int err;
 
-       if (priv->phy_id != -1) {
+       if (priv->phy_id != -1)
                phy = priv->mdio->phy_map[priv->phy_id];
-       } else {
+       else
                phy = phy_find_first(priv->mdio);
-       }
 
        if (!phy) {
                dev_err(&dev->dev, "no PHY found\n");
@@ -1025,7 +1023,7 @@ static int ethoc_probe(struct platform_device *pdev)
        dev_dbg(&pdev->dev, "ethoc: num_tx: %d num_rx: %d\n",
                priv->num_tx, priv->num_rx);
 
-       priv->vma = devm_kzalloc(&pdev->dev, num_bd*sizeof(void*), GFP_KERNEL);
+       priv->vma = devm_kzalloc(&pdev->dev, num_bd*sizeof(void *), GFP_KERNEL);
        if (!priv->vma) {
                ret = -ENOMEM;
                goto error;
@@ -1041,7 +1039,7 @@ static int ethoc_probe(struct platform_device *pdev)
 
 #ifdef CONFIG_OF
                {
-               const uint8_tmac;
+               const uint8_t *mac;
 
                mac = of_get_property(pdev->dev.of_node,
                                      "local-mac-address",
@@ -1053,12 +1051,14 @@ static int ethoc_probe(struct platform_device *pdev)
        }
 
        /* Check that the given MAC address is valid. If it isn't, read the
-        * current MAC from the controller. */
+        * current MAC from the controller.
+        */
        if (!is_valid_ether_addr(netdev->dev_addr))
                ethoc_get_mac_address(netdev, netdev->dev_addr);
 
        /* Check the MAC again for validity, if it still isn't choose and
-        * program a random one. */
+        * program a random one.
+        */
        if (!is_valid_ether_addr(netdev->dev_addr)) {
                eth_random_addr(netdev->dev_addr);
                random_mac = true;