aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRomain Perier <romain.perier@collabora.com>2017-09-04 04:41:36 -0400
committerDavid S. Miller <davem@davemloft.net>2017-09-05 17:36:18 -0400
commit5e369aefdce4818c8264b81d3436eccf35af561e (patch)
treea44117a7562f1dc30538d802ed1f5a3510b9dda5
parent5d621672bc1a1e5090c1ac5432a18c79e0e13e03 (diff)
net: stmmac: Delete dead code for MDIO registration
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>
-rw-r--r--drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c16
-rw-r--r--drivers/net/phy/phy_device.c22
2 files changed, 19 insertions, 19 deletions
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c
index 72ec711fcba2..f5f37bfa1d58 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c
@@ -248,9 +248,6 @@ int stmmac_mdio_register(struct net_device *ndev)
248 found = 0; 248 found = 0;
249 for (addr = 0; addr < PHY_MAX_ADDR; addr++) { 249 for (addr = 0; addr < PHY_MAX_ADDR; addr++) {
250 struct phy_device *phydev = mdiobus_get_phy(new_bus, addr); 250 struct phy_device *phydev = mdiobus_get_phy(new_bus, addr);
251 int act = 0;
252 char irq_num[4];
253 char *irq_str;
254 251
255 if (!phydev) 252 if (!phydev)
256 continue; 253 continue;
@@ -273,19 +270,6 @@ int stmmac_mdio_register(struct net_device *ndev)
273 if (priv->plat->phy_addr == -1) 270 if (priv->plat->phy_addr == -1)
274 priv->plat->phy_addr = addr; 271 priv->plat->phy_addr = addr;
275 272
276 act = (priv->plat->phy_addr == addr);
277 switch (phydev->irq) {
278 case PHY_POLL:
279 irq_str = "POLL";
280 break;
281 case PHY_IGNORE_INTERRUPT:
282 irq_str = "IGNORE";
283 break;
284 default:
285 sprintf(irq_num, "%d", phydev->irq);
286 irq_str = irq_num;
287 break;
288 }
289 phy_attached_info(phydev); 273 phy_attached_info(phydev);
290 found = 1; 274 found = 1;
291 } 275 }
diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c
index 810f6fd2f639..8cf0c5901f95 100644
--- a/drivers/net/phy/phy_device.c
+++ b/drivers/net/phy/phy_device.c
@@ -874,21 +874,37 @@ void phy_attached_info(struct phy_device *phydev)
874} 874}
875EXPORT_SYMBOL(phy_attached_info); 875EXPORT_SYMBOL(phy_attached_info);
876 876
877#define ATTACHED_FMT "attached PHY driver [%s] (mii_bus:phy_addr=%s, irq=%d)" 877#define ATTACHED_FMT "attached PHY driver [%s] (mii_bus:phy_addr=%s, irq=%s)"
878void phy_attached_print(struct phy_device *phydev, const char *fmt, ...) 878void phy_attached_print(struct phy_device *phydev, const char *fmt, ...)
879{ 879{
880 const char *drv_name = phydev->drv ? phydev->drv->name : "unbound"; 880 const char *drv_name = phydev->drv ? phydev->drv->name : "unbound";
881 char *irq_str;
882 char irq_num[4];
883
884 switch(phydev->irq) {
885 case PHY_POLL:
886 irq_str = "POLL";
887 break;
888 case PHY_IGNORE_INTERRUPT:
889 irq_str = "IGNORE";
890 break;
891 default:
892 snprintf(irq_num, sizeof(irq_num), "%d", phydev->irq);
893 irq_str = irq_num;
894 break;
895 }
896
881 897
882 if (!fmt) { 898 if (!fmt) {
883 dev_info(&phydev->mdio.dev, ATTACHED_FMT "\n", 899 dev_info(&phydev->mdio.dev, ATTACHED_FMT "\n",
884 drv_name, phydev_name(phydev), 900 drv_name, phydev_name(phydev),
885 phydev->irq); 901 irq_str);
886 } else { 902 } else {
887 va_list ap; 903 va_list ap;
888 904
889 dev_info(&phydev->mdio.dev, ATTACHED_FMT, 905 dev_info(&phydev->mdio.dev, ATTACHED_FMT,
890 drv_name, phydev_name(phydev), 906 drv_name, phydev_name(phydev),
891 phydev->irq); 907 irq_str);
892 908
893 va_start(ap, fmt); 909 va_start(ap, fmt);
894 vprintk(fmt, ap); 910 vprintk(fmt, ap);