[PATCH net-next v2 2/3] net: phy: bcm5481x: Implement MII-Lite mode
From: Kamil Horák - 2N <hidden>
Date: 2025-06-23 15:11:39
Also in:
linux-devicetree, lkml
Subsystem:
broadcom ethernet phy drivers, ethernet phy library, networking drivers, the rest · Maintainers:
Florian Fainelli, Andrew Lunn, Heiner Kallweit, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
From: Kamil Horák (2N) <redacted> The Broadcom bcm54810 and bcm54811 PHYs are capable to operate in simplified MII mode, without TXER, RXER, CRS and COL signals as defined for the MII. While the PHY can be strapped for MII mode, the selection between MII and MII-Lite must be done by software. The MII-Lite mode can be used with some Ethernet controllers, usually those used in automotive applications. The absence of COL signal makes half-duplex link modes impossible but does not interfere with BroadR-Reach link modes on Broadcom PHYs, because they are full-duplex only. The MII-Lite mode can be also used on an Ethernet controller with full MII interface by just leaving the input signals (RXER, CRS, COL) inactive. Signed-off-by: Kamil Horák (2N) <redacted> --- drivers/net/phy/broadcom.c | 32 +++++++++++++++++++++++++++++++- include/linux/brcmphy.h | 6 ++++++ 2 files changed, 37 insertions(+), 1 deletion(-)
diff --git a/drivers/net/phy/broadcom.c b/drivers/net/phy/broadcom.c
index 75dbb88bec5a..d0ecb12d2d2e 100644
--- a/drivers/net/phy/broadcom.c
+++ b/drivers/net/phy/broadcom.c@@ -16,7 +16,6 @@ #include <linux/delay.h> #include <linux/module.h> #include <linux/phy.h> -#include <linux/device.h> #include <linux/brcmphy.h> #include <linux/of.h> #include <linux/interrupt.h>
@@ -39,6 +38,7 @@ struct bcm54xx_phy_priv { int wake_irq; bool wake_irq_enabled; bool brr_mode; + bool mii_lite_mode; }; /* Link modes for BCM58411 PHY */
@@ -680,6 +680,12 @@ static int bcm5481x_read_abilities(struct phy_device *phydev) priv->brr_mode = of_property_read_bool(np, "brr-mode"); + /* Enable MII Lite (No TXER, RXER, CRS, COL) if configured */ + err = bcm_phy_modify_exp(phydev, BCM_EXP_SYNC_ETHERNET, + BCM_EXP_SYNC_ETHERNET_MII_LITE, + priv->mii_lite_mode ? + BCM_EXP_SYNC_ETHERNET_MII_LITE : 0); + /* Set BroadR-Reach mode as configured in the DT. */ err = bcm5481x_set_brrmode(phydev, priv->brr_mode); if (err)
@@ -1140,6 +1146,7 @@ static int bcm54xx_phy_probe(struct phy_device *phydev) struct bcm54xx_phy_priv *priv; struct gpio_desc *wakeup_gpio; int ret = 0; + struct device_node *np = phydev->mdio.dev.of_node; priv = devm_kzalloc(&phydev->mdio.dev, sizeof(*priv), GFP_KERNEL); if (!priv)
@@ -1159,6 +1166,29 @@ static int bcm54xx_phy_probe(struct phy_device *phydev) if (IS_ERR(priv->ptp)) return PTR_ERR(priv->ptp); + priv->mii_lite_mode = of_property_read_bool(np, "mii-lite-mode"); + if (!phy_interface_is_rgmii(phydev) || + phydev->interface == PHY_INTERFACE_MODE_MII) { + /* Enable MII Lite (No TXER, RXER, CRS, COL) if configured */ + ret = bcm_phy_modify_exp(phydev, BCM_EXP_SYNC_ETHERNET, + BCM_EXP_SYNC_ETHERNET_MII_LITE, + priv->mii_lite_mode ? + BCM_EXP_SYNC_ETHERNET_MII_LITE : 0); + if (ret < 0) + return ret; + /* Misc Control: GMII/MII Mode (not RGMII) */ + ret = phy_write(phydev, MII_BCM54XX_AUX_CTL, + MII_BCM54XX_AUXCTL_MISC_WREN | + MII_BCM54XX_AUXCTL_SHDWSEL_MASK | + MII_BCM54XX_AUXCTL_SHDWSEL_MISC | + (MII_BCM54XX_AUXCTL_SHDWSEL_MISC + << MII_BCM54XX_AUXCTL_SHDWSEL_READ_SHIFT) | + MII_BCM54XX_AUXCTL_SHDWSEL_MISC_RGMII_SKEW_EN | + MII_BCM54XX_AUXCTL_SHDWSEL_MISC_RSVD); + if (ret < 0) + return ret; + } + /* We cannot utilize the _optional variant here since we want to know * whether the GPIO descriptor exists or not to advertise Wake-on-LAN * support or not.
diff --git a/include/linux/brcmphy.h b/include/linux/brcmphy.h
index 350846b010e9..115a964f3006 100644
--- a/include/linux/brcmphy.h
+++ b/include/linux/brcmphy.h@@ -183,6 +183,12 @@ #define BCM_LED_MULTICOLOR_ACT 0x9 #define BCM_LED_MULTICOLOR_PROGRAM 0xa +/* + * Broadcom Synchronous Ethernet Controls (expansion register 0x0E) + */ +#define BCM_EXP_SYNC_ETHERNET (MII_BCM54XX_EXP_SEL_ER + 0x0E) +#define BCM_EXP_SYNC_ETHERNET_MII_LITE BIT(11) + /* * BCM5482: Shadow registers * Shadow values go into bits [14:10] of register 0x1c to select a shadow
--
2.39.5