net: phy: fix checkpatch errors
authorFlorian Fainelli <f.fainelli@gmail.com>
Wed, 18 Dec 2013 05:38:12 +0000 (21:38 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Dec 2013 05:49:02 +0000 (00:49 -0500)
checkpatch spotted a few checkpatch errors such as whitespace damages
and switch/case labels not being on the same column, fix them.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/phy.c
drivers/net/phy/phy_device.c

index dea609f86aee4063af265cfa7c899c6c69917e74..20908865709fb186814d32fbfbb247387cfb6e79 100644 (file)
@@ -332,7 +332,7 @@ int phy_mii_ioctl(struct phy_device *phydev,
 
        case SIOCSMIIREG:
                if (mii_data->phy_id == phydev->addr) {
-                       switch(mii_data->reg_num) {
+                       switch (mii_data->reg_num) {
                        case MII_BMCR:
                                if ((val & (BMCR_RESET|BMCR_ANENABLE)) == 0)
                                        phydev->autoneg = AUTONEG_DISABLE;
@@ -714,16 +714,16 @@ void phy_start(struct phy_device *phydev)
        mutex_lock(&phydev->lock);
 
        switch (phydev->state) {
-               case PHY_STARTING:
-                       phydev->state = PHY_PENDING;
-                       break;
-               case PHY_READY:
-                       phydev->state = PHY_UP;
-                       break;
-               case PHY_HALTED:
-                       phydev->state = PHY_RESUMING;
-               default:
-                       break;
+       case PHY_STARTING:
+               phydev->state = PHY_PENDING;
+               break;
+       case PHY_READY:
+               phydev->state = PHY_UP;
+               break;
+       case PHY_HALTED:
+               phydev->state = PHY_RESUMING;
+       default:
+               break;
        }
        mutex_unlock(&phydev->lock);
 }
@@ -747,153 +747,138 @@ void phy_state_machine(struct work_struct *work)
        if (phydev->adjust_state)
                phydev->adjust_state(phydev->attached_dev);
 
-       switch(phydev->state) {
-               case PHY_DOWN:
-               case PHY_STARTING:
-               case PHY_READY:
-               case PHY_PENDING:
-                       break;
-               case PHY_UP:
-                       needs_aneg = 1;
+       switch (phydev->state) {
+       case PHY_DOWN:
+       case PHY_STARTING:
+       case PHY_READY:
+       case PHY_PENDING:
+               break;
+       case PHY_UP:
+               needs_aneg = 1;
 
-                       phydev->link_timeout = PHY_AN_TIMEOUT;
+               phydev->link_timeout = PHY_AN_TIMEOUT;
+
+               break;
+       case PHY_AN:
+               err = phy_read_status(phydev);
 
+               if (err < 0)
                        break;
-               case PHY_AN:
-                       err = phy_read_status(phydev);
 
-                       if (err < 0)
-                               break;
+               /* If the link is down, give up on
+                * negotiation for now */
+               if (!phydev->link) {
+                       phydev->state = PHY_NOLINK;
+                       netif_carrier_off(phydev->attached_dev);
+                       phydev->adjust_link(phydev->attached_dev);
+                       break;
+               }
 
-                       /* If the link is down, give up on
-                        * negotiation for now */
-                       if (!phydev->link) {
-                               phydev->state = PHY_NOLINK;
-                               netif_carrier_off(phydev->attached_dev);
-                               phydev->adjust_link(phydev->attached_dev);
-                               break;
-                       }
+               /* Check if negotiation is done.  Break
+                * if there's an error */
+               err = phy_aneg_done(phydev);
+               if (err < 0)
+                       break;
 
-                       /* Check if negotiation is done.  Break
-                        * if there's an error */
-                       err = phy_aneg_done(phydev);
-                       if (err < 0)
-                               break;
+               /* If AN is done, we're running */
+               if (err > 0) {
+                       phydev->state = PHY_RUNNING;
+                       netif_carrier_on(phydev->attached_dev);
+                       phydev->adjust_link(phydev->attached_dev);
 
-                       /* If AN is done, we're running */
-                       if (err > 0) {
-                               phydev->state = PHY_RUNNING;
-                               netif_carrier_on(phydev->attached_dev);
-                               phydev->adjust_link(phydev->attached_dev);
+               } else if (0 == phydev->link_timeout--) {
+                       needs_aneg = 1;
+                       /* If we have the magic_aneg bit,
+                        * we try again */
+                       if (phydev->drv->flags & PHY_HAS_MAGICANEG)
+                               break;
+               }
+               break;
+       case PHY_NOLINK:
+               err = phy_read_status(phydev);
 
-                       } else if (0 == phydev->link_timeout--) {
-                               needs_aneg = 1;
-                               /* If we have the magic_aneg bit,
-                                * we try again */
-                               if (phydev->drv->flags & PHY_HAS_MAGICANEG)
-                                       break;
-                       }
+               if (err)
                        break;
-               case PHY_NOLINK:
-                       err = phy_read_status(phydev);
 
-                       if (err)
-                               break;
+               if (phydev->link) {
+                       phydev->state = PHY_RUNNING;
+                       netif_carrier_on(phydev->attached_dev);
+                       phydev->adjust_link(phydev->attached_dev);
+               }
+               break;
+       case PHY_FORCING:
+               err = genphy_update_link(phydev);
 
-                       if (phydev->link) {
-                               phydev->state = PHY_RUNNING;
-                               netif_carrier_on(phydev->attached_dev);
-                               phydev->adjust_link(phydev->attached_dev);
-                       }
+               if (err)
                        break;
-               case PHY_FORCING:
-                       err = genphy_update_link(phydev);
 
-                       if (err)
-                               break;
+               if (phydev->link) {
+                       phydev->state = PHY_RUNNING;
+                       netif_carrier_on(phydev->attached_dev);
+               } else {
+                       if (0 == phydev->link_timeout--)
+                               needs_aneg = 1;
+               }
 
-                       if (phydev->link) {
-                               phydev->state = PHY_RUNNING;
-                               netif_carrier_on(phydev->attached_dev);
-                       } else {
-                               if (0 == phydev->link_timeout--)
-                                       needs_aneg = 1;
-                       }
+               phydev->adjust_link(phydev->attached_dev);
+               break;
+       case PHY_RUNNING:
+               /* Only register a CHANGE if we are
+                * polling or ignoring interrupts
+                */
+               if (!phy_interrupt_is_valid(phydev))
+                       phydev->state = PHY_CHANGELINK;
+               break;
+       case PHY_CHANGELINK:
+               err = phy_read_status(phydev);
 
-                       phydev->adjust_link(phydev->attached_dev);
+               if (err)
                        break;
-               case PHY_RUNNING:
-                       /* Only register a CHANGE if we are
-                        * polling or ignoring interrupts
-                        */
-                       if (!phy_interrupt_is_valid(phydev))
-                               phydev->state = PHY_CHANGELINK;
-                       break;
-               case PHY_CHANGELINK:
-                       err = phy_read_status(phydev);
 
-                       if (err)
-                               break;
+               if (phydev->link) {
+                       phydev->state = PHY_RUNNING;
+                       netif_carrier_on(phydev->attached_dev);
+               } else {
+                       phydev->state = PHY_NOLINK;
+                       netif_carrier_off(phydev->attached_dev);
+               }
 
-                       if (phydev->link) {
-                               phydev->state = PHY_RUNNING;
-                               netif_carrier_on(phydev->attached_dev);
-                       } else {
-                               phydev->state = PHY_NOLINK;
-                               netif_carrier_off(phydev->attached_dev);
-                       }
+               phydev->adjust_link(phydev->attached_dev);
 
+               if (phy_interrupt_is_valid(phydev))
+                       err = phy_config_interrupt(phydev,
+                                       PHY_INTERRUPT_ENABLED);
+               break;
+       case PHY_HALTED:
+               if (phydev->link) {
+                       phydev->link = 0;
+                       netif_carrier_off(phydev->attached_dev);
                        phydev->adjust_link(phydev->attached_dev);
+                       do_suspend = 1;
+               }
+               break;
+       case PHY_RESUMING:
 
-                       if (phy_interrupt_is_valid(phydev))
-                               err = phy_config_interrupt(phydev,
-                                               PHY_INTERRUPT_ENABLED);
-                       break;
-               case PHY_HALTED:
-                       if (phydev->link) {
-                               phydev->link = 0;
-                               netif_carrier_off(phydev->attached_dev);
-                               phydev->adjust_link(phydev->attached_dev);
-                               do_suspend = 1;
-                       }
-                       break;
-               case PHY_RESUMING:
+               err = phy_clear_interrupt(phydev);
 
-                       err = phy_clear_interrupt(phydev);
+               if (err)
+                       break;
 
-                       if (err)
-                               break;
+               err = phy_config_interrupt(phydev,
+                               PHY_INTERRUPT_ENABLED);
 
-                       err = phy_config_interrupt(phydev,
-                                       PHY_INTERRUPT_ENABLED);
+               if (err)
+                       break;
 
-                       if (err)
+               if (AUTONEG_ENABLE == phydev->autoneg) {
+                       err = phy_aneg_done(phydev);
+                       if (err < 0)
                                break;
 
-                       if (AUTONEG_ENABLE == phydev->autoneg) {
-                               err = phy_aneg_done(phydev);
-                               if (err < 0)
-                                       break;
-
-                               /* err > 0 if AN is done.
-                                * Otherwise, it's 0, and we're
-                                * still waiting for AN */
-                               if (err > 0) {
-                                       err = phy_read_status(phydev);
-                                       if (err)
-                                               break;
-
-                                       if (phydev->link) {
-                                               phydev->state = PHY_RUNNING;
-                                               netif_carrier_on(phydev->attached_dev);
-                                       } else
-                                               phydev->state = PHY_NOLINK;
-                                       phydev->adjust_link(phydev->attached_dev);
-                               } else {
-                                       phydev->state = PHY_AN;
-                                       phydev->link_timeout = PHY_AN_TIMEOUT;
-                               }
-                       } else {
+                       /* err > 0 if AN is done.
+                        * Otherwise, it's 0, and we're
+                        * still waiting for AN */
+                       if (err > 0) {
                                err = phy_read_status(phydev);
                                if (err)
                                        break;
@@ -904,8 +889,23 @@ void phy_state_machine(struct work_struct *work)
                                } else
                                        phydev->state = PHY_NOLINK;
                                phydev->adjust_link(phydev->attached_dev);
+                       } else {
+                               phydev->state = PHY_AN;
+                               phydev->link_timeout = PHY_AN_TIMEOUT;
                        }
-                       break;
+               } else {
+                       err = phy_read_status(phydev);
+                       if (err)
+                               break;
+
+                       if (phydev->link) {
+                               phydev->state = PHY_RUNNING;
+                               netif_carrier_on(phydev->attached_dev);
+                       } else
+                               phydev->state = PHY_NOLINK;
+                       phydev->adjust_link(phydev->attached_dev);
+               }
+               break;
        }
 
        mutex_unlock(&phydev->lock);
index 4eb5bba1db5e012f096c4753678267f639cb2495..d53242449c90d256a6f2663bd8672f899fbe3007 100644 (file)
@@ -162,7 +162,7 @@ struct phy_device *phy_device_create(struct mii_bus *bus, int addr, int phy_id,
        dev = kzalloc(sizeof(*dev), GFP_KERNEL);
 
        if (NULL == dev)
-               return (struct phy_device*) PTR_ERR((void*)-ENOMEM);
+               return (struct phy_device *)PTR_ERR((void *)-ENOMEM);
 
        dev->dev.release = phy_device_release;
 
@@ -459,7 +459,7 @@ EXPORT_SYMBOL(phy_connect_direct);
  *   choose to call only the subset of functions which provide
  *   the desired functionality.
  */
-struct phy_device * phy_connect(struct net_device *dev, const char *bus_id,
+struct phy_device *phy_connect(struct net_device *dev, const char *bus_id,
                void (*handler)(struct net_device *),
                phy_interface_t interface)
 {
@@ -494,7 +494,7 @@ void phy_disconnect(struct phy_device *phydev)
                phy_stop_interrupts(phydev);
 
        phy_stop_machine(phydev);
-       
+
        phydev->adjust_link = NULL;
 
        phy_detach(phydev);
@@ -792,7 +792,7 @@ int genphy_setup_forced(struct phy_device *phydev)
 
        if (DUPLEX_FULL == phydev->duplex)
                ctl |= BMCR_FULLDPLX;
-       
+
        err = phy_write(phydev, MII_BMCR, ctl);
 
        return err;
@@ -965,14 +965,14 @@ int genphy_read_status(struct phy_device *phydev)
                                phydev->duplex = DUPLEX_FULL;
                } else if (lpa & (LPA_100FULL | LPA_100HALF)) {
                        phydev->speed = SPEED_100;
-                       
+
                        if (lpa & LPA_100FULL)
                                phydev->duplex = DUPLEX_FULL;
                } else
                        if (lpa & LPA_10FULL)
                                phydev->duplex = DUPLEX_FULL;
 
-               if (phydev->duplex == DUPLEX_FULL){
+               if (phydev->duplex == DUPLEX_FULL) {
                        phydev->pause = lpa & LPA_PAUSE_CAP ? 1 : 0;
                        phydev->asym_pause = lpa & LPA_PAUSE_ASYM ? 1 : 0;
                }
@@ -1213,7 +1213,7 @@ static struct phy_driver genphy_driver = {
        .read_status    = genphy_read_status,
        .suspend        = genphy_suspend,
        .resume         = genphy_resume,
-       .driver         = {.owner= THIS_MODULE, },
+       .driver         = { .owner = THIS_MODULE, },
 };
 
 static int __init phy_init(void)