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)
commit335c997dce5c448ee06b3fd4dfe49fc7279f73ce
tree094a2306bd6d8bf952bb510ee12366cf975fc1ef
parentc02462d8a3e96d99967fb26fca7375d449328ce6
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