Skip to contentSkip to conversation

dmaengine: ipu: Make sure the interrupt routine checks all interrupts.

PATCH New Magnus Lilja 2 participants 2 messages Dec 21, 2016 – Jan 2, 2017

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 dmaengine: ipu: Make sure the interrupt routine checks all interrupts.

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 <[email protected]>
Cc <[email protected]> # 4.1.x
---
 drivers/dma/ipu/ipu_irq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/dma/ipu/ipu_irq.c +1 -1
index dd184b5..2846278 100644
@@ -272,7 +272,7 @@ static void ipu_irq_handler(struct irq_desc *desc)
 	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);
Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/1 dmaengine: ipu: Make sure the interrupt routine checks all interrupts.

Quoted 6 lines · Magnus Lilja

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).

Applied thnx

Signature · 1 line
~Vinod