diff options
| author | Geert Uytterhoeven <geert+renesas@glider.be> | 2025-11-19 14:19:11 +0100 |
|---|---|---|
| committer | Rob Herring (Arm) <robh@kernel.org> | 2025-11-19 15:52:27 -0600 |
| commit | 6ba51b7b34caac6a64efdb5ec49d1c0e45e6197b (patch) | |
| tree | b51a68779cb9cade572ca2a40a318f7da2feefe2 | |
| parent | d8c8a575f5aa7afc7cc7718b779c29ae5f7abc58 (diff) | |
of/irq: Handle explicit interrupt parent
If an interrupt controller is used as a proxy, it may have an
"interrupt-parent" property, but lack "interrupts" and
"interrupts-extended" properties. In that case, the "interrupt-parent"
property in the interrupt controller node is ignored, causing the
interrupt controller to fail to probe, and leading to system boot
failures or crashes.
Fix this by also considering an explicit "interrupt-parent" property in
the interrupt controller node itself.
Fixes: 1b1f04d8271e7ba7 ("of/irq: Ignore interrupt parent for nodes without interrupts")
Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
Closes: https://lore.kernel.org/20251118115037.1866871-1-m.szyprowski@samsung.com
Reported-by: Mark Brown <broonie@kernel.org>
Closes: https://lore.kernel.org/b037f67a-b241-4689-9914-57ff578c1454@sirena.org.uk
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Tested-by: Mark Brown <broonie@kernel.org>
Link: https://patch.msgid.link/e89669c9b3a4fbac4a972ffadcbe00fddb365472.1763557994.git.geert+renesas@glider.be
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
| -rw-r--r-- | drivers/of/irq.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/of/irq.c b/drivers/of/irq.c index 96db232b666a..2271110b5f7c 100644 --- a/drivers/of/irq.c +++ b/drivers/of/irq.c @@ -595,6 +595,8 @@ void __init of_irq_init(const struct of_device_id *matches) desc->interrupt_parent = of_parse_phandle(np, "interrupts-extended", 0); if (!desc->interrupt_parent && of_property_present(np, "interrupts")) desc->interrupt_parent = of_irq_find_parent(np); + else if (!desc->interrupt_parent) + desc->interrupt_parent = of_parse_phandle(np, "interrupt-parent", 0); if (desc->interrupt_parent == np) { of_node_put(desc->interrupt_parent); desc->interrupt_parent = NULL; |