From: Andy Shevchenko <andriy.shevche...@linux.intel.com>

There is no need to disable interrupts in the IRQ handler. The driver
guarantess that at one time only one descriptor is active, besides the fact
that each call to the same channel will be serialized in idma64_chan_irq()
handler anyway.

Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
Signed-off-by: Vinod Koul <vinod.k...@intel.com>
(cherry picked from commit 37580559f314bfba0c8bdae002bc5c10088ac457)
Signed-off-by: Tan Jui Nee <jui.nee....@intel.com>
---
 drivers/dma/idma64.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/drivers/dma/idma64.c b/drivers/dma/idma64.c
index 7d56b47..6bba090 100644
--- a/drivers/dma/idma64.c
+++ b/drivers/dma/idma64.c
@@ -178,20 +178,12 @@ static irqreturn_t idma64_irq(int irq, void *dev)
        if (!status)
                return IRQ_NONE;
 
-       /* Disable interrupts */
-       channel_clear_bit(idma64, MASK(XFER), idma64->all_chan_mask);
-       channel_clear_bit(idma64, MASK(ERROR), idma64->all_chan_mask);
-
        status_xfer = dma_readl(idma64, RAW(XFER));
        status_err = dma_readl(idma64, RAW(ERROR));
 
        for (i = 0; i < idma64->dma.chancnt; i++)
                idma64_chan_irq(idma64, i, status_err, status_xfer);
 
-       /* Re-enable interrupts */
-       channel_set_bit(idma64, MASK(XFER), idma64->all_chan_mask);
-       channel_set_bit(idma64, MASK(ERROR), idma64->all_chan_mask);
-
        return IRQ_HANDLED;
 }
 
-- 
1.9.1

-- 
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to