net: bcmgenet: remove PHY_BRCM_100MBPS_WAR
authorFlorian Fainelli <f.fainelli@gmail.com>
Fri, 19 Sep 2014 20:07:52 +0000 (13:07 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 19 Sep 2014 20:27:07 +0000 (16:27 -0400)
Now that we have removed the need for the PHY_BRCM_100MBPS_WAR flag, we
can remove it from the GENET driver and the broadcom shared header file.
The PHY driver checks the PHY supported bitmask instead.

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
include/linux/brcmphy.h

index c88f7ae9963698b97837dc094f96d8b163c512c7..71f2ef7d33a9e9af9ff388cd2f182a6622d462cb 100644 (file)
@@ -296,7 +296,6 @@ static int bcmgenet_mii_probe(struct net_device *dev)
        struct bcmgenet_priv *priv = netdev_priv(dev);
        struct device_node *dn = priv->pdev->dev.of_node;
        struct phy_device *phydev;
-       unsigned int phy_flags;
        int ret;
 
        if (priv->phydev) {
@@ -338,15 +337,6 @@ static int bcmgenet_mii_probe(struct net_device *dev)
                return ret;
        }
 
-       phy_flags = PHY_BRCM_100MBPS_WAR;
-
-       /* workarounds are only needed for 100Mpbs PHYs, and
-        * never on GENET V1 hardware
-        */
-       if ((phydev->supported & PHY_GBIT_FEATURES) || GENET_IS_V1(priv))
-               phy_flags = 0;
-
-       phydev->dev_flags |= phy_flags;
        phydev->advertising = phydev->supported;
 
        /* The internal PHY has its link interrupts routed to the
index e4ee9c6679e8dc67a3684e34e190dba7fadd71a2..3f626fe48c9a3eb2b7434f4da8a4086be86cad62 100644 (file)
@@ -40,7 +40,6 @@
 #define PHY_BRCM_CLEAR_RGMII_MODE      0x00004000
 #define PHY_BRCM_DIS_TXCRXC_NOENRGY    0x00008000
 /* Broadcom BCM7xxx specific workarounds */
-#define PHY_BRCM_100MBPS_WAR           0x00010000
 #define PHY_BRCM_7XXX_REV(x)           (((x) >> 8) & 0xff)
 #define PHY_BRCM_7XXX_PATCH(x)         ((x) & 0xff)
 #define PHY_BCM_FLAGS_VALID            0x80000000