| From adee40b265d7568296e218f079f478197ffa15bf Mon Sep 17 00:00:00 2001 |
| From: Magnus Lilja <lilja.magnus@gmail.com> |
| Date: Wed, 21 Dec 2016 22:13:58 +0100 |
| Subject: dmaengine: ipu: Make sure the interrupt routine checks all interrupts. |
| |
| From: Magnus Lilja <lilja.magnus@gmail.com> |
| |
| commit adee40b265d7568296e218f079f478197ffa15bf upstream. |
| |
| Commit 3d8cc00073d6 ("dmaengine: ipu: Consolidate duplicated irq handlers") |
| consolidated the two interrupts routines into one, but the remaining |
| interrupt routine only checks the status of the error interrupts, not the |
| normal interrupts. |
| |
| This patch fixes that problem (tested on i.MX31 PDK board). |
| |
| Fixes: 3d8cc00073d6 ("dmaengine: ipu: Consolidate duplicated irq handlers") |
| Cc: Vinod Koul <vinod.koul@intel.com> |
| Signed-off-by: Magnus Lilja <lilja.magnus@gmail.com> |
| Signed-off-by: Vinod Koul <vinod.koul@intel.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/dma/ipu/ipu_irq.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/dma/ipu/ipu_irq.c |
| +++ b/drivers/dma/ipu/ipu_irq.c |
| @@ -272,7 +272,7 @@ static void ipu_irq_handler(struct irq_d |
| u32 status; |
| int i, line; |
| |
| - for (i = IPU_IRQ_NR_FN_BANKS; i < IPU_IRQ_NR_BANKS; i++) { |
| + for (i = 0; i < IPU_IRQ_NR_BANKS; i++) { |
| struct ipu_irq_bank *bank = irq_bank + i; |
| |
| raw_spin_lock(&bank_lock); |