Commit 280e7f90 authored by Barry Song's avatar Barry Song Committed by Vinod Koul

dmaengine: milbeaut-xdmac: remove redundant irqsave and irqrestore in hardIRQ

Running in hardIRQ, disabling IRQ is redundant since hardIRQ has disabled
IRQ. This patch removes the irqsave and irqstore to save some instruction
cycles.
Signed-off-by: default avatarBarry Song <song.bao.hua@hisilicon.com>
Link: https://lore.kernel.org/r/20201027215252.25820-6-song.bao.hua@hisilicon.comSigned-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 654115e3
...@@ -160,10 +160,9 @@ static irqreturn_t milbeaut_xdmac_interrupt(int irq, void *dev_id) ...@@ -160,10 +160,9 @@ static irqreturn_t milbeaut_xdmac_interrupt(int irq, void *dev_id)
{ {
struct milbeaut_xdmac_chan *mc = dev_id; struct milbeaut_xdmac_chan *mc = dev_id;
struct milbeaut_xdmac_desc *md; struct milbeaut_xdmac_desc *md;
unsigned long flags;
u32 val; u32 val;
spin_lock_irqsave(&mc->vc.lock, flags); spin_lock(&mc->vc.lock);
/* Ack and Stop */ /* Ack and Stop */
val = FIELD_PREP(M10V_XDDSD_IS_MASK, 0x0); val = FIELD_PREP(M10V_XDDSD_IS_MASK, 0x0);
...@@ -177,7 +176,7 @@ static irqreturn_t milbeaut_xdmac_interrupt(int irq, void *dev_id) ...@@ -177,7 +176,7 @@ static irqreturn_t milbeaut_xdmac_interrupt(int irq, void *dev_id)
milbeaut_xdmac_start(mc); milbeaut_xdmac_start(mc);
out: out:
spin_unlock_irqrestore(&mc->vc.lock, flags); spin_unlock(&mc->vc.lock);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
......
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