]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
gpio: sifive: fix static checker warning
authorYash Shah <yash.shah@sifive.com>
Tue, 28 Jan 2020 05:24:21 +0000 (10:54 +0530)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 10 Feb 2020 12:54:17 +0000 (13:54 +0100)
Typcasting "irq_state" leads to the below static checker warning:
The fix is to declare "irq_state" as unsigned long instead of u32.

drivers/gpio/gpio-sifive.c:97 sifive_gpio_irq_enable()
warn: passing casted pointer '&chip->irq_state' to
'assign_bit()' 32 vs 64.

Fixes: 96868dce644d ("gpio/sifive: Add GPIO driver for SiFive SoCs")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Yash Shah <yash.shah@sifive.com>
Link: https://lore.kernel.org/r/1580189061-14091-1-git-send-email-yash.shah@sifive.com
Reviewed-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-sifive.c

index 147a1bd0451521bf5e9c78b6d7d4cdd968d6559c..c54dd08f2cbfd3c75d74ffec723f4e0e5f72e4f7 100644 (file)
@@ -35,7 +35,7 @@ struct sifive_gpio {
        void __iomem            *base;
        struct gpio_chip        gc;
        struct regmap           *regs;
-       u32                     irq_state;
+       unsigned long           irq_state;
        unsigned int            trigger[SIFIVE_GPIO_MAX];
        unsigned int            irq_parent[SIFIVE_GPIO_MAX];
 };
@@ -94,7 +94,7 @@ static void sifive_gpio_irq_enable(struct irq_data *d)
        spin_unlock_irqrestore(&gc->bgpio_lock, flags);
 
        /* Enable interrupts */
-       assign_bit(offset, (unsigned long *)&chip->irq_state, 1);
+       assign_bit(offset, &chip->irq_state, 1);
        sifive_gpio_set_ie(chip, offset);
 }
 
@@ -104,7 +104,7 @@ static void sifive_gpio_irq_disable(struct irq_data *d)
        struct sifive_gpio *chip = gpiochip_get_data(gc);
        int offset = irqd_to_hwirq(d) % SIFIVE_GPIO_MAX;
 
-       assign_bit(offset, (unsigned long *)&chip->irq_state, 0);
+       assign_bit(offset, &chip->irq_state, 0);
        sifive_gpio_set_ie(chip, offset);
        irq_chip_disable_parent(d);
 }