Commit 2b4f130e authored by Sascha Hauer's avatar Sascha Hauer Committed by Vinod Koul

dma: imx-sdma: start transfer in issue_pending

The DMA engine API requires that transfers are started in issue_pending
instead of tx_submit. Fix this.
Signed-off-by: default avatarSascha Hauer <s.hauer@pengutronix.de>
[corrected change log to DMA engine API insteadof DMA API]
Signed-off-by: default avatarVinod Koul <vinod.koul@linux.intel.com>
parent 5b316876
...@@ -838,15 +838,12 @@ static dma_cookie_t sdma_tx_submit(struct dma_async_tx_descriptor *tx) ...@@ -838,15 +838,12 @@ static dma_cookie_t sdma_tx_submit(struct dma_async_tx_descriptor *tx)
{ {
unsigned long flags; unsigned long flags;
struct sdma_channel *sdmac = to_sdma_chan(tx->chan); struct sdma_channel *sdmac = to_sdma_chan(tx->chan);
struct sdma_engine *sdma = sdmac->sdma;
dma_cookie_t cookie; dma_cookie_t cookie;
spin_lock_irqsave(&sdmac->lock, flags); spin_lock_irqsave(&sdmac->lock, flags);
cookie = sdma_assign_cookie(sdmac); cookie = sdma_assign_cookie(sdmac);
sdma_enable_channel(sdma, sdmac->channel);
spin_unlock_irqrestore(&sdmac->lock, flags); spin_unlock_irqrestore(&sdmac->lock, flags);
return cookie; return cookie;
...@@ -1135,9 +1132,11 @@ static enum dma_status sdma_tx_status(struct dma_chan *chan, ...@@ -1135,9 +1132,11 @@ static enum dma_status sdma_tx_status(struct dma_chan *chan,
static void sdma_issue_pending(struct dma_chan *chan) static void sdma_issue_pending(struct dma_chan *chan)
{ {
/* struct sdma_channel *sdmac = to_sdma_chan(chan);
* Nothing to do. We only have a single descriptor struct sdma_engine *sdma = sdmac->sdma;
*/
if (sdmac->status == DMA_IN_PROGRESS)
sdma_enable_channel(sdma, sdmac->channel);
} }
#define SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V1 34 #define SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V1 34
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment