phy: broadcom: split out the BCM54213PE from the BCM54210E IDs
authorJonathan Bell <jonathan@raspberrypi.org>
Tue, 14 May 2019 16:00:41 +0000 (17:00 +0100)
committerpopcornmix <popcornmix@gmail.com>
Wed, 27 Jan 2021 19:13:24 +0000 (19:13 +0000)
The last nibble is a revision ID, and the 54213pe is a later rev
than the 54210e. Running the 54210e setup code on a 54213pe results
in a broken RGMII interface.

Signed-off-by: Jonathan Bell <jonathan@raspberrypi.org>
drivers/net/phy/broadcom.c
include/linux/brcmphy.h

index 84ec5f2..e7aa799 100644 (file)
@@ -196,7 +196,8 @@ static void bcm54xx_adjust_rxrefclk(struct phy_device *phydev)
            BRCM_PHY_MODEL(phydev) != PHY_ID_BCM50610 &&
            BRCM_PHY_MODEL(phydev) != PHY_ID_BCM50610M &&
            BRCM_PHY_MODEL(phydev) != PHY_ID_BCM54810 &&
-           BRCM_PHY_MODEL(phydev) != PHY_ID_BCM54811)
+           BRCM_PHY_MODEL(phydev) != PHY_ID_BCM54811 &&
+           BRCM_PHY_MODEL(phydev) != PHY_ID_BCM54213PE)
                return;
 
        val = bcm_phy_read_shadow(phydev, BCM54XX_SHD_SCR3);
@@ -694,13 +695,21 @@ static struct phy_driver broadcom_drivers[] = {
        .config_intr    = bcm_phy_config_intr,
 }, {
        .phy_id         = PHY_ID_BCM54210E,
-       .phy_id_mask    = 0xfffffff0,
+       .phy_id_mask    = 0xffffffff,
        .name           = "Broadcom BCM54210E",
        /* PHY_GBIT_FEATURES */
        .config_init    = bcm54xx_config_init,
        .ack_interrupt  = bcm_phy_ack_intr,
        .config_intr    = bcm_phy_config_intr,
 }, {
+       .phy_id         = PHY_ID_BCM54213PE,
+       .phy_id_mask    = 0xffffffff,
+       .name           = "Broadcom BCM54213PE",
+       /* PHY_GBIT_FEATURES */
+       .config_init    = bcm54xx_config_init,
+       .ack_interrupt  = bcm_phy_ack_intr,
+       .config_intr    = bcm_phy_config_intr,
+}, {
        .phy_id         = PHY_ID_BCM5461,
        .phy_id_mask    = 0xfffffff0,
        .name           = "Broadcom BCM5461",
@@ -855,7 +864,8 @@ module_phy_driver(broadcom_drivers);
 static struct mdio_device_id __maybe_unused broadcom_tbl[] = {
        { PHY_ID_BCM5411, 0xfffffff0 },
        { PHY_ID_BCM5421, 0xfffffff0 },
-       { PHY_ID_BCM54210E, 0xfffffff0 },
+       { PHY_ID_BCM54210E, 0xffffffff },
+       { PHY_ID_BCM54213PE, 0xffffffff },
        { PHY_ID_BCM5461, 0xfffffff0 },
        { PHY_ID_BCM54612E, 0xfffffff0 },
        { PHY_ID_BCM54616S, 0xfffffff0 },
index d0bd226..055e591 100644 (file)
@@ -22,6 +22,7 @@
 #define PHY_ID_BCM5411                 0x00206070
 #define PHY_ID_BCM5421                 0x002060e0
 #define PHY_ID_BCM54210E               0x600d84a0
+#define PHY_ID_BCM54213PE              0x600d84a2
 #define PHY_ID_BCM5464                 0x002060b0
 #define PHY_ID_BCM5461                 0x002060c0
 #define PHY_ID_BCM54612E               0x03625e60