Commit 55f8b254 authored by Barry Song's avatar Barry Song Committed by Vinod Koul

dmaengine: zx: remove redundant irqsave in hardIRQ

Running in hardIRQ context, disabling IRQ is redundant.

Cc: Jun Nie <jun.nie@linaro.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: Vinod Koul <vkoul@kernel.org>
Signed-off-by: default avatarBarry Song <song.bao.hua@hisilicon.com>
Link: https://lore.kernel.org/r/20200912094036.32112-1-song.bao.hua@hisilicon.comSigned-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent e3a52158
...@@ -285,9 +285,7 @@ static irqreturn_t zx_dma_int_handler(int irq, void *dev_id) ...@@ -285,9 +285,7 @@ static irqreturn_t zx_dma_int_handler(int irq, void *dev_id)
p = &d->phy[i]; p = &d->phy[i];
c = p->vchan; c = p->vchan;
if (c) { if (c) {
unsigned long flags; spin_lock(&c->vc.lock);
spin_lock_irqsave(&c->vc.lock, flags);
if (c->cyclic) { if (c->cyclic) {
vchan_cyclic_callback(&p->ds_run->vd); vchan_cyclic_callback(&p->ds_run->vd);
} else { } else {
...@@ -295,7 +293,7 @@ static irqreturn_t zx_dma_int_handler(int irq, void *dev_id) ...@@ -295,7 +293,7 @@ static irqreturn_t zx_dma_int_handler(int irq, void *dev_id)
p->ds_done = p->ds_run; p->ds_done = p->ds_run;
task = 1; task = 1;
} }
spin_unlock_irqrestore(&c->vc.lock, flags); spin_unlock(&c->vc.lock);
irq_chan |= BIT(i); irq_chan |= BIT(i);
} }
} }
......
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