Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-25 Thread Takashi Iwai
On Sun, 25 Feb 2018 09:59:00 +0100, Lukas Wunner wrote: > > On Wed, Feb 21, 2018 at 01:39:34PM +0100, Rafael J. Wysocki wrote: > > On Wednesday, February 21, 2018 10:57:14 AM CET Rafael J. Wysocki wrote: > > > So if pci_pm_runtime_suspend() is modified to call pci_save_state() > > > before returni

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-25 Thread Lukas Wunner
On Wed, Feb 21, 2018 at 01:39:34PM +0100, Rafael J. Wysocki wrote: > On Wednesday, February 21, 2018 10:57:14 AM CET Rafael J. Wysocki wrote: > > So if pci_pm_runtime_suspend() is modified to call pci_save_state() > > before returning 0 in the !dev->driver case, we can just move the > > pci_restore

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-21 Thread Rafael J. Wysocki
On Wednesday, February 21, 2018 10:57:14 AM CET Rafael J. Wysocki wrote: > On Tue, Feb 20, 2018 at 10:29 PM, Bjorn Helgaas wrote: > > On Sun, Feb 18, 2018 at 09:38:32AM +0100, Lukas Wunner wrote: > >> PCI devices not bound to a driver are supposed to stay in D0 during > >> runtime suspend. > > > >

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-21 Thread Rafael J. Wysocki
On Tue, Feb 20, 2018 at 10:29 PM, Bjorn Helgaas wrote: > On Sun, Feb 18, 2018 at 09:38:32AM +0100, Lukas Wunner wrote: >> PCI devices not bound to a driver are supposed to stay in D0 during >> runtime suspend. > > Doesn't "runtime suspend" mean an individual device is suspended while > the rest of

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-20 Thread Bjorn Helgaas
On Sun, Feb 18, 2018 at 09:38:32AM +0100, Lukas Wunner wrote: > PCI devices not bound to a driver are supposed to stay in D0 during > runtime suspend. Doesn't "runtime suspend" mean an individual device is suspended while the rest of the system remains active? If so, maybe it would be more direct

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-19 Thread Rafael J. Wysocki
On Sun, Feb 18, 2018 at 9:38 AM, Lukas Wunner wrote: > PCI devices not bound to a driver are supposed to stay in D0 during > runtime suspend. But they may have a parent which is bound and can be > transitioned to D3cold at runtime. Once the parent goes to D3cold, the > unbound child may go to D3

[PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-18 Thread Lukas Wunner
PCI devices not bound to a driver are supposed to stay in D0 during runtime suspend. But they may have a parent which is bound and can be transitioned to D3cold at runtime. Once the parent goes to D3cold, the unbound child may go to D3cold as well. When the child comes out of D3cold, its BARs ar