net: ethernet: faraday: Use phy_find_first() instead of open coding it
authorGuenter Roeck <linux@roeck-us.net>
Sun, 10 Jan 2016 20:04:32 +0000 (12:04 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 11 Jan 2016 03:05:30 +0000 (22:05 -0500)
Use phy_find_first() to find the first phy device instead of
open coding it.

Cc: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/faraday/ftgmac100.c

index bb116ad..84384e1 100644 (file)
@@ -834,20 +834,9 @@ static void ftgmac100_adjust_link(struct net_device *netdev)
 static int ftgmac100_mii_probe(struct ftgmac100 *priv)
 {
        struct net_device *netdev = priv->netdev;
-       struct phy_device *phydev = NULL;
-       int i;
-
-       /* search for connect PHY device */
-       for (i = 0; i < PHY_MAX_ADDR; i++) {
-               struct phy_device *tmp = mdiobus_get_phy(priv->mii_bus, i);
-
-               if (tmp) {
-                       phydev = tmp;
-                       break;
-               }
-       }
+       struct phy_device *phydev;
 
-       /* now we are supposed to have a proper phydev, to attach to... */
+       phydev = phy_find_first(priv->mii_bus);
        if (!phydev) {
                netdev_info(netdev, "%s: no PHY found\n", netdev->name);
                return -ENODEV;