irqchip/renesas-rzg2l: Prevent spurious interrupts when setting trigger type
authorBiju Das <biju.das.jz@bp.renesas.com>
Tue, 5 Mar 2024 18:39:21 +0000 (18:39 +0000)
committerThomas Gleixner <tglx@linutronix.de>
Mon, 18 Mar 2024 11:20:01 +0000 (12:20 +0100)
commit853a6030303f8a8fa54929b68e5665d9b21aa405
tree49f8b4d5bb01e51eb75f6874690656aedd64d04c
parentb4b5cd61a6fdd92ede0dc39f0850a182affd1323
irqchip/renesas-rzg2l: Prevent spurious interrupts when setting trigger type

RZ/G2L interrupt chips require that the interrupt is masked before changing
the NMI, IRQ, TINT interrupt settings. Aside of that, after setting an edge
trigger type it is required to clear the interrupt status register in order
to avoid spurious interrupts.

The current implementation fails to do either of that and therefore is
prone to generate spurious interrupts when setting the trigger type.

Address this by:

  - Ensuring that the interrupt is masked at the chip level across the
    update for the TINT chip

  - Clearing the interrupt status register after updating the trigger mode
    for edge type interrupts

[ tglx: Massaged changelog and reverted the spin_lock_irqsave() change as
   the set_type() callback is always called with interrupts disabled. ]

Fixes: 3fed09559cd8 ("irqchip: Add RZ/G2L IA55 Interrupt Controller driver")
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
drivers/irqchip/irq-renesas-rzg2l.c