Re: [PATCH] usb: core: Add runtime resume checking

On 11 August 2016 at 16:44, Peter Chen wrote: > On Thu, Aug 11, 2016 at 04:41:27PM +0800, Baolin Wang wrote: >> >> >> >> >> >> OK. But that is a real problem. It will pm_runtime_resume() falied >> >> >> (issued in choose_wakeup()), cause usb controller has powered-off and

Re: [PATCH] usb: core: Add runtime resume checking

On 11 August 2016 at 16:44, Peter Chen wrote: > On Thu, Aug 11, 2016 at 04:41:27PM +0800, Baolin Wang wrote: >> >> >> >> >> >> OK. But that is a real problem. It will pm_runtime_resume() falied >> >> >> (issued in choose_wakeup()), cause usb controller has powered-off and >> >> >> xHCI controller

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 04:41:27PM +0800, Baolin Wang wrote: > >> >> > >> >> OK. But that is a real problem. It will pm_runtime_resume() falied > >> >> (issued in choose_wakeup()), cause usb controller has powered-off and > >> >> xHCI controller has suspended and we have no method to notify the

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 04:41:27PM +0800, Baolin Wang wrote: > >> >> > >> >> OK. But that is a real problem. It will pm_runtime_resume() falied > >> >> (issued in choose_wakeup()), cause usb controller has powered-off and > >> >> xHCI controller has suspended and we have no method to notify the

Re: [PATCH] usb: core: Add runtime resume checking

On 11 August 2016 at 16:19, Peter Chen wrote: > On Thu, Aug 11, 2016 at 04:07:21PM +0800, Baolin Wang wrote: >> Hi Peter, >> >> On 11 August 2016 at 14:54, Peter Chen wrote: >> > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: >> >> Hi

Re: [PATCH] usb: core: Add runtime resume checking

On 11 August 2016 at 16:19, Peter Chen wrote: > On Thu, Aug 11, 2016 at 04:07:21PM +0800, Baolin Wang wrote: >> Hi Peter, >> >> On 11 August 2016 at 14:54, Peter Chen wrote: >> > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: >> >> Hi Alan, >> >> >> >> On 10 August 2016 at 22:25,

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 04:07:21PM +0800, Baolin Wang wrote: > Hi Peter, > > On 11 August 2016 at 14:54, Peter Chen wrote: > > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: > >> Hi Alan, > >> > >> On 10 August 2016 at 22:25, Alan Stern

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 04:07:21PM +0800, Baolin Wang wrote: > Hi Peter, > > On 11 August 2016 at 14:54, Peter Chen wrote: > > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: > >> Hi Alan, > >> > >> On 10 August 2016 at 22:25, Alan Stern wrote: > >> > On Wed, 10 Aug 2016, Baolin

Re: [PATCH] usb: core: Add runtime resume checking

Hi Peter, On 11 August 2016 at 14:54, Peter Chen wrote: > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: >> Hi Alan, >> >> On 10 August 2016 at 22:25, Alan Stern wrote: >> > On Wed, 10 Aug 2016, Baolin Wang wrote: >> > >> >> >> >>

Re: [PATCH] usb: core: Add runtime resume checking

Hi Peter, On 11 August 2016 at 14:54, Peter Chen wrote: > On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: >> Hi Alan, >> >> On 10 August 2016 at 22:25, Alan Stern wrote: >> > On Wed, 10 Aug 2016, Baolin Wang wrote: >> > >> >> >> >> For example: No slave attached> usb interface

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: > Hi Alan, > > On 10 August 2016 at 22:25, Alan Stern wrote: > > On Wed, 10 Aug 2016, Baolin Wang wrote: > > > >> >> >> For example: No slave attached> usb interface runtime suspend > >> >> >> > usb

Re: [PATCH] usb: core: Add runtime resume checking

On Thu, Aug 11, 2016 at 11:08:52AM +0800, Baolin Wang wrote: > Hi Alan, > > On 10 August 2016 at 22:25, Alan Stern wrote: > > On Wed, 10 Aug 2016, Baolin Wang wrote: > > > >> >> >> For example: No slave attached> usb interface runtime suspend > >> >> >> > usb device runtime suspend

Re: [PATCH] usb: core: Add runtime resume checking

Hi Alan, On 10 August 2016 at 22:25, Alan Stern wrote: > On Wed, 10 Aug 2016, Baolin Wang wrote: > >> >> >> For example: No slave attached> usb interface runtime suspend >> >> >> > usb device runtime suspend -> xhci suspend -> power off >> >> >> usb

Re: [PATCH] usb: core: Add runtime resume checking

Hi Alan, On 10 August 2016 at 22:25, Alan Stern wrote: > On Wed, 10 Aug 2016, Baolin Wang wrote: > >> >> >> For example: No slave attached> usb interface runtime suspend >> >> >> > usb device runtime suspend -> xhci suspend -> power off >> >> >> usb controller. After that if the

Re: [PATCH] usb: core: Add runtime resume checking

Hi Peter, On 10 August 2016 at 14:18, Peter Chen wrote: > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> Hi Greg, >> >> On 9 August 2016 at 18:26, Greg KH wrote: >> > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote:

