Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-03 Thread Rafael J. Wysocki
sourceforge.net, > > linux-...@vger.kernel.org, "Dave Airlie" , "Rafael J. > > Wysocki" > > Sent: Saturday, 2 February, 2013 10:12:03 PM > > Subject: Re: [PATCH 3/5] PCI: revert preparing for wakeup in > > runtime-suspend finalization >

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-03 Thread David Airlie
> Wysocki" > Sent: Saturday, 2 February, 2013 10:12:03 PM > Subject: Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend > finalization > > Rafael J. Wysocki wrote: > > On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: > >>

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-03 Thread David Airlie
> Rafael J. Wysocki wrote: > > On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: > >> On Tuesday, January 29, 2013 11:04:57 AM Konstantin Khlebnikov > >> wrote: > >>> Rafael J. Wysocki wrote: > On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: > > [+cc Rafael] >

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-03 Thread Rafael J. Wysocki
On Sunday, February 03, 2013 02:14:46 PM Konstantin Khlebnikov wrote: > Rafael J. Wysocki wrote: > > On Saturday, February 02, 2013 09:58:45 PM Rafael J. Wysocki wrote: > >> On Saturday, February 02, 2013 04:12:03 PM Konstantin Khlebnikov wrote: > >>> Rafael J. Wysocki wrote: > On Tuesday, Jan

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-03 Thread Konstantin Khlebnikov
Rafael J. Wysocki wrote: On Saturday, February 02, 2013 09:58:45 PM Rafael J. Wysocki wrote: On Saturday, February 02, 2013 04:12:03 PM Konstantin Khlebnikov wrote: Rafael J. Wysocki wrote: On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: On Tuesday, January 29, 2013 11:04:57

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-02 Thread Rafael J. Wysocki
On Saturday, February 02, 2013 09:58:45 PM Rafael J. Wysocki wrote: > On Saturday, February 02, 2013 04:12:03 PM Konstantin Khlebnikov wrote: > > Rafael J. Wysocki wrote: > > > On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: > > >> On Tuesday, January 29, 2013 11:04:57 AM Konstant

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-02 Thread Rafael J. Wysocki
On Saturday, February 02, 2013 04:12:03 PM Konstantin Khlebnikov wrote: > Rafael J. Wysocki wrote: > > On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: > >> On Tuesday, January 29, 2013 11:04:57 AM Konstantin Khlebnikov wrote: > >>> Rafael J. Wysocki wrote: > On Monday, Januar

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-02-02 Thread Konstantin Khlebnikov
Rafael J. Wysocki wrote: On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: On Tuesday, January 29, 2013 11:04:57 AM Konstantin Khlebnikov wrote: Rafael J. Wysocki wrote: On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: [+cc Rafael] On Fri, Jan 18, 2013 at 4:42 AM,

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-30 Thread Rafael J. Wysocki
On Tuesday, January 29, 2013 12:55:15 PM Rafael J. Wysocki wrote: > On Tuesday, January 29, 2013 11:04:57 AM Konstantin Khlebnikov wrote: > > Rafael J. Wysocki wrote: > > > On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: > > >> [+cc Rafael] > > >> > > >> On Fri, Jan 18, 2013 at 4:42 AM

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-29 Thread Rafael J. Wysocki
On Tuesday, January 29, 2013 11:04:57 AM Konstantin Khlebnikov wrote: > Rafael J. Wysocki wrote: > > On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: > >> [+cc Rafael] > >> > >> On Fri, Jan 18, 2013 at 4:42 AM, Konstantin Khlebnikov > >> wrote: > >>> This patch effectively reverts com

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-28 Thread Konstantin Khlebnikov
Rafael J. Wysocki wrote: On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: [+cc Rafael] On Fri, Jan 18, 2013 at 4:42 AM, Konstantin Khlebnikov wrote: This patch effectively reverts commit 42eca2302146fed51335b95128e949ee6f54478f ("PCI: Don't touch card regs after runtime suspend D

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-28 Thread Rafael J. Wysocki
On Monday, January 28, 2013 04:17:42 PM Bjorn Helgaas wrote: > [+cc Rafael] > > On Fri, Jan 18, 2013 at 4:42 AM, Konstantin Khlebnikov > wrote: > > This patch effectively reverts commit > > 42eca2302146fed51335b95128e949ee6f54478f > > ("PCI: Don't touch card regs after runtime suspend D3") > > >

Re: [PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-28 Thread Bjorn Helgaas
[+cc Rafael] On Fri, Jan 18, 2013 at 4:42 AM, Konstantin Khlebnikov wrote: > This patch effectively reverts commit 42eca2302146fed51335b95128e949ee6f54478f > ("PCI: Don't touch card regs after runtime suspend D3") > > | This patch checks whether the pci state is saved and doesn't attempt to hit >

[PATCH 3/5] PCI: revert preparing for wakeup in runtime-suspend finalization

2013-01-18 Thread Konstantin Khlebnikov
This patch effectively reverts commit 42eca2302146fed51335b95128e949ee6f54478f ("PCI: Don't touch card regs after runtime suspend D3") | This patch checks whether the pci state is saved and doesn't attempt to hit | any registers after that point if it is. This seems completely wrong. Yes, PCI con