net: phy: broadcom: Add support for BCM54612E
authorXo Wang <xow@google.com>
Fri, 21 Oct 2016 17:20:13 +0000 (10:20 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 26 Oct 2016 21:15:26 +0000 (17:15 -0400)
This PHY has internal delays enabled after reset. This clears the
internal delay enables unless the interface specifically requests them.

Signed-off-by: Xo Wang <xow@google.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/broadcom.c
include/linux/brcmphy.h

index 870327efccf78ef0db0361212caf0aa899974625..583ef8a2ec8d3af2179bc3d83de7ac0dc38b481e 100644 (file)
@@ -337,6 +337,41 @@ static int bcm5481_config_aneg(struct phy_device *phydev)
        return ret;
 }
 
+static int bcm54612e_config_aneg(struct phy_device *phydev)
+{
+       int ret;
+
+       /* First, auto-negotiate. */
+       ret = genphy_config_aneg(phydev);
+
+       /* Clear TX internal delay unless requested. */
+       if ((phydev->interface != PHY_INTERFACE_MODE_RGMII_ID) &&
+           (phydev->interface != PHY_INTERFACE_MODE_RGMII_TXID)) {
+               /* Disable TXD to GTXCLK clock delay (default set) */
+               /* Bit 9 is the only field in shadow register 00011 */
+               bcm_phy_write_shadow(phydev, 0x03, 0);
+       }
+
+       /* Clear RX internal delay unless requested. */
+       if ((phydev->interface != PHY_INTERFACE_MODE_RGMII_ID) &&
+           (phydev->interface != PHY_INTERFACE_MODE_RGMII_RXID)) {
+               u16 reg;
+
+               /* Errata: reads require filling in the write selector field */
+               bcm54xx_auxctl_write(phydev, MII_BCM54XX_AUXCTL_SHDWSEL_MISC,
+                                    MII_BCM54XX_AUXCTL_MISC_RDSEL_MISC);
+               reg = phy_read(phydev, MII_BCM54XX_AUX_CTL);
+               /* Disable RXD to RXC delay (default set) */
+               reg &= ~MII_BCM54XX_AUXCTL_MISC_RXD_RXC_SKEW;
+               /* Clear shadow selector field */
+               reg &= ~MII_BCM54XX_AUXCTL_SHDWSEL_MASK;
+               bcm54xx_auxctl_write(phydev, MII_BCM54XX_AUXCTL_SHDWSEL_MISC,
+                                    MII_BCM54XX_AUXCTL_MISC_WREN | reg);
+       }
+
+       return ret;
+}
+
 static int brcm_phy_setbits(struct phy_device *phydev, int reg, int set)
 {
        int val;
@@ -484,6 +519,18 @@ static struct phy_driver broadcom_drivers[] = {
        .read_status    = genphy_read_status,
        .ack_interrupt  = bcm_phy_ack_intr,
        .config_intr    = bcm_phy_config_intr,
+}, {
+       .phy_id         = PHY_ID_BCM54612E,
+       .phy_id_mask    = 0xfffffff0,
+       .name           = "Broadcom BCM54612E",
+       .features       = PHY_GBIT_FEATURES |
+                         SUPPORTED_Pause | SUPPORTED_Asym_Pause,
+       .flags          = PHY_HAS_MAGICANEG | PHY_HAS_INTERRUPT,
+       .config_init    = bcm54xx_config_init,
+       .config_aneg    = bcm54612e_config_aneg,
+       .read_status    = genphy_read_status,
+       .ack_interrupt  = bcm_phy_ack_intr,
+       .config_intr    = bcm_phy_config_intr,
 }, {
        .phy_id         = PHY_ID_BCM54616S,
        .phy_id_mask    = 0xfffffff0,
@@ -600,6 +647,7 @@ static struct mdio_device_id __maybe_unused broadcom_tbl[] = {
        { PHY_ID_BCM5411, 0xfffffff0 },
        { PHY_ID_BCM5421, 0xfffffff0 },
        { PHY_ID_BCM5461, 0xfffffff0 },
+       { PHY_ID_BCM54612E, 0xfffffff0 },
        { PHY_ID_BCM54616S, 0xfffffff0 },
        { PHY_ID_BCM5464, 0xfffffff0 },
        { PHY_ID_BCM5481, 0xfffffff0 },
index 22c4421c916cfb8e7b7211b337932b9a604705d8..60def78c4e1221ea75756fd34c07a66abf9fbc16 100644 (file)
@@ -18,6 +18,7 @@
 #define PHY_ID_BCM5421                 0x002060e0
 #define PHY_ID_BCM5464                 0x002060b0
 #define PHY_ID_BCM5461                 0x002060c0
+#define PHY_ID_BCM54612E               0x03625e60
 #define PHY_ID_BCM54616S               0x03625d10
 #define PHY_ID_BCM57780                        0x03625d90