Re: [PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-13 Thread Rafael J. Wysocki
On Tuesday, June 13, 2017 10:52:52 AM Greg KH wrote: > On Mon, Jun 12, 2017 at 10:49:40PM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > hcd_pci_resume_noirq() used as a universal _resume_noirq handler for > > PCI USB controllers calls

Re: [PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-13 Thread Rafael J. Wysocki
On Tuesday, June 13, 2017 10:52:52 AM Greg KH wrote: > On Mon, Jun 12, 2017 at 10:49:40PM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > hcd_pci_resume_noirq() used as a universal _resume_noirq handler for > > PCI USB controllers calls pci_back_from_sleep() which is

Re: [PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-13 Thread Greg KH
On Mon, Jun 12, 2017 at 10:49:40PM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > hcd_pci_resume_noirq() used as a universal _resume_noirq handler for > PCI USB controllers calls pci_back_from_sleep() which is unnecessary > and may become problematic. >

Re: [PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-13 Thread Greg KH
On Mon, Jun 12, 2017 at 10:49:40PM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > hcd_pci_resume_noirq() used as a universal _resume_noirq handler for > PCI USB controllers calls pci_back_from_sleep() which is unnecessary > and may become problematic. > > It is unnecessary,

[PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki hcd_pci_resume_noirq() used as a universal _resume_noirq handler for PCI USB controllers calls pci_back_from_sleep() which is unnecessary and may become problematic. It is unnecessary, because the PCI bus type carries out post-suspend cleanup

[PATCH v2 2/8] USB / PCI / PM: Allow the PCI core to do the resume cleanup

2017-06-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki hcd_pci_resume_noirq() used as a universal _resume_noirq handler for PCI USB controllers calls pci_back_from_sleep() which is unnecessary and may become problematic. It is unnecessary, because the PCI bus type carries out post-suspend cleanup of all PCI devices during