]> git.baikalelectronics.ru Git - kernel.git/commit
r8169: remove old PHY reset hack
authorHeiner Kallweit <hkallweit1@gmail.com>
Sat, 30 Jun 2018 22:25:19 +0000 (00:25 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 2 Jul 2018 13:40:31 +0000 (22:40 +0900)
commitbfcb25e1be43a754eaa86d768ed8982caea9ae57
tree094a2306bd6d8bf952bb510ee12366cf975fc1ef
parent8b2bb75f8563e5c4a6ca6e5be87b5fe463525b1d
r8169: remove old PHY reset hack

This hack (affecting the non-PCIe models only) was introduced in 2004
to deal with link negotiation failures in 1GBit mode. Based on a
comment in the r8169 vendor driver I assume the issue affects RTL8169sb
in combination with particular 1GBit switch models.

Resetting the PHY every 10s and hoping that one fine day we will make
it to establish the link seems to be very hacky to me. I'd say:
If 1GBit doesn't work reliably in a users environment then the user
should remove 1GBit from the advertised modes, e.g. by using
ethtool -s <if> advertise <10/100 modes>

If the issue affects one chip version only and that with most link
partners, then we could also think of removing 1GBit from the
advertised modes for this chip version in the driver.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169.c