r8169: fix failing WoL
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 3 Jun 2020 20:29:06 +0000 (22:29 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 4 Jun 2020 22:51:54 +0000 (15:51 -0700)
commit120068481405f9e3d60a95bd464496dff6b54669
tree78a9183f488fce084f47298a1d5fc88cdb31e284
parentf6c1fb0a76d97447ea7f928ee6a113ee15318df1
r8169: fix failing WoL

Th referenced change added an extra hw reset to rtl8169_net_suspend()
what makes WoL fail on few chip versions. Therefore skip the extra
reset if we're going down and WoL is enabled.
In rtl_shutdown() rtl8169_hw_reset() is called by rtl8169_net_suspend()
already if needed, therefore avoid issues issue by removing the extra
call. The fix was tested on a system with RTL8168g.

Meanwhile rtl8169_hw_reset() does more than a hw reset and should be
renamed. But that's net-next material.

Fixes: 8ac8e8c64b53 ("r8169: make rtl8169_down central chip quiesce function")
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169_main.c