diff options
author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2021-09-23 16:23:33 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-09-24 16:03:04 +0300 |
commit | 325fd36ae76a6d089983b2d2eccb41237d35b221 (patch) | |
tree | 6155c696c4b6759e697a7b1a907ce99525499d8f /drivers/nvmem/bcm-ocotp.c | |
parent | 420070197b11a0525591116326178794c1f7b9d0 (diff) | |
download | linux-325fd36ae76a6d089983b2d2eccb41237d35b221.tar.xz |
net: enetc: fix the incorrect clearing of IF_MODE bits
The enetc phylink .mac_config handler intends to clear the IFMODE field
(bits 1:0) of the PM0_IF_MODE register, but incorrectly clears all the
other fields instead.
For normal operation, the bug was inconsequential, due to the fact that
we write the PM0_IF_MODE register in two stages, first in
phylink .mac_config (which incorrectly cleared out a bunch of stuff),
then we update the speed and duplex to the correct values in
phylink .mac_link_up.
Judging by the code (not tested), it looks like maybe loopback mode was
broken, since this is one of the settings in PM0_IF_MODE which is
incorrectly cleared.
Fixes: c76a97218dcb ("net: enetc: force the RGMII speed and duplex instead of operating in inband mode")
Reported-by: Pavel Machek (CIP) <pavel@denx.de>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/nvmem/bcm-ocotp.c')
0 files changed, 0 insertions, 0 deletions