|
Takashi Iwai |
4af6b8 |
From 338db6ec41cad7b3f3846ef91e58ae563993bbd6 Mon Sep 17 00:00:00 2001
|
|
Takashi Iwai |
4af6b8 |
From: Marc Zyngier <maz@kernel.org>
|
|
Takashi Iwai |
4af6b8 |
Date: Wed, 22 Sep 2021 14:19:41 +0100
|
|
Takashi Iwai |
4af6b8 |
Subject: [PATCH] irqchip/armada-370-xp: Fix ack/eoi breakage
|
|
Takashi Iwai |
4af6b8 |
Git-commit: 2a7313dc81e88adc7bb09d0f056985fa8afc2b89
|
|
Takashi Iwai |
4af6b8 |
Patch-mainline: v5.15-rc3
|
|
Takashi Iwai |
4af6b8 |
References: stable-5.14.9
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
commit 2a7313dc81e88adc7bb09d0f056985fa8afc2b89 upstream.
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
When converting the driver to using handle_percpu_devid_irq,
|
|
Takashi Iwai |
4af6b8 |
we forgot to repaint the irq_eoi() callback into irq_ack(),
|
|
Takashi Iwai |
4af6b8 |
as handle_percpu_devid_fasteoi_ipi() was actually using EOI
|
|
Takashi Iwai |
4af6b8 |
really early in the handling. Yes this was a stupid idea.
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
Fix this by using the HW ack method as irq_ack().
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
Fixes: e52e73b7e9f7 ("irqchip/armada-370-xp: Make IPIs use handle_percpu_devid_irq()")
|
|
Takashi Iwai |
4af6b8 |
Reported-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
|
|
Takashi Iwai |
4af6b8 |
Tested-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
|
|
Takashi Iwai |
4af6b8 |
Signed-off-by: Marc Zyngier <maz@kernel.org>
|
|
Takashi Iwai |
4af6b8 |
Cc: Valentin Schneider <valentin.schneider@arm.com>
|
|
Takashi Iwai |
4af6b8 |
Cc: stable@vger.kernel.org
|
|
Takashi Iwai |
4af6b8 |
Link: https://lore.kernel.org/r/87tuiexq5f.fsf@pengutronix.de
|
|
Takashi Iwai |
4af6b8 |
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
|
Takashi Iwai |
4af6b8 |
Acked-by: Takashi Iwai <tiwai@suse.de>
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
---
|
|
Takashi Iwai |
4af6b8 |
drivers/irqchip/irq-armada-370-xp.c | 4 ++--
|
|
Takashi Iwai |
4af6b8 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
diff --git a/drivers/irqchip/irq-armada-370-xp.c b/drivers/irqchip/irq-armada-370-xp.c
|
|
Takashi Iwai |
4af6b8 |
index 7557ab551295..53e0fb0562c1 100644
|
|
Takashi Iwai |
4af6b8 |
--- a/drivers/irqchip/irq-armada-370-xp.c
|
|
Takashi Iwai |
4af6b8 |
+++ b/drivers/irqchip/irq-armada-370-xp.c
|
|
Takashi Iwai |
4af6b8 |
@@ -359,16 +359,16 @@ static void armada_370_xp_ipi_send_mask(struct irq_data *d,
|
|
Takashi Iwai |
4af6b8 |
ARMADA_370_XP_SW_TRIG_INT_OFFS);
|
|
Takashi Iwai |
4af6b8 |
}
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
-static void armada_370_xp_ipi_eoi(struct irq_data *d)
|
|
Takashi Iwai |
4af6b8 |
+static void armada_370_xp_ipi_ack(struct irq_data *d)
|
|
Takashi Iwai |
4af6b8 |
{
|
|
Takashi Iwai |
4af6b8 |
writel(~BIT(d->hwirq), per_cpu_int_base + ARMADA_370_XP_IN_DRBEL_CAUSE_OFFS);
|
|
Takashi Iwai |
4af6b8 |
}
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
static struct irq_chip ipi_irqchip = {
|
|
Takashi Iwai |
4af6b8 |
.name = "IPI",
|
|
Takashi Iwai |
4af6b8 |
+ .irq_ack = armada_370_xp_ipi_ack,
|
|
Takashi Iwai |
4af6b8 |
.irq_mask = armada_370_xp_ipi_mask,
|
|
Takashi Iwai |
4af6b8 |
.irq_unmask = armada_370_xp_ipi_unmask,
|
|
Takashi Iwai |
4af6b8 |
- .irq_eoi = armada_370_xp_ipi_eoi,
|
|
Takashi Iwai |
4af6b8 |
.ipi_send_mask = armada_370_xp_ipi_send_mask,
|
|
Takashi Iwai |
4af6b8 |
};
|
|
Takashi Iwai |
4af6b8 |
|
|
Takashi Iwai |
4af6b8 |
--
|
|
Takashi Iwai |
4af6b8 |
2.26.2
|
|
Takashi Iwai |
4af6b8 |
|