]> git.sur5r.net Git - u-boot/commit
net: phy: meson-gxl: detect LPA corruption
authorJerome Brunet <jbrunet@baylibre.com>
Tue, 12 Dec 2017 15:03:55 +0000 (16:03 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 18 Dec 2017 18:06:30 +0000 (13:06 -0500)
commit30cbb524bca7c92ab7661de96839c54d794cb3cd
tree93e018e4aa2f6aa18f8897dc2e32536d4c64b935
parent3ef3fbbf9bfb50e78c46823adc6a7989cc1eb129
net: phy: meson-gxl: detect LPA corruption

This patch is ported from the Linux patch posted at [1] and applied to
net tree as commit f1e2400a80ff.

The purpose of this change is to fix the incorrect detection of the link
partner (LP) advertised capabilities which sometimes happens with this PHY
(roughly 1 time in a dozen)

This issue may cause the link to be negotiated at 10Mbps/Full or
10Mbps/Half when 100MBps/Full is actually possible. In some case, the link
is even completely broken and no communication is possible.

To detect the corruption, we must look for a magic undocumented bit in the
WOL bank (hint given by the SoC vendor kernel) but this is not enough to
cover all cases. We also have to look at the LPA ack. If the LP supports
Aneg but did not ack our base code when aneg is completed, we assume
something went wrong.

The detection of a corrupted LPA triggers a restart of the aneg process.
This solves the problem but may take up to 6 retries to complete.

[1] https://lkml.kernel.org/r/20171208110811.30789-1-jbrunet@baylibre.com

Fixes: 8995a96d1d67 ("net: phy: Add Amlogic Meson GXL Internal PHY support")
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
drivers/net/phy/meson-gxl.c