diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2020-01-17 19:17:16 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2020-01-20 08:53:19 +0100 |
commit | 70fcc51b1066d69b7042035d64e797eea5e75d6c (patch) | |
tree | 2a9b44b06b9176cee4710c0bdb9de84d8e255ec6 /common | |
parent | 6745b22a18733425fd0b9d28c744cdc414a8d7a9 (diff) | |
download | barebox-70fcc51b1066d69b7042035d64e797eea5e75d6c.tar.gz barebox-70fcc51b1066d69b7042035d64e797eea5e75d6c.tar.xz |
net: designware: eqos: don't funnel all MDIO writes to register 0
eqos_mdio_write ended up using the addr parameter instead of the
computed miiaddr variable, which would've factored in the reg parameter.
This had the effect that all writes went to PHY register 0, which was
fine as long as there were only register 0 writes. As soon there are more
writes, e.g. because a PHY driver was enabled, register 0 became
clobbered and erratic behavior ensued.
Fix the typo and while at it rename the val parameter to a more
descriptive name.
Fixes: a4f709bbb ("net: add Designware Ethernet QoS for STM32MP")
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'common')
0 files changed, 0 insertions, 0 deletions