net: phy: dp83822: use BMCR_ANENABLE instead of BMSR_ANEGCAPABLE for DP83620
authorAlvaro Gamez Machado <alvaro.gamez@hazent.com>
Fri, 8 Jun 2018 10:23:39 +0000 (12:23 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 10 Jun 2018 19:38:03 +0000 (12:38 -0700)
commitb718e8c8f4f5920aaddc2e52d5e32f494c91129c
treef7147d1b843d1f8219bba61bb006f3e0da6f6fec
parent6d8c50dcb029872b298eea68cc6209c866fd3e14
net: phy: dp83822: use BMCR_ANENABLE instead of BMSR_ANEGCAPABLE for DP83620

DP83620 register set is compatible with the DP83848, but it also supports
100base-FX. When the hardware is configured such as that fiber mode is
enabled, autonegotiation is not possible.

The chip, however, doesn't expose this information via BMSR_ANEGCAPABLE.
Instead, this bit is always set high, even if the particular hardware
configuration makes it so that auto negotiation is not possible [1]. Under
these circumstances, the phy subsystem keeps trying for autonegotiation to
happen, without success.

Hereby, we inspect BMCR_ANENABLE bit after genphy_config_init, which on
reset is set to 0 when auto negotiation is disabled, and so we use this
value instead of BMSR_ANEGCAPABLE.

[1] https://e2e.ti.com/support/interface/ethernet/f/903/p/697165/2571170

Signed-off-by: Alvaro Gamez Machado <alvaro.gamez@hazent.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/dp83848.c