Commit 654115e3 authored by Barry Song's avatar Barry Song Committed by Vinod Koul

dmaengine: tegra210-adma: 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>
Acked-by: default avatarJon Hunter <jonathanh@nvidia.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Link: https://lore.kernel.org/r/20201027215252.25820-5-song.bao.hua@hisilicon.comSigned-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 302b3b38
...@@ -408,19 +408,18 @@ static irqreturn_t tegra_adma_isr(int irq, void *dev_id) ...@@ -408,19 +408,18 @@ static irqreturn_t tegra_adma_isr(int irq, void *dev_id)
{ {
struct tegra_adma_chan *tdc = dev_id; struct tegra_adma_chan *tdc = dev_id;
unsigned long status; unsigned long status;
unsigned long flags;
spin_lock_irqsave(&tdc->vc.lock, flags); spin_lock(&tdc->vc.lock);
status = tegra_adma_irq_clear(tdc); status = tegra_adma_irq_clear(tdc);
if (status == 0 || !tdc->desc) { if (status == 0 || !tdc->desc) {
spin_unlock_irqrestore(&tdc->vc.lock, flags); spin_unlock(&tdc->vc.lock);
return IRQ_NONE; return IRQ_NONE;
} }
vchan_cyclic_callback(&tdc->desc->vd); vchan_cyclic_callback(&tdc->desc->vd);
spin_unlock_irqrestore(&tdc->vc.lock, flags); spin_unlock(&tdc->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