Re: [PATCH 03/11] irqchip: bcm7120-l2: Eliminate bad IRQ check

2014-10-29 Thread Florian Fainelli
On 10/28/2014 08:58 PM, Kevin Cernekee wrote: > This check may be prone to race conditions, e.g. > > 1) Some external event (e.g. GPIO level) causes an IRQ to become pending > 2) Peripheral asserts the L2 IRQ > 3) CPU takes an interrupt > 4) The event from #1 goes away > 5) bcm7120_l2_intc_irq_han

[PATCH 03/11] irqchip: bcm7120-l2: Eliminate bad IRQ check

2014-10-28 Thread Kevin Cernekee
This check may be prone to race conditions, e.g. 1) Some external event (e.g. GPIO level) causes an IRQ to become pending 2) Peripheral asserts the L2 IRQ 3) CPU takes an interrupt 4) The event from #1 goes away 5) bcm7120_l2_intc_irq_handle() reads back a 0 status Unlike the hardware supported b