From: Wei Yongjun <yongjun_...@trendmicro.com.cn>

Using for_each_set_bit() to simplify the code.

spatch with a semantic match is used to found this.
(http://coccinelle.lip6.fr/)

Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>
---
 drivers/gpio/gpio-pxa.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/gpio/gpio-pxa.c b/drivers/gpio/gpio-pxa.c
index 9cac88a..a9e1a74 100644
--- a/drivers/gpio/gpio-pxa.c
+++ b/drivers/gpio/gpio-pxa.c
@@ -413,12 +413,10 @@ static void pxa_gpio_demux_handler(unsigned int irq, 
struct irq_desc *desc)
                        gedr = gedr & c->irq_mask;
                        writel_relaxed(gedr, c->regbase + GEDR_OFFSET);
 
-                       n = find_first_bit(&gedr, BITS_PER_LONG);
-                       while (n < BITS_PER_LONG) {
+                       for_each_set_bit(n, &gedr, BITS_PER_LONG) {
                                loop = 1;
 
                                generic_handle_irq(gpio_to_irq(gpio_base + n));
-                               n = find_next_bit(&gedr, BITS_PER_LONG, n + 1);
                        }
                }
        } while (loop);


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to