On 8/23/2016 2:03 PM, Florian Fainelli wrote:
+others,

On 08/23/2016 04:13 AM, Christophe Leroy wrote:
In ERRATA DS80000700A dated 05 May 2016, Microship recommends to
not use software power down mode on KSZ8041 family.

s/Microship/Microchip/

They say they have no plan to fix this ERRATA in future releases.

The errata applies to specific revisions, is this revision present in
the lower 4 bits of the MII_PHYSID2 register such that it could be used
to key the disabling of the power down?


http://ww1.microchip.com/downloads/en/DeviceDoc/80000700A.pdf

Signed-off-by: Christophe Leroy <christophe.le...@c-s.fr>
---
  drivers/net/phy/micrel.c | 4 ----
  1 file changed, 4 deletions(-)

diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c
index 053e879..f456c55 100644
--- a/drivers/net/phy/micrel.c
+++ b/drivers/net/phy/micrel.c
@@ -837,8 +837,6 @@ static struct phy_driver ksphy_driver[] = {
        .get_sset_count = kszphy_get_sset_count,
        .get_strings    = kszphy_get_strings,
        .get_stats      = kszphy_get_stats,
-       .suspend        = genphy_suspend,
-       .resume         = genphy_resume,
  }, {
        .phy_id         = PHY_ID_KSZ8041RNLI,
        .phy_id_mask    = MICREL_PHY_ID_MASK,
@@ -856,8 +854,6 @@ static struct phy_driver ksphy_driver[] = {
        .get_sset_count = kszphy_get_sset_count,
        .get_strings    = kszphy_get_strings,
        .get_stats      = kszphy_get_stats,
-       .suspend        = genphy_suspend,
-       .resume         = genphy_resume,
  }, {
        .phy_id         = PHY_ID_KSZ8051,
        .phy_id_mask    = MICREL_PHY_ID_MASK,



It doesn't appear that KSZ9031 is affected by this errata; no mention of 
suspend:

http://ww1.microchip.com/downloads/en/DeviceDoc/KSZ9031MNX_Errata_Rev%20A_A2_032515.pdf

--
Xander Huff
Staff Software Engineer
National Instruments

Reply via email to