]> git.baikalelectronics.ru Git - kernel.git/commitdiff
net: phy: micrel: add Asym Pause workaround for KSZ9021
authorHans Andersson <hans.andersson@cellavision.se>
Thu, 26 Sep 2019 07:54:37 +0000 (09:54 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Sep 2019 18:27:26 +0000 (20:27 +0200)
The Micrel KSZ9031 PHY may fail to establish a link when the Asymmetric
Pause capability is set. This issue is described in a Silicon Errata
(DS80000691D or DS80000692D), which advises to always disable the
capability.

Micrel KSZ9021 has no errata, but has the same issue with Asymmetric Pause.
This patch apply the same workaround as the one for KSZ9031.

Fixes: 1a9757c46d77 ("net: phy: micrel: add Asym Pause workaround")
Signed-off-by: Hans Andersson <hans.andersson@cellavision.se>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/micrel.c

index 3c8186f269f9e088cee19bd509ba03e84ed58e1c..2fea5541c35a8a02910a09bbbaa0de7b9fcbc6c1 100644 (file)
@@ -763,6 +763,8 @@ static int ksz9031_get_features(struct phy_device *phydev)
         * Whenever the device's Asymmetric Pause capability is set to 1,
         * link-up may fail after a link-up to link-down transition.
         *
+        * The Errata Sheet is for ksz9031, but ksz9021 has the same issue
+        *
         * Workaround:
         * Do not enable the Asymmetric Pause capability bit.
         */
@@ -1076,6 +1078,7 @@ static struct phy_driver ksphy_driver[] = {
        /* PHY_GBIT_FEATURES */
        .driver_data    = &ksz9021_type,
        .probe          = kszphy_probe,
+       .get_features   = ksz9031_get_features,
        .config_init    = ksz9021_config_init,
        .ack_interrupt  = kszphy_ack_interrupt,
        .config_intr    = kszphy_config_intr,