diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2013-05-27 16:14:36 +0400 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2013-08-05 08:02:24 +0400 |
| commit | 4aa9fe0a1f307df4635822c3019bb196aa30b938 (patch) | |
| tree | 55f70aeba36ecf6f0aa6de630235f79587f19cd2 | |
| parent | 9b0b0bdc6a3ba70b73ea2fdd6c09ee46b34d3f94 (diff) | |
| download | linux-4aa9fe0a1f307df4635822c3019bb196aa30b938.tar.xz | |
mmp_pdma: remove useless use of lock
Accordingly to dma_cookie_status() description locking is not required.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
| -rw-r--r-- | drivers/dma/mmp_pdma.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma/mmp_pdma.c index c26699f9c4df..ad4b0a7167f6 100644 --- a/drivers/dma/mmp_pdma.c +++ b/drivers/dma/mmp_pdma.c @@ -632,15 +632,7 @@ static int mmp_pdma_control(struct dma_chan *dchan, enum dma_ctrl_cmd cmd, static enum dma_status mmp_pdma_tx_status(struct dma_chan *dchan, dma_cookie_t cookie, struct dma_tx_state *txstate) { - struct mmp_pdma_chan *chan = to_mmp_pdma_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); } /** |