diff mbox series

[net] net: dsa: bcm_sf2: Only 7278 supports 2Gb/sec IMP port

Message ID 20200206192352.19939-1-f.fainelli@gmail.com
State Accepted
Delegated to: David Miller
Headers show
Series [net] net: dsa: bcm_sf2: Only 7278 supports 2Gb/sec IMP port | expand

Commit Message

Florian Fainelli Feb. 6, 2020, 7:23 p.m. UTC
The 7445 switch clocking profiles do not allow us to run the IMP port at
2Gb/sec in a way that it is reliable and consistent. Make sure that the
setting is only applied to the 7278 family.

Fixes: 8f1880cbe8d0 ("net: dsa: bcm_sf2: Configure IMP port for 2Gb/sec")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 drivers/net/dsa/bcm_sf2.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

David Miller Feb. 7, 2020, 10:28 a.m. UTC | #1
From: Florian Fainelli <f.fainelli@gmail.com>
Date: Thu,  6 Feb 2020 11:23:52 -0800

> The 7445 switch clocking profiles do not allow us to run the IMP port at
> 2Gb/sec in a way that it is reliable and consistent. Make sure that the
> setting is only applied to the 7278 family.
> 
> Fixes: 8f1880cbe8d0 ("net: dsa: bcm_sf2: Configure IMP port for 2Gb/sec")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Also applied and queued up for -stable.
diff mbox series

Patch

diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c
index 3e8635311d0d..d1955543acd1 100644
--- a/drivers/net/dsa/bcm_sf2.c
+++ b/drivers/net/dsa/bcm_sf2.c
@@ -68,7 +68,9 @@  static void bcm_sf2_imp_setup(struct dsa_switch *ds, int port)
 
 		/* Force link status for IMP port */
 		reg = core_readl(priv, offset);
-		reg |= (MII_SW_OR | LINK_STS | GMII_SPEED_UP_2G);
+		reg |= (MII_SW_OR | LINK_STS);
+		if (priv->type == BCM7278_DEVICE_ID)
+			reg |= GMII_SPEED_UP_2G;
 		core_writel(priv, reg, offset);
 
 		/* Enable Broadcast, Multicast, Unicast forwarding to IMP port */