[tip: irq/urgent] irqchip/sg2042-msi: Add missing chip flags

From: tip-bot2 for Inochi Amaoto
Date: Mon Apr 07 2025 - 03:39:44 EST


The following commit has been merged into the irq/urgent branch of tip:

Commit-ID: 305825d09b15586d2e4311e0c12f10f2a0c18ac5
Gitweb: https://git.kernel.org/tip/305825d09b15586d2e4311e0c12f10f2a0c18ac5
Author: Inochi Amaoto <inochiama@xxxxxxxxx>
AuthorDate: Sat, 05 Apr 2025 13:56:24 +08:00
Committer: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
CommitterDate: Mon, 07 Apr 2025 09:23:55 +02:00

irqchip/sg2042-msi: Add missing chip flags

The sg2042-msi driver uses the fallback callbacks set by
msi_lib_init_dev_msi_info(). commit 1c000dcaad2b ("irqchip/irq-msi-lib:
Optionally set default irq_eoi()/irq_ack()") changed the behavior of the
fallback mechanism by making it opt-in.

The sg2042-msi was not fixed up for this, which causes a NULL pointer
dereference due to the missing irq_ack() callback.

Add the missing chip flag to msi_parent_ops.

Fixes: c66741549424 ("irqchip: Add the Sophgo SG2042 MSI interrupt controller")
Signed-off-by: Inochi Amaoto <inochiama@xxxxxxxxx>
Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Link: https://lore.kernel.org/all/20250217085657.789309-3-apatel@xxxxxxxxxxxxxxxx
Link: https://lore.kernel.org/all/20250405055625.1530180-1-inochiama@xxxxxxxxx

---
drivers/irqchip/irq-sg2042-msi.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/irqchip/irq-sg2042-msi.c b/drivers/irqchip/irq-sg2042-msi.c
index ee682e8..375b55a 100644
--- a/drivers/irqchip/irq-sg2042-msi.c
+++ b/drivers/irqchip/irq-sg2042-msi.c
@@ -151,6 +151,7 @@ static const struct irq_domain_ops sg2042_msi_middle_domain_ops = {
static const struct msi_parent_ops sg2042_msi_parent_ops = {
.required_flags = SG2042_MSI_FLAGS_REQUIRED,
.supported_flags = SG2042_MSI_FLAGS_SUPPORTED,
+ .chip_flags = MSI_CHIP_FLAG_SET_ACK,
.bus_select_mask = MATCH_PCI_MSI,
.bus_select_token = DOMAIN_BUS_NEXUS,
.prefix = "SG2042-",