diff options
author | Antoine Tenart <antoine.tenart@bootlin.com> | 2019-03-01 11:52:08 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-03-02 08:23:34 +0100 |
commit | 8b318f30ab4ef9bbc1241e6f8c1db366dbd347f2 (patch) | |
tree | 074c336590e8a41827d28eb031c4cdcffe2c11e2 /drivers/net/ethernet/marvell | |
parent | net: mvpp2: reconfiguring the port interface is PPv2.2 specific (diff) | |
download | linux-8b318f30ab4ef9bbc1241e6f8c1db366dbd347f2.tar.xz linux-8b318f30ab4ef9bbc1241e6f8c1db366dbd347f2.zip |
net: mvpp2: fix validate for PPv2.1
The Phylink validate function is the Marvell PPv2 driver makes a check
on the GoP id. This is valid an has to be done when using PPv2.2 engines
but makes no sense when using PPv2.1. The check done when using an RGMII
interface makes sure the GoP id is not 0, but this breaks PPv2.1. Fixes
it.
Fixes: 0fb628f0f250 ("net: mvpp2: fix phylink handling of invalid PHY modes")
Signed-off-by: Antoine Tenart <antoine.tenart@bootlin.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/marvell')
-rw-r--r-- | drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c index 9467a2258ee0..e7d768381f4f 100644 --- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c +++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c @@ -4363,7 +4363,7 @@ static void mvpp2_phylink_validate(struct net_device *dev, case PHY_INTERFACE_MODE_RGMII_ID: case PHY_INTERFACE_MODE_RGMII_RXID: case PHY_INTERFACE_MODE_RGMII_TXID: - if (port->gop_id == 0) + if (port->priv->hw_version == MVPP22 && port->gop_id == 0) goto empty_set; break; default: |