OSDN Git Service

r8169: set RX_MULTI_EN bit in RxConfig for 8168F-family chips
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Thu, 11 Oct 2018 14:02:10 +0000 (16:02 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Oct 2018 19:08:04 +0000 (12:08 -0700)
commit511cfd580f23b0e0fcd5659931ef14c6e2c062b0
treebc23d803d7d71a1eebfe1eb370af75eeb3a64f87
parent2a1e89df785082a0fd7264ca6d3d834abe84fa25
r8169: set RX_MULTI_EN bit in RxConfig for 8168F-family chips

It has been reported that since
commit 05212ba8132b42 ("r8169: set RxConfig after tx/rx is enabled for RTL8169sb/8110sb devices")
at least RTL_GIGA_MAC_VER_38 NICs work erratically after a resume from
suspend.
The problem has been traced to a missing RX_MULTI_EN bit in the RxConfig
register.
We already set this bit for RTL_GIGA_MAC_VER_35 NICs of the same 8168F
chip family so let's do it also for its other siblings: RTL_GIGA_MAC_VER_36
and RTL_GIGA_MAC_VER_38.

Curiously, the NIC seems to work fine after a system boot without having
this bit set as long as the system isn't suspended and resumed.

Fixes: 05212ba8132b42 ("r8169: set RxConfig after tx/rx is enabled for RTL8169sb/8110sb devices")
Reported-by: Chris Clayton <chris2553@googlemail.com>
Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Reviewed-by: Heiner Kallweit <hkallweit1@gmail.com>
Tested-by: Chris Clayton <chris2553@googlemail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169.c