[PATCH 05/12] fsldma: remove useless use of lock

Andy Shevchenko andriy.shevchenko at linux.intel.com
Mon May 27 22:14:35 EST 2013


Accordingly to dma_cookie_status() description locking is not required.

Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
Cc: Li Yang <leoli at freescale.com>
Cc: Zhang Wei <zw at zh-kernel.org>
Cc: linuxppc-dev at lists.ozlabs.org
---
 drivers/dma/fsldma.c | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c
index 49e8fbd..b3f3e90 100644
--- a/drivers/dma/fsldma.c
+++ b/drivers/dma/fsldma.c
@@ -979,15 +979,7 @@ static enum dma_status fsl_tx_status(struct dma_chan *dchan,
 					dma_cookie_t cookie,
 					struct dma_tx_state *txstate)
 {
-	struct fsldma_chan *chan = to_fsl_chan(dchan);
-	enum dma_status ret;
-	unsigned long flags;
-
-	spin_lock_irqsave(&chan->desc_lock, flags);
-	ret = dma_cookie_status(dchan, cookie, txstate);
-	spin_unlock_irqrestore(&chan->desc_lock, flags);
-
-	return ret;
+	return dma_cookie_status(dchan, cookie, txstate);
 }
 
 /*----------------------------------------------------------------------------*/
-- 
1.8.2.rc0.22.gb3600c3



More information about the Linuxppc-dev mailing list