pinctrl: cy8c95x0: Initialise boolean variable with boolean values
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 5 Feb 2025 09:51:16 +0000 (11:51 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Sun, 16 Feb 2025 23:17:26 +0000 (00:17 +0100)
The 'ret' variable in cy8c95x0_irq_handler() is defined as bool,
but is intialised with integers. Avoid implicit castings and
initialise boolean variable with boolean values.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/20250205095243.512292-7-andriy.shevchenko@linux.intel.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-cy8c95x0.c

index 19f92ec8387147ff15d4c097e0debe7c68c4bcda..7e79f20f4d78049c6e1453e2509c4012e762452c 100644 (file)
@@ -1076,7 +1076,7 @@ static irqreturn_t cy8c95x0_irq_handler(int irq, void *devid)
        if (!ret)
                return IRQ_RETVAL(0);
 
-       ret = 0;
+       ret = false;
        for_each_set_bit(level, pending, MAX_LINE) {
                /* Already accounted for 4bit gap in GPort2 */
                nested_irq = irq_find_mapping(gc->irq.domain, level);
@@ -1095,7 +1095,7 @@ static irqreturn_t cy8c95x0_irq_handler(int irq, void *devid)
                else
                        handle_nested_irq(nested_irq);
 
-               ret = 1;
+               ret = true;
        }
 
        return IRQ_RETVAL(ret);