[PATCH 1/3] irqchip/renesas-rzg2l: Fix shared IRQ bit not cleared on free

Biju
2026-03-28T10:33:29Z

From: Biju Das <biju.das.jz@bp.renesas.com>

Calling irq_domain_free_irqs_common() internally calls
irq_domain_reset_irq_data(), which explicitly sets irq_data->hwirq
to 0. Consequently, irqd_to_hwirq(d) returns 0 when called after it.
Since 0 falls outside the valid shared IRQ ranges,
rzg2l_irqc_is_shared_and_get_irq_num() evaluates to false, completely
bypassing the test_and_clear_bit() operation. This leaves the bit set
in priv->used_irqs, causing future allocations to fail with -EBUSY.
Fix this by retrieving irq_data and caching hwirq before calling
irq_domain_free_irqs_common().

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
drivers/irqchip/irq-renesas-rzg2l.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/irqchip/irq-renesas-rzg2l.c b/drivers/irqchip/irq-renesas-rzg2l.c
index f5c4d7e0aec3..3cc1efd8d914 100644
--- a/drivers/irqchip/irq-renesas-rzg2l.c
+++ b/drivers/irqchip/irq-renesas-rzg2l.c
@@ -699,15 +699,14 @@ static int rzg2l_irqc_alloc(struct irq_domain *domain, unsigned int virq,

static void rzg2l_irqc_free(struct irq_domain *domain, unsigned int virq, unsigned int nr_irqs)
{
+ struct irq_data *d = irq_domain_get_irq_data(domain, virq);
struct rzg2l_irqc_priv *priv = domain->host_data;
+ irq_hw_number_t hwirq = irqd_to_hwirq(d);

irq_domain_free_irqs_common(domain, virq, nr_irqs);

- if (priv->info.shared_irq_cnt) {
- struct irq_data *d = irq_domain_get_irq_data(domain, virq);
-
- rzg2l_irqc_shared_irq_free(priv, irqd_to_hwirq(d));
- }
+ if (priv->info.shared_irq_cnt)
+ rzg2l_irqc_shared_irq_free(priv, hwirq);
}

static const struct irq_domain_ops rzg2l_irqc_domain_ops = {
--
2.43.0