aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/malta
diff options
context:
space:
mode:
authorMark Mentovai <mark@moxienet.com>2021-05-08 11:00:55 -0400
committerPetr Štetiar <ynezz@true.cz>2021-05-08 18:45:42 +0200
commitd62825dd77b3990b8ff2cf341ff3a8b16f4883d1 (patch)
tree16c9214756637f0692547e5c9182316c403794ec /target/linux/malta
parentf68c9474acf9a65b5a9538db8e45c173462487e3 (diff)
downloadupstream-d62825dd77b3990b8ff2cf341ff3a8b16f4883d1.tar.gz
upstream-d62825dd77b3990b8ff2cf341ff3a8b16f4883d1.tar.bz2
upstream-d62825dd77b3990b8ff2cf341ff3a8b16f4883d1.zip
ipq806x: dwmac: fix GMACs connected via SGMII fixed-link
fa731838c524 cleared the forced speed in the QSGMII PCS_ALL_CH_CTL register during probe, but this is only correct for GMACs that are not configured with fixed-link. This prevented GMACs configured with both phy-mode = "sgmii" and fixed-link from working properly, as discussed at https://github.com/openwrt/openwrt/pull/3954#issuecomment-834625090 and the comments that follow. Notably, this prevented all communication between gmac2 and the switch on the Netgear R7800. The correct behavior is to set the QSGMII PCS_ALL_CH_CTL register by considering the gmac's fixed-link child, setting the speed as directed by fixed-link if present, and otherwise clearing it as was done previously. Fixes: fa731838c524 ("ipq806x: dwmac: clear forced speed during probe") Signed-off-by: Mark Mentovai <mark@moxienet.com> Tested-by: Hannu Nyman <hannu.nyman@iki.fi> Run-tested: ipq806x/ubnt,unifi-ac-hd, ipq806x/netgear,r7800 Cc: Petr Štetiar <ynezz@true.cz> Cc: Ansuel Smith <ansuelsmth@gmail.com> Tested-by: Ansuel Smith <ansuelsmth@gmail.com>
Diffstat (limited to 'target/linux/malta')
0 files changed, 0 insertions, 0 deletions