From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH] sky2: Fix recent WOL regression Date: Thu, 9 Oct 2008 23:01:35 +0200 Message-ID: <200810092301.36633.rjw@sisk.pl> References: <200810022339.34025.rjw@sisk.pl> <200810090055.56598.rjw@sisk.pl> <20081008.161210.79287275.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Cc: shemminger@vyatta.com, jeff@garzik.org, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-pm@lists.linux-foundation.org, tino.keitel@gmx.de, netdev@vger.kernel.org, jbarnes@virtuousgeek.org To: David Miller Return-path: Received: from ogre.sisk.pl ([217.79.144.158]:47299 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754312AbYJIU52 (ORCPT ); Thu, 9 Oct 2008 16:57:28 -0400 In-Reply-To: <20081008.161210.79287275.davem@davemloft.net> Content-Disposition: inline Sender: netdev-owner@vger.kernel.org List-ID: On Thursday, 9 of October 2008, David Miller wrote: > From: "Rafael J. Wysocki" > Date: Thu, 9 Oct 2008 00:55:55 +0200 > > > On Friday, 3 of October 2008, Stephen Hemminger wrote: > > > On Thu, 2 Oct 2008 23:39:33 +0200 > > > "Rafael J. Wysocki" wrote: > > > > > > > From: Rafael J. Wysocki > > > > > > > > sky2: Fix recent WOL regression > > > > > > > > Since dev->power.should_wakeup bit is used by the PCI core to > > > > decide whether the device should wake up the system from sleep > > > > states, set/unset this bit whenever WOL is enabled/disabled using > > > > sky2_set_wol() (this fixes a regression from 2.6.26 on the Tino's > > > > machine). > > > > > > > > Remove an open-coded reference to the standard PCI PM registers that > > > > is not used any more. > > > > > > > > Signed-off-by: Rafael J. Wysocki > > > > Reported-by: Tino Keitel > > > > --- > > > > > > Acked-by: Stephen Hemminger > > > > > > Thanks for fixing this. > > > > Can you please include this patch into your next push for Jeff? > > > > It seems to have been overlooked. > > I missed this ACK but I did add the patch to my tree as you > can see from another email of mine. I saw it, thanks!