Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-11 Thread Baolin Wang
Hi Peter, On 11 August 2016 at 14:52, Peter Chen wrote: > On Thu, Aug 11, 2016 at 11:11:08AM +0800, Baolin Wang wrote: >> On 10 August 2016 at 22:31, Alan Stern wrote: >> > On Wed, 10 Aug 2016, Baolin Wang wrote: >> > >> >> Considering strict

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-11 Thread Baolin Wang
Hi Peter, On 11 August 2016 at 14:52, Peter Chen wrote: > On Thu, Aug 11, 2016 at 11:11:08AM +0800, Baolin Wang wrote: >> On 10 August 2016 at 22:31, Alan Stern wrote: >> > On Wed, 10 Aug 2016, Baolin Wang wrote: >> > >> >> Considering strict power management for mobile device, we should also

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-11 Thread Peter Chen
On Thu, Aug 11, 2016 at 11:11:08AM +0800, Baolin Wang wrote: > On 10 August 2016 at 22:31, Alan Stern wrote: > > On Wed, 10 Aug 2016, Baolin Wang wrote: > > > >> Considering strict power management for mobile device, we should also power > >> off the usb controller if

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-11 Thread Peter Chen
On Thu, Aug 11, 2016 at 11:11:08AM +0800, Baolin Wang wrote: > On 10 August 2016 at 22:31, Alan Stern wrote: > > On Wed, 10 Aug 2016, Baolin Wang wrote: > > > >> Considering strict power management for mobile device, we should also power > >> off the usb controller if there are no slaves attached

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Baolin Wang
On 10 August 2016 at 22:31, Alan Stern wrote: > On Wed, 10 Aug 2016, Baolin Wang wrote: > >> Considering strict power management for mobile device, we should also power >> off the usb controller if there are no slaves attached even though it is usb >> host function, but

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Baolin Wang
On 10 August 2016 at 22:31, Alan Stern wrote: > On Wed, 10 Aug 2016, Baolin Wang wrote: > >> Considering strict power management for mobile device, we should also power >> off the usb controller if there are no slaves attached even though it is usb >> host function, but it will meet usb device

[PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Baolin Wang
Considering strict power management for mobile device, we should also power off the usb controller if there are no slaves attached even though it is usb host function, but it will meet usb device resume failure in below situation. Suppose that no slave attached > usb interface runtime suspend

[PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Baolin Wang
Considering strict power management for mobile device, we should also power off the usb controller if there are no slaves attached even though it is usb host function, but it will meet usb device resume failure in below situation. Suppose that no slave attached > usb interface runtime suspend

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Alan Stern
On Wed, 10 Aug 2016, Baolin Wang wrote: > Considering strict power management for mobile device, we should also power > off the usb controller if there are no slaves attached even though it is usb > host function, but it will meet usb device resume failure in below situation. > > Suppose that no

Re: [PATCH v2] usb: core: Add runtime resume checking in choose_wakeup()

2016-08-10 Thread Alan Stern
On Wed, 10 Aug 2016, Baolin Wang wrote: > Considering strict power management for mobile device, we should also power > off the usb controller if there are no slaves attached even though it is usb > host function, but it will meet usb device resume failure in below situation. > > Suppose that no