Messages in this thread | | | Subject | Re: [PATCH 2/2] dma: dw_dmac: clear suspend during termination | From | Andy Shevchenko <> | Date | Mon, 17 Dec 2012 17:26:57 +0200 |
| |
On Mon, 2012-12-17 at 20:42 +0530, Viresh Kumar wrote: > On 17 December 2012 19:37, Andy Shevchenko > <andriy.shevchenko@linux.intel.com> wrote: > > From: Heikki Krogerus <heikki.krogerus@linux.intel.com> > > > > Clear DWC_CFGL_CH_SUSP bit during termination if the channel > > was paused. > > > > Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Insufficient commit log :( > You don't have to say here, what you are doing, but why you are doing it.
Fighting against bug. The workflow in our case is to a) pause the dma, b) do something in the slave driver, c) terminate all transfers, d) submit new transfer. But d) doesn't happen because the channel is still paused.
> Please mention the reason here first, then repost the series after getting my > comments
Okay.
-- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy
| |