As you are already in ISR, it is unnecessary to call spin_lock_irqsave.

Signed-off-by: jun qian <hangdia...@163.com>
Cc: Barry song <21cn...@gmail.com>
---
 drivers/dma/k3dma.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/dma/k3dma.c b/drivers/dma/k3dma.c
index 6bfa217ed6d0..30052258ffd5 100644
--- a/drivers/dma/k3dma.c
+++ b/drivers/dma/k3dma.c
@@ -216,22 +216,21 @@ static irqreturn_t k3_dma_int_handler(int irq, void 
*dev_id)
                i = __ffs(stat);
                stat &= ~BIT(i);
                if (likely(tc1 & BIT(i)) || (tc2 & BIT(i))) {
-                       unsigned long flags;
 
                        p = &d->phy[i];
                        c = p->vchan;
                        if (c && (tc1 & BIT(i))) {
-                               spin_lock_irqsave(&c->vc.lock, flags);
+                               spin_lock(&c->vc.lock);
                                vchan_cookie_complete(&p->ds_run->vd);
                                p->ds_done = p->ds_run;
                                p->ds_run = NULL;
-                               spin_unlock_irqrestore(&c->vc.lock, flags);
+                               spin_unlock(&c->vc.lock);
                        }
                        if (c && (tc2 & BIT(i))) {
-                               spin_lock_irqsave(&c->vc.lock, flags);
+                               spin_lock(&c->vc.lock);
                                if (p->ds_run != NULL)
                                        vchan_cyclic_callback(&p->ds_run->vd);
-                               spin_unlock_irqrestore(&c->vc.lock, flags);
+                               spin_unlock(&c->vc.lock);
                        }
                        irq_chan |= BIT(i);
                }
-- 
2.17.1



Reply via email to