]> asedeno.scripts.mit.edu Git - linux.git/commit
irqchip/csky: fixup handle_irq_perbit break irq
authorGuo Ren <ren_guo@c-sky.com>
Tue, 8 Jan 2019 12:49:24 +0000 (20:49 +0800)
committerGuo Ren <ren_guo@c-sky.com>
Tue, 8 Jan 2019 16:18:46 +0000 (00:18 +0800)
commit56752b21755aef598709d143684cb363db98a7d2
treef2fd36e145d1415f306a295c61aff66783134b31
parent2a60aa14a9a0333b3eef58150bc1ef654f7321ef
irqchip/csky: fixup handle_irq_perbit break irq

The handle_irq_perbit function loop every bit in hwirq local variable.

handle_irq_perbit(hwirq) {
  for_everyt_bit_in(hwirq) {
handle_domain_irq()
->irq_exit()
->invoke_softirq()
->__do_softirq()
->local_irq_enable() // Here will cause new interrupt.
  }
}

When new interrupt coming at local_irq_enable, it will finish another
interrupt handler and pull down the interrupt source. But hwirq is the
local variable for handle_irq_perbit(), it can't get new interrupt
controller pending reg status. So we need update hwirq with pending reg
in every loop.

Also change write_relax to writel could prevent stw from fast retire.
When local_irq is enabled, intc regs is really set-in.

Signed-off-by: Guo Ren <ren_guo@c-sky.com>
Cc: Lu Baoquan <lu.baoquan@intellif.com>
drivers/irqchip/irq-csky-apb-intc.c