summaryrefslogtreecommitdiff
path: root/drivers/dma
diff options
context:
space:
mode:
authorZhaoxiong Yuan <yuanzhx326@gmail.com>2018-09-07 22:02:10 +0300
committerVinod Koul <vkoul@kernel.org>2018-09-11 10:43:40 +0300
commit7645d26f3b35c2bd6e0e1d3867636e5bdad97b1b (patch)
treecfbc6641e07d5f14f3d56cf517643d37c1e49ed3 /drivers/dma
parent5b394b2ddf0347bef56e50c69a58773c94343ff3 (diff)
downloadlinux-7645d26f3b35c2bd6e0e1d3867636e5bdad97b1b.tar.xz
dmaengine: idma64: replace spin_lock_irqsave with spin_lock
idma64_chan_irq() is invoked in hardirq handle function, it is unnecessary to call spin_lock_irqsave. Signed-off-by: Zhaoxiong Yuan <yuanzhx326@gmail.com> Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma')
-rw-r--r--drivers/dma/idma64.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/dma/idma64.c b/drivers/dma/idma64.c
index 1fbf9cb9b742..5b9c1566ad73 100644
--- a/drivers/dma/idma64.c
+++ b/drivers/dma/idma64.c
@@ -142,9 +142,8 @@ static void idma64_chan_irq(struct idma64 *idma64, unsigned short c,
{
struct idma64_chan *idma64c = &idma64->chan[c];
struct idma64_desc *desc;
- unsigned long flags;
- spin_lock_irqsave(&idma64c->vchan.lock, flags);
+ spin_lock(&idma64c->vchan.lock);
desc = idma64c->desc;
if (desc) {
if (status_err & (1 << c)) {
@@ -161,7 +160,7 @@ static void idma64_chan_irq(struct idma64 *idma64, unsigned short c,
if (idma64c->desc == NULL || desc->status == DMA_ERROR)
idma64_stop_transfer(idma64c);
}
- spin_unlock_irqrestore(&idma64c->vchan.lock, flags);
+ spin_unlock(&idma64c->vchan.lock);
}
static irqreturn_t idma64_irq(int irq, void *dev)