Skip to content

Commit 2a7313d

Browse files
author
Marc Zyngier
committed
irqchip/armada-370-xp: Fix ack/eoi breakage
When converting the driver to using handle_percpu_devid_irq, we forgot to repaint the irq_eoi() callback into irq_ack(), as handle_percpu_devid_fasteoi_ipi() was actually using EOI really early in the handling. Yes this was a stupid idea. Fix this by using the HW ack method as irq_ack(). Fixes: e52e73b ("irqchip/armada-370-xp: Make IPIs use handle_percpu_devid_irq()") Reported-by: Steffen Trumtrar <[email protected]> Tested-by: Steffen Trumtrar <[email protected]> Signed-off-by: Marc Zyngier <[email protected]> Cc: Valentin Schneider <[email protected]> Cc: [email protected] Link: https://lore.kernel.org/r/[email protected]
1 parent 0ddc5e5 commit 2a7313d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/irqchip/irq-armada-370-xp.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -359,16 +359,16 @@ static void armada_370_xp_ipi_send_mask(struct irq_data *d,
359359
ARMADA_370_XP_SW_TRIG_INT_OFFS);
360360
}
361361

362-
static void armada_370_xp_ipi_eoi(struct irq_data *d)
362+
static void armada_370_xp_ipi_ack(struct irq_data *d)
363363
{
364364
writel(~BIT(d->hwirq), per_cpu_int_base + ARMADA_370_XP_IN_DRBEL_CAUSE_OFFS);
365365
}
366366

367367
static struct irq_chip ipi_irqchip = {
368368
.name = "IPI",
369+
.irq_ack = armada_370_xp_ipi_ack,
369370
.irq_mask = armada_370_xp_ipi_mask,
370371
.irq_unmask = armada_370_xp_ipi_unmask,
371-
.irq_eoi = armada_370_xp_ipi_eoi,
372372
.ipi_send_mask = armada_370_xp_ipi_send_mask,
373373
};
374374

0 commit comments

Comments
 (0)