gpio: max77620: Replace 8 with MAX77620_GPIO_NR
authorDmitry Osipenko <digetx@gmail.com>
Thu, 9 Jul 2020 17:11:58 +0000 (20:11 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 16 Jul 2020 08:50:40 +0000 (10:50 +0200)
The MAX77620_GPIO_NR enum value represents the total number of GPIOs,
let's use it instead of a raw value in order to improve the code's
readability a tad.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
Link: https://lore.kernel.org/r/20200709171203.12950-2-digetx@gmail.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-max77620.c

index 313bd02..4c0c9ec 100644 (file)
@@ -19,8 +19,8 @@ struct max77620_gpio {
        struct regmap           *rmap;
        struct device           *dev;
        struct mutex            buslock; /* irq_bus_lock */
-       unsigned int            irq_type[8];
-       bool                    irq_enabled[8];
+       unsigned int            irq_type[MAX77620_GPIO_NR];
+       bool                    irq_enabled[MAX77620_GPIO_NR];
 };
 
 static irqreturn_t max77620_gpio_irqhandler(int irq, void *data)
@@ -38,7 +38,7 @@ static irqreturn_t max77620_gpio_irqhandler(int irq, void *data)
 
        pending = value;
 
-       for_each_set_bit(offset, &pending, 8) {
+       for_each_set_bit(offset, &pending, MAX77620_GPIO_NR) {
                unsigned int virq;
 
                virq = irq_find_mapping(gpio->gpio_chip.irq.domain, offset);