[PATCH v2 2/3] genirq: use irqd_get_parent_data() helper in __irq_get_irqchip_state()
From: Kemeng Shi
Date: Wed Jul 01 2026 - 23:54:32 EST
__irq_get_irqchip_state() manually dereferences parent_data under
CONFIG_IRQ_DOMAIN_HIERARCHY and sets it to NULL otherwise. This
pattern is already covered by irqd_get_parent_data(), which handles
the conditional internally. Replace the open-coded logic with the
helper to reduce duplication and improve readability.
Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx>
---
kernel/irq/manage.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index 2e8072437826..c03844c82a97 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -2692,11 +2692,7 @@ static int __irq_get_irqchip_state(struct irq_data *data, enum irqchip_irq_state
return -ENODEV;
if (chip->irq_get_irqchip_state)
break;
-#ifdef CONFIG_IRQ_DOMAIN_HIERARCHY
- data = data->parent_data;
-#else
- data = NULL;
-#endif
+ data = irqd_get_parent_data(data);
} while (data);
if (data)
--
2.36.1