net: ethernet: ixp4xx: Retire ancient phy retrieveal
authorLinus Walleij <linus.walleij@linaro.org>
Sun, 25 Apr 2021 00:30:37 +0000 (02:30 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Apr 2021 01:27:13 +0000 (18:27 -0700)
This driver was using a really dated way of obtaining the
phy by printing a string and using it with phy_connect().
Switch to using more reasonable modern interfaces.

Suggested-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/xscale/ixp4xx_eth.c

index 0152f1e70783418d78ef31070d84befcf89a29fd..9d323e8595e2b86c013a0cb6a0372844ee54d2a9 100644 (file)
@@ -1360,7 +1360,6 @@ static const struct net_device_ops ixp4xx_netdev_ops = {
 
 static int ixp4xx_eth_probe(struct platform_device *pdev)
 {
-       char phy_id[MII_BUS_ID_SIZE + 3];
        struct phy_device *phydev = NULL;
        struct device *dev = &pdev->dev;
        struct eth_plat_info *plat;
@@ -1459,14 +1458,15 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
        __raw_writel(DEFAULT_CORE_CNTRL, &port->regs->core_control);
        udelay(50);
 
-       snprintf(phy_id, MII_BUS_ID_SIZE + 3, PHY_ID_FMT,
-               mdio_bus->id, plat->phy);
-       phydev = phy_connect(ndev, phy_id, &ixp4xx_adjust_link,
-                            PHY_INTERFACE_MODE_MII);
+       phydev = mdiobus_get_phy(mdio_bus, plat->phy);
        if (IS_ERR(phydev)) {
                err = PTR_ERR(phydev);
                goto err_free_mem;
        }
+       err = phy_connect_direct(ndev, phydev, ixp4xx_adjust_link,
+                                PHY_INTERFACE_MODE_MII);
+       if (err)
+               goto err_free_mem;
 
        phydev->irq = PHY_POLL;