net: stmmac: Delete dead code for MDIO registration
authorRomain Perier <romain.perier@collabora.com>
Mon, 4 Sep 2017 08:41:36 +0000 (10:41 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 5 Sep 2017 21:36:18 +0000 (14:36 -0700)
This code is no longer used, the logging function was changed by commit
fbca164776e4 ("net: stmmac: Use the right logging function in stmmac_mdio_register").
It was previously showing information about the type of the IRQ, if it's
polled, ignored or a normal interrupt. As we don't want information loss,
I have moved this code to phy_attached_print().

Fixes: fbca164776e4 ("net: stmmac: Use the right logging function in stmmac_mdio_register")
Signed-off-by: Romain Perier <romain.perier@collabora.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c
drivers/net/phy/phy_device.c

index 72ec711..f5f37bf 100644 (file)
@@ -248,9 +248,6 @@ int stmmac_mdio_register(struct net_device *ndev)
        found = 0;
        for (addr = 0; addr < PHY_MAX_ADDR; addr++) {
                struct phy_device *phydev = mdiobus_get_phy(new_bus, addr);
-               int act = 0;
-               char irq_num[4];
-               char *irq_str;
 
                if (!phydev)
                        continue;
@@ -273,19 +270,6 @@ int stmmac_mdio_register(struct net_device *ndev)
                if (priv->plat->phy_addr == -1)
                        priv->plat->phy_addr = addr;
 
-               act = (priv->plat->phy_addr == addr);
-               switch (phydev->irq) {
-               case PHY_POLL:
-                       irq_str = "POLL";
-                       break;
-               case PHY_IGNORE_INTERRUPT:
-                       irq_str = "IGNORE";
-                       break;
-               default:
-                       sprintf(irq_num, "%d", phydev->irq);
-                       irq_str = irq_num;
-                       break;
-               }
                phy_attached_info(phydev);
                found = 1;
        }
index 810f6fd..8cf0c59 100644 (file)
@@ -874,21 +874,37 @@ void phy_attached_info(struct phy_device *phydev)
 }
 EXPORT_SYMBOL(phy_attached_info);
 
-#define ATTACHED_FMT "attached PHY driver [%s] (mii_bus:phy_addr=%s, irq=%d)"
+#define ATTACHED_FMT "attached PHY driver [%s] (mii_bus:phy_addr=%s, irq=%s)"
 void phy_attached_print(struct phy_device *phydev, const char *fmt, ...)
 {
        const char *drv_name = phydev->drv ? phydev->drv->name : "unbound";
+       char *irq_str;
+       char irq_num[4];
+
+       switch(phydev->irq) {
+       case PHY_POLL:
+               irq_str = "POLL";
+               break;
+       case PHY_IGNORE_INTERRUPT:
+               irq_str = "IGNORE";
+               break;
+       default:
+               snprintf(irq_num, sizeof(irq_num), "%d", phydev->irq);
+               irq_str = irq_num;
+               break;
+       }
+
 
        if (!fmt) {
                dev_info(&phydev->mdio.dev, ATTACHED_FMT "\n",
                         drv_name, phydev_name(phydev),
-                        phydev->irq);
+                        irq_str);
        } else {
                va_list ap;
 
                dev_info(&phydev->mdio.dev, ATTACHED_FMT,
                         drv_name, phydev_name(phydev),
-                        phydev->irq);
+                        irq_str);
 
                va_start(ap, fmt);
                vprintk(fmt, ap);