net: bcmgenet: Remove special handling of "internal" phy-mode
authorFlorian Fainelli <f.fainelli@gmail.com>
Fri, 23 Jun 2017 17:33:15 +0000 (10:33 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 23 Jun 2017 19:06:43 +0000 (15:06 -0400)
The PHY library now supports an "internal" phy-mode, thus making our
custom parsing code now unnecessary.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/genet/bcmmii.c

index 285676f8da6bf3b9b0bb42e8a5407be56e478796..071fcbd14e6a4e17534950107d5081662eba5c3c 100644 (file)
@@ -251,11 +251,8 @@ int bcmgenet_mii_config(struct net_device *dev)
        priv->ext_phy = !priv->internal_phy &&
                        (priv->phy_interface != PHY_INTERFACE_MODE_MOCA);
 
-       if (priv->internal_phy)
-               priv->phy_interface = PHY_INTERFACE_MODE_NA;
-
        switch (priv->phy_interface) {
-       case PHY_INTERFACE_MODE_NA:
+       case PHY_INTERFACE_MODE_INTERNAL:
        case PHY_INTERFACE_MODE_MOCA:
                /* Irrespective of the actually configured PHY speed (100 or
                 * 1000) GENETv4 only has an internal GPHY so we will just end
@@ -471,7 +468,6 @@ static int bcmgenet_mii_of_init(struct bcmgenet_priv *priv)
 {
        struct device_node *dn = priv->pdev->dev.of_node;
        struct device *kdev = &priv->pdev->dev;
-       const char *phy_mode_str = NULL;
        struct phy_device *phydev = NULL;
        char *compat;
        int phy_mode;
@@ -510,23 +506,19 @@ static int bcmgenet_mii_of_init(struct bcmgenet_priv *priv)
 
        /* Get the link mode */
        phy_mode = of_get_phy_mode(dn);
+       if (phy_mode < 0) {
+               dev_err(kdev, "invalid PHY mode property\n");
+               return phy_mode;
+       }
+
        priv->phy_interface = phy_mode;
 
        /* We need to specifically look up whether this PHY interface is internal
         * or not *before* we even try to probe the PHY driver over MDIO as we
         * may have shut down the internal PHY for power saving purposes.
         */
-       if (phy_mode < 0) {
-               ret = of_property_read_string(dn, "phy-mode", &phy_mode_str);
-               if (ret < 0) {
-                       dev_err(kdev, "invalid PHY mode property\n");
-                       return ret;
-               }
-
-               priv->phy_interface = PHY_INTERFACE_MODE_NA;
-               if (!strcasecmp(phy_mode_str, "internal"))
-                       priv->internal_phy = true;
-       }
+       if (priv->phy_interface == PHY_INTERFACE_MODE_INTERNAL)
+               priv->internal_phy = true;
 
        /* Make sure we initialize MoCA PHYs with a link down */
        if (phy_mode == PHY_INTERFACE_MODE_MOCA) {