blob: b706a75db98535f2a21c24e1818bc65f645b469a [file] [log] [blame]
From c7e3d1e8f6879e8165e0a061a33e2ad68e460df4 Mon Sep 17 00:00:00 2001
From: Russell King <>
Date: Tue, 10 Dec 2019 22:33:05 +0000
Subject: [PATCH] net: marvell: mvpp2: phylink requires the link interrupt
commit f3f2364ea14d1cf6bf966542f31eadcf178f1577 upstream.
phylink requires the MAC to report when its link status changes when
operating in inband modes. Failure to report link status changes
means that phylink has no idea when the link events happen, which
results in either the network interface's carrier remaining up or
remaining permanently down.
For example, with a fiber module, if the interface is brought up and
link is initially established, taking the link down at the far end
will cut the optical power. The SFP module's LOS asserts, we
deactivate the link, and the network interface reports no carrier.
When the far end is brought back up, the SFP module's LOS deasserts,
but the MAC may be slower to establish link. If this happens (which
in my tests is a certainty) then phylink never hears that the MAC
has established link with the far end, and the network interface is
stuck reporting no carrier. This means the interface is
Avoiding the link interrupt when we have phylink is basically not
an option, so remove the !port->phylink from the test.
Fixes: 4bb043262878 ("net: mvpp2: phylink support")
Tested-by: Sven Auhagen <>
Tested-by: Antoine Tenart <>
Signed-off-by: Russell King <>
Signed-off-by: Jakub Kicinski <>
Signed-off-by: Paul Gortmaker <>
diff --git a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
index 885529701de9..b8c576d9656d 100644
--- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
+++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
@@ -3445,7 +3445,7 @@ static int mvpp2_open(struct net_device *dev)
valid = true;
- if (priv->hw_version == MVPP22 && port->link_irq && !port->phylink) {
+ if (priv->hw_version == MVPP22 && port->link_irq) {
err = request_irq(port->link_irq, mvpp2_link_status_isr, 0,
dev->name, port);
if (err) {