svn commit: r274635 - head/sys/dev/pci

[Chagin Dmitry]

On Mon, Nov 17, 2014 at 05:45:23PM -0500, John Baldwin wrote:
> On Monday, November 17, 2014 08:25:21 PM Dmitry Chagin wrote:
> > Author: dchagin
> > Date: Mon Nov 17 20:25:21 2014
> > New Revision: 274635
> > URL: https://svnweb.freebsd.org/changeset/base/274635
> > 
> > Log:
> >   Use the correct device as the power_for_sleep() method
> >   always pass request up to parent bridge.
> > 
> >   Reviewed by:	jhb
> >   MFC after:	1 week
> >   xMFC:		r274386,r274397
> 
> Hmm, I had misread this diff as being another instance of what Adrian had 
> found.  I think this is actually not correct.  The parent of a pci device 
> ('dev' here) is a pciX device.  pciX devices do not implement the 
> PCIB_POWER_FOR_SLEEP method from pcib_if.m, so this probably just broke 
> Adrian's Asus laptop as now it will always use D3 again and ignore _SxD.
ugh.. i understood, let me see more closely
> 
> You said that this change fixed your laptop, yes?  Did it work before Adrian's 
> change in r274386 and break afterwards?
yes

-- 
Have fun!
chd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20141118/e12e2c5c/attachment.sig>