net: phy: broadcom: fix BCM54213PE per-PHY init never called

BCM54213PE (PHY_ID = 0x600d84a2) and BCM54210E share the same model ID
when masked by PHY_ID_MATCH_MODEL_MASK (0xfffffff0): both reduce to
0x600d84a0.  The dispatch switch in bcm54xx_config_init() switches on
phydev->drv->phy_id & PHY_ID_MATCH_MODEL_MASK, so the separate
case PHY_ID_BCM54213PE: could never match — the expression always
evaluated to 0x600d84a0, not 0x600d84a2.  bcm54213pe_config_init() was
silently never called; BCM54213PE instead fell through to the
BCM54210E path.

Replace the dead case label with an exact driver ID check inside the
BCM54210E case, which already handles the same model ID family.

Fixes: 1001c6f ("phy: broadcom: Add bcm54213pe configuration")
Signed-off-by: Nicolai Buchwitz <nb@tipi-net.de>

Author: nbuchwitz

drivers/net/phy/broadcom.c

@@ -498,14 +498,17 @@ static int bcm54xx_config_init(struct phy_device *phydev)
 		err = bcm54xx_config_clock_delay(phydev);
 		break;
 	case PHY_ID_BCM54210E:
-		err = bcm54210e_config_init(phydev);
+		/* BCM54213PE shares the same model ID as BCM54210E; use the
+		 * exact driver ID to dispatch to the right per-PHY init.
+		 */
+		if (phydev->drv->phy_id == PHY_ID_BCM54213PE)
+			err = bcm54213pe_config_init(phydev);
+		else
+			err = bcm54210e_config_init(phydev);
 		break;
 	case PHY_ID_BCM54612E:
 		err = bcm54612e_config_init(phydev);
 		break;
-	case PHY_ID_BCM54213PE:
-		err = bcm54213pe_config_init(phydev);
-		break;
 	case PHY_ID_BCM54616S:
 		err = bcm54616s_config_init(phydev);
 		break;