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

2017-01-17 Thread Bin Liu
On Tue, Jan 17, 2017 at 09:39:23AM -0800, Tony Lindgren wrote: > * Bin Liu [170117 08:49]: > > On Tue, Jan 17, 2017 at 08:31:03AM -0800, Tony Lindgren wrote: > > > * Bin Liu [170117 08:22]: > > > > On Tue, Jan 17, 2017 at 08:11:39AM -0800, Tony Lindgren wrote: > > > >

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

2017-01-17 Thread Tony Lindgren
* Bin Liu [170117 11:47]: > On Tue, Jan 17, 2017 at 09:39:23AM -0800, Tony Lindgren wrote: > > * Bin Liu [170117 08:49]: > > > On Tue, Jan 17, 2017 at 08:31:03AM -0800, Tony Lindgren wrote: > > > > * Bin Liu [170117 08:22]: > > > > > On Tue, Jan 17,

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

2017-01-17 Thread Tony Lindgren
* Bin Liu [170117 08:49]: > On Tue, Jan 17, 2017 at 08:31:03AM -0800, Tony Lindgren wrote: > > * Bin Liu [170117 08:22]: > > > On Tue, Jan 17, 2017 at 08:11:39AM -0800, Tony Lindgren wrote: > > > > * Bin Liu [170117 05:00]: > > > > > On Mon, Jan 16,

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

2017-01-17 Thread Bin Liu
On Tue, Jan 17, 2017 at 08:31:03AM -0800, Tony Lindgren wrote: > * Bin Liu [170117 08:22]: > > On Tue, Jan 17, 2017 at 08:11:39AM -0800, Tony Lindgren wrote: > > > * Bin Liu [170117 05:00]: > > > > On Mon, Jan 16, 2017 at 03:54:29PM -0800, Tony Lindgren wrote: > > > >

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

2017-01-17 Thread Tony Lindgren
* Bin Liu [170117 08:22]: > On Tue, Jan 17, 2017 at 08:11:39AM -0800, Tony Lindgren wrote: > > * Bin Liu [170117 05:00]: > > > On Mon, Jan 16, 2017 at 03:54:29PM -0800, Tony Lindgren wrote: > > > > Anyways, for the -rc series oops, we can just leave out the WARN_ON >

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

2017-01-17 Thread Bin Liu
On Tue, Jan 17, 2017 at 08:11:39AM -0800, Tony Lindgren wrote: > * Bin Liu [170117 05:00]: > > On Mon, Jan 16, 2017 at 03:54:29PM -0800, Tony Lindgren wrote: > > > Anyways, for the -rc series oops, we can just leave out the WARN_ON > > > parts for now until

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

2017-01-17 Thread Tony Lindgren
* Bin Liu [170117 05:00]: > On Mon, Jan 16, 2017 at 03:54:29PM -0800, Tony Lindgren wrote: > > Anyways, for the -rc series oops, we can just leave out the WARN_ON > > parts for now until drivers/usb/musb/musb_cppi41.c is fixed too. > > Giving that cppi is a submodule inside the usb

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

2017-01-17 Thread Bin Liu
Tony, On Mon, Jan 16, 2017 at 03:54:29PM -0800, Tony Lindgren wrote: > * Tony Lindgren [170116 15:36]: > > * Tony Lindgren [170113 14:00]: > > > * Grygorii Strashko [170113 13:37]: > > > > > Simplified diff with fix on top of your

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

2017-01-16 Thread Tony Lindgren
* Tony Lindgren [170116 15:36]: > * Tony Lindgren [170113 14:00]: > > * Grygorii Strashko [170113 13:37]: > > > > Simplified diff with fix on top of your patch: > > > > > > > > diff --git a/drivers/dma/cppi41.c

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

2017-01-16 Thread Tony Lindgren
* Tony Lindgren [170113 14:00]: > * Grygorii Strashko [170113 13:37]: > > > Simplified diff with fix on top of your patch: > > > > > > diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c > > > index ce37a1a..9e9403a 100644 > > > ---

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

2017-01-13 Thread Tony Lindgren
* Grygorii Strashko [170113 13:37]: > > Simplified diff with fix on top of your patch: > > > > diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c > > index ce37a1a..9e9403a 100644 > > --- a/drivers/dma/cppi41.c > > +++ b/drivers/dma/cppi41.c > > @@ -319,12 +319,6

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

2017-01-13 Thread Grygorii Strashko
On 01/13/2017 01:57 PM, Grygorii Strashko wrote: > > > On 01/13/2017 01:53 PM, Grygorii Strashko wrote: >> >> >> On 01/13/2017 01:01 PM, Tony Lindgren wrote: >>> * Grygorii Strashko [170113 10:37]: On 01/13/2017 12:01 PM, Tony Lindgren wrote: >

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

2017-01-13 Thread Grygorii Strashko
On 01/13/2017 01:53 PM, Grygorii Strashko wrote: > > > On 01/13/2017 01:01 PM, Tony Lindgren wrote: >> * Grygorii Strashko [170113 10:37]: >>> >>> >>> On 01/13/2017 12:01 PM, Tony Lindgren wrote: Commit fdea2d09b997 ("dmaengine: cppi41: Add basic PM runtime

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

2017-01-13 Thread Grygorii Strashko
On 01/13/2017 01:01 PM, Tony Lindgren wrote: > * Grygorii Strashko [170113 10:37]: >> >> >> On 01/13/2017 12:01 PM, Tony Lindgren wrote: >>> Commit fdea2d09b997 ("dmaengine: cppi41: Add basic PM runtime support") >>> together with recent MUSB changes allowed USB and

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

2017-01-13 Thread Tony Lindgren
* Grygorii Strashko [170113 10:37]: > > > On 01/13/2017 12:01 PM, 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

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

2017-01-13 Thread Grygorii Strashko
On 01/13/2017 12:01 PM, 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. > > Looks like

[PATCHv2] dmaengine: cppi41: Fix oops in cppi41_runtime_resume

2017-01-13 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