Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Bin Liu [170118 13:14]: > On Wed, Jan 18, 2017 at 12:48:59PM -0800, Tony Lindgren wrote: > > And here's a fix for the error -71 regression. > > > > Bin, can you review and test your earlier case mentioned in > > commit 9298b4aad37e ("usb: musb: fix device hotplug behind hub")

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Bin Liu
On Wed, Jan 18, 2017 at 12:48:59PM -0800, Tony Lindgren wrote: > * Bin Liu [170118 10:49]: > > On Wed, Jan 18, 2017 at 10:44:32AM -0800, Tony Lindgren wrote: > > > * Tony Lindgren [170118 10:15]: > > > > * Grygorii Strashko [170118

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Grygorii Strashko
On 01/18/2017 12:15 PM, Tony Lindgren wrote: > * Grygorii Strashko [170118 10:05]: >> >> >> On 01/18/2017 10:53 AM, Tony Lindgren wrote: >>> Hi, >>> >>> * Bin Liu [170118 06:26]: With this v3, I still get -71 error when a device is plugged to a hub

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Bin Liu
On Wed, Jan 18, 2017 at 10:44:32AM -0800, Tony Lindgren wrote: > * Tony Lindgren [170118 10:15]: > > * Grygorii Strashko [170118 10:05]: > > > > > > > > > On 01/18/2017 10:53 AM, Tony Lindgren wrote: > > > > Hi, > > > > > > > > * Bin Liu

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Bin Liu [170118 10:49]: > On Wed, Jan 18, 2017 at 10:44:32AM -0800, Tony Lindgren wrote: > > * Tony Lindgren [170118 10:15]: > > > * Grygorii Strashko [170118 10:05]: > > > > > > > > > > > > On 01/18/2017 10:53 AM, Tony Lindgren

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Grygorii Strashko [170118 11:18]: > Just thinking, may be cppi41 should not be platform device at all > and it might be reasonable to have it as lib with > cppi41_init()/cppi41_remove(), > so musb SoC glue layer will initialize it, because it provides services to >

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Grygorii Strashko [170118 10:05]: > > > On 01/18/2017 10:53 AM, Tony Lindgren wrote: > > Hi, > > > > * Bin Liu [170118 06:26]: > >> With this v3, I still get -71 error when a device is plugged to a hub to > >> musb. It doesn't happen though if the

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Bin Liu
On Wed, Jan 18, 2017 at 08:53:09AM -0800, Tony Lindgren wrote: > Hi, > > * Bin Liu [170118 06:26]: > > With this v3, I still get -71 error when a device is plugged to a hub to > > musb. It doesn't happen though if the device is already plugged to the hub > > before attaching the

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Bin Liu [170118 10:43]: > On Wed, Jan 18, 2017 at 08:53:09AM -0800, Tony Lindgren wrote: > > Hi, > > > > * Bin Liu [170118 06:26]: > > > With this v3, I still get -71 error when a device is plugged to a hub to > > > musb. It doesn't happen though if the device is

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
* Tony Lindgren [170118 10:15]: > * Grygorii Strashko [170118 10:05]: > > > > > > On 01/18/2017 10:53 AM, Tony Lindgren wrote: > > > Hi, > > > > > > * Bin Liu [170118 06:26]: > > >> With this v3, I still get -71 error when a device is

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Grygorii Strashko
On 01/18/2017 10:53 AM, Tony Lindgren wrote: > Hi, > > * Bin Liu [170118 06:26]: >> With this v3, I still get -71 error when a device is plugged to a hub to >> musb. It doesn't happen though if the device is already plugged to the hub >> before attaching the hub to musb. >> >> [

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Tony Lindgren
Hi, * Bin Liu [170118 06:26]: > With this v3, I still get -71 error when a device is plugged to a hub to > musb. It doesn't happen though if the device is already plugged to the hub > before attaching the hub to musb. > > [ 177.579300] usb 1-1: reset high-speed USB device number

Re: [PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-18 Thread Bin Liu
On Tue, Jan 17, 2017 at 09:55:24AM -0800, Tony Lindgren wrote: > Commit fdea2d09b997 ("dmaengine: cppi41: Add basic PM runtime support") > together with recent MUSB changes allowed USB and DMA on BeagleBone to idle > when no cable is connected. But looks like few corner case issues still > remain.

[PATCHv3] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-17 Thread Tony Lindgren
Commit fdea2d09b997 ("dmaengine: cppi41: Add basic PM runtime support") together with recent MUSB changes allowed USB and DMA on BeagleBone to idle when no cable is connected. But looks like few corner case issues still remain. Looks like just by re-plugging USB cable about ten or so times on