Subject: Re: Regression: Wake on LAN doesn't work in sky2 with 2.6.33-rc4-git2

From: Stephen Hemminger <[email-masked]>
Date: Mon, 18 Jan 2010 14:43:40 -0800

On Mon, 18 Jan 2010 22:34:55 +0100
"Rafael J. Wysocki" <[email-masked]> wrote:

> On Monday 18 January 2010, Tino Keitel wrote:
> > Hi,
> > 
> > reverting this commit fixes WoL for me:
> > 
> > commit 166a0fd4c788ec7f10ca8194ec6d526afa12db75
> > Author: stephen hemminger <[email-masked]>
> > Date:   Mon Dec 14 08:50:12 2009 +0000
> > 
> >     sky2: leave PCI config space writeable
> >     
> >     Since power management is done by PCI subsystem as well as driver,
> >     don't toggle the bit that disables PCI register writes.
> >     
> >     Signed-off-by: Stephen Hemminger <[email-masked]>
> >     Acked-by: Rafael J. Wysocki <[email-masked]>
> >     Signed-off-by: David S. Miller <[email-masked]>
> 
> Hmm, this is kind of interesting.
> 
> Perhaps we clear the WoL setting somewhere we shouldn't, but the
> "disable PCI register writes" bit prevents this from actually happening?
> 
> Rafael

How about getting a register dump of pci config space with
good/bad version?

--