net: phy: split out PHY speed and duplex string generation
authorRussell King <rmk+kernel@armlinux.org.uk>
Tue, 25 Jul 2017 14:02:42 +0000 (15:02 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Aug 2017 03:55:28 +0000 (20:55 -0700)
Other code would like to make use of this, so make the speed and duplex
string generation visible, and place it in a separate file.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/phy-core.c
drivers/net/phy/phy.c
include/linux/phy.h

index 6739b73..bf01a24 100644 (file)
@@ -9,6 +9,55 @@
 #include <linux/export.h>
 #include <linux/phy.h>
 
+const char *phy_speed_to_str(int speed)
+{
+       switch (speed) {
+       case SPEED_10:
+               return "10Mbps";
+       case SPEED_100:
+               return "100Mbps";
+       case SPEED_1000:
+               return "1Gbps";
+       case SPEED_2500:
+               return "2.5Gbps";
+       case SPEED_5000:
+               return "5Gbps";
+       case SPEED_10000:
+               return "10Gbps";
+       case SPEED_14000:
+               return "14Gbps";
+       case SPEED_20000:
+               return "20Gbps";
+       case SPEED_25000:
+               return "25Gbps";
+       case SPEED_40000:
+               return "40Gbps";
+       case SPEED_50000:
+               return "50Gbps";
+       case SPEED_56000:
+               return "56Gbps";
+       case SPEED_100000:
+               return "100Gbps";
+       case SPEED_UNKNOWN:
+               return "Unknown";
+       default:
+               return "Unsupported (update phy-core.c)";
+       }
+}
+EXPORT_SYMBOL_GPL(phy_speed_to_str);
+
+const char *phy_duplex_to_str(unsigned int duplex)
+{
+       if (duplex == DUPLEX_HALF)
+               return "Half";
+       if (duplex == DUPLEX_FULL)
+               return "Full";
+       if (duplex == DUPLEX_UNKNOWN)
+               return "Unknown";
+       return "Unsupported (update phy-core.c)";
+}
+EXPORT_SYMBOL_GPL(phy_duplex_to_str);
+
 static void mmd_phy_indirect(struct mii_bus *bus, int phy_addr, int devad,
                             u16 regnum)
 {
index abae916..7b7fe6b 100644 (file)
 
 #include <asm/irq.h>
 
-static const char *phy_speed_to_str(int speed)
-{
-       switch (speed) {
-       case SPEED_10:
-               return "10Mbps";
-       case SPEED_100:
-               return "100Mbps";
-       case SPEED_1000:
-               return "1Gbps";
-       case SPEED_2500:
-               return "2.5Gbps";
-       case SPEED_5000:
-               return "5Gbps";
-       case SPEED_10000:
-               return "10Gbps";
-       case SPEED_14000:
-               return "14Gbps";
-       case SPEED_20000:
-               return "20Gbps";
-       case SPEED_25000:
-               return "25Gbps";
-       case SPEED_40000:
-               return "40Gbps";
-       case SPEED_50000:
-               return "50Gbps";
-       case SPEED_56000:
-               return "56Gbps";
-       case SPEED_100000:
-               return "100Gbps";
-       case SPEED_UNKNOWN:
-               return "Unknown";
-       default:
-               return "Unsupported (update phy.c)";
-       }
-}
-
 #define PHY_STATE_STR(_state)                  \
        case PHY_##_state:                      \
                return __stringify(_state);     \
@@ -109,7 +73,7 @@ void phy_print_status(struct phy_device *phydev)
                netdev_info(phydev->attached_dev,
                        "Link is Up - %s/%s - flow control %s\n",
                        phy_speed_to_str(phydev->speed),
-                       DUPLEX_FULL == phydev->duplex ? "Full" : "Half",
+                       phy_duplex_to_str(phydev->duplex),
                        phydev->pause ? "rx/tx" : "off");
        } else  {
                netdev_info(phydev->attached_dev, "Link is Down\n");
index 0bb5b21..e8264c7 100644 (file)
@@ -667,6 +667,9 @@ struct phy_fixup {
        int (*run)(struct phy_device *phydev);
 };
 
+const char *phy_speed_to_str(int speed);
+const char *phy_duplex_to_str(unsigned int duplex);
+
 /**
  * phy_read_mmd - Convenience function for reading a register
  * from an MMD on a given PHY.