Re: [PATCH] usb: core: Add runtime resume checking

Hi Peter, On 10 August 2016 at 14:18, Peter Chen wrote: > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> Hi Greg, >> >> On 9 August 2016 at 18:26, Greg KH wrote: >> > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: >> >> When the usb device has entered suspend

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: > Hi Greg, > > On 9 August 2016 at 18:26, Greg KH wrote: > > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: > >> When the usb device has entered suspend state by runtime suspend method, > >>

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: > Hi Greg, > > On 9 August 2016 at 18:26, Greg KH wrote: > > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: > >> When the usb device has entered suspend state by runtime suspend method, > >> and > >> the sustem also try

Re: [PATCH] usb: core: Add runtime resume checking

Hi Greg, On 10 August 2016 at 16:17, Greg KH wrote: > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> Hi Greg, >> >> On 9 August 2016 at 18:26, Greg KH wrote: >> > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang

Re: [PATCH] usb: core: Add runtime resume checking

Hi Greg, On 10 August 2016 at 16:17, Greg KH wrote: > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> Hi Greg, >> >> On 9 August 2016 at 18:26, Greg KH wrote: >> > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: >> >> When the usb device has entered suspend state by

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, 10 Aug 2016, Baolin Wang wrote: > >> >> For example: No slave attached> usb interface runtime suspend > >> >> > usb device runtime suspend -> xhci suspend -> power off > >> >> usb controller. After that if the system wants to enter suspend state, > >> >> then it also will

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, 10 Aug 2016, Baolin Wang wrote: > >> >> For example: No slave attached> usb interface runtime suspend > >> >> > usb device runtime suspend -> xhci suspend -> power off > >> >> usb controller. After that if the system wants to enter suspend state, > >> >> then it also will

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, Aug 10, 2016 at 02:43:50PM +0800, Baolin Wang wrote: > Hi Peter, > > On 10 August 2016 at 14:18, Peter Chen wrote: > > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: > >> Hi Greg, > >> > >> On 9 August 2016 at 18:26, Greg KH

Re: [PATCH] usb: core: Add runtime resume checking

On 10 August 2016 at 17:07, Peter Chen wrote: > On Wed, Aug 10, 2016 at 02:43:50PM +0800, Baolin Wang wrote: >> Hi Peter, >> >> On 10 August 2016 at 14:18, Peter Chen wrote: >> > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> >> Hi

Re: [PATCH] usb: core: Add runtime resume checking

On Wed, Aug 10, 2016 at 02:43:50PM +0800, Baolin Wang wrote: > Hi Peter, > > On 10 August 2016 at 14:18, Peter Chen wrote: > > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: > >> Hi Greg, > >> > >> On 9 August 2016 at 18:26, Greg KH wrote: > >> > On Tue, Aug 09, 2016 at 05:33:33PM

Re: [PATCH] usb: core: Add runtime resume checking

On 10 August 2016 at 17:07, Peter Chen wrote: > On Wed, Aug 10, 2016 at 02:43:50PM +0800, Baolin Wang wrote: >> Hi Peter, >> >> On 10 August 2016 at 14:18, Peter Chen wrote: >> > On Wed, Aug 10, 2016 at 10:33:31AM +0800, Baolin Wang wrote: >> >> Hi Greg, >> >> >> >> On 9 August 2016 at 18:26,

Re: [PATCH] usb: core: Add runtime resume checking

Hi Greg, On 9 August 2016 at 18:26, Greg KH wrote: > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: >> When the usb device has entered suspend state by runtime suspend method, and >> the sustem also try to enter suspend state by issuing

Re: [PATCH] usb: core: Add runtime resume checking

Hi Greg, On 9 August 2016 at 18:26, Greg KH wrote: > On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: >> When the usb device has entered suspend state by runtime suspend method, and >> the sustem also try to enter suspend state by issuing usb_dev_suspend(), it >> will issue

Re: [PATCH] usb: core: Add runtime resume checking

On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: > When the usb device has entered suspend state by runtime suspend method, and > the sustem also try to enter suspend state by issuing usb_dev_suspend(), it > will issue pm_runtime_resume() function to deal with wrong wakeup setting in >

Re: [PATCH] usb: core: Add runtime resume checking

On Tue, Aug 09, 2016 at 05:33:33PM +0800, Baolin Wang wrote: > When the usb device has entered suspend state by runtime suspend method, and > the sustem also try to enter suspend state by issuing usb_dev_suspend(), it > will issue pm_runtime_resume() function to deal with wrong wakeup setting in >

[PATCH] usb: core: Add runtime resume checking

When the usb device has entered suspend state by runtime suspend method, and the sustem also try to enter suspend state by issuing usb_dev_suspend(), it will issue pm_runtime_resume() function to deal with wrong wakeup setting in choose_wakeup() function. But if usb device resumes failed due to

[PATCH] usb: core: Add runtime resume checking

When the usb device has entered suspend state by runtime suspend method, and the sustem also try to enter suspend state by issuing usb_dev_suspend(), it will issue pm_runtime_resume() function to deal with wrong wakeup setting in choose_wakeup() function. But if usb device resumes failed due to