diff options
author | 2016-08-31 11:10:28 -0400 | |
---|---|---|
committer | 2016-08-31 21:27:32 +0530 | |
commit | 55c370e5198e8cf28b1529299e9c1bfe237c9c1e (patch) | |
tree | 8097b01a52958ab06e950e69eae052d2f8859a7a /drivers/dma/qcom | |
parent | dmaengine: qcom_hidma: release the descriptor before the callback (diff) | |
download | linux-55c370e5198e8cf28b1529299e9c1bfe237c9c1e.tar.gz linux-55c370e5198e8cf28b1529299e9c1bfe237c9c1e.tar.bz2 linux-55c370e5198e8cf28b1529299e9c1bfe237c9c1e.zip |
dmaengine: qcom_hidma: report transfer errors with new interface
Pass the DMA errors to the client by passing a result argument. The HW only
supports a generic error when something goes wrong. That's why, using
DMA_TRANS_ABORTED all the time.
Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/qcom')
-rw-r--r-- | drivers/dma/qcom/hidma.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/dma/qcom/hidma.c b/drivers/dma/qcom/hidma.c index b8493bafdb3f..ea24863794b9 100644 --- a/drivers/dma/qcom/hidma.c +++ b/drivers/dma/qcom/hidma.c @@ -126,6 +126,7 @@ static void hidma_process_completed(struct hidma_chan *mchan) list_for_each_entry_safe(mdesc, next, &list, node) { enum dma_status llstat; struct dmaengine_desc_callback cb; + struct dmaengine_result result; desc = &mdesc->desc; @@ -141,10 +142,15 @@ static void hidma_process_completed(struct hidma_chan *mchan) spin_lock_irqsave(&mchan->lock, irqflags); list_move(&mdesc->node, &mchan->free); - spin_unlock_irqrestore(&mchan->lock, irqflags); if (llstat == DMA_COMPLETE) - dmaengine_desc_callback_invoke(&cb, NULL); + result.result = DMA_TRANS_NOERROR; + else + result.result = DMA_TRANS_ABORTED; + + spin_unlock_irqrestore(&mchan->lock, irqflags); + + dmaengine_desc_callback_invoke(&cb, &result); } } |