Re: [PATCH] r8169: Fix WOL in power down case

[Stefan Becker <chemobejk@gmail.com>]

Marc Ballarin <ballarin.marc <at> gmx.de> writes:
> 
> On Sun, 9 Oct 2011 13:54:38 +0200
> Francois Romieu <romieu <at> fr.zoreil.com> wrote:
> 
> > ...
> > Moreover, the patch duplicates a chunk of conditional code:
> > 1) it bloats the driver (ok, we have seen worse...).
> > 2) you can bet that both instances won't be updated if / when
> >    RTL_GIGA_MAC_VER_xy appears.
> 
> I thought so 
> 
> > 
> > The patch below should fix it.
> 
> Yes, it works fine here. Thanks a lot.
> 

I just upgraded to Fedora 16 with Kernel 3.1.0 and WoL stopped working in the
power down case on my R8168 :-(

[    8.557372] r8169 0000:02:00.0: eth0: RTL8168c/8111c at 0xf809a000,
00:1f:d0:5a:22:77, XID 1c4000c0 IRQ 43

02:00.0 Ethernet controller [0200]: Realtek Semiconductor Co., Ltd.
RTL8111/8168B PCI Express Gigabit Ethernet controller [10ec:8168] (rev 02)

I've checked the SRPM and git commit 649b3b8c4e868 with the fix discussed here
is included. It obviously doesn't work for my HW.

According to the code the combination "RTL8168c/8111c" means RTL_GIGA_MAC_VER_19
to _22, so the code changes in the fix commit don't apply to them. Could it be
that the fix doesn't cover all the necessary HW?

Regards,

           Stefan

PS: Please CC me in your reply, as I don't read the mailing list.