Hi Paul, On Fri, Aug 19, 2016 at 06:11:19PM +0100, Paul Burton wrote: > The MIPS GIC driver has previously iterated over bits set in a bitmap > representing pending IRQs by calling find_first_bit, clearing that bit > then calling find_first_bit again until all bits are clear. If multiple > interrupts are pending then this is wasteful, as find_first_bit will > have to loop over the whole bitmap from the start. Use the > for_each_set_bit macro which performs exactly what we need here instead. > It will use find_next_bit and thus only scan over the relevant part of > the bitmap, and it makes the intent of the code clearer. > > Signed-off-by: Paul Burton <paul.burton@xxxxxxxxxx> > --- > drivers/irqchip/irq-mips-gic.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) Applied to irqchip/core. thx, Jason.
