plat-nomadik: fix compilation warning
authorLinus Walleij <linus.walleij@linaro.org>
Wed, 16 Feb 2011 09:37:52 +0000 (10:37 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 14 Mar 2011 13:05:16 +0000 (14:05 +0100)
The compiler warns that [rf]wimsc may be used uninitialized in
this function - the warning is actually false since the uses are
in identical if()-clauses, but it can't hurt very much to read
out the values to be modified early anyway and rid the warning.

Cc: Rabin Vincent <rabin.vincent@stericsson.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
arch/arm/plat-nomadik/gpio.c

index 5e6653f..45b1cf9 100644 (file)
@@ -134,15 +134,12 @@ static void __nmk_gpio_set_mode_safe(struct nmk_gpio_chip *nmk_chip,
                                     unsigned offset, int gpio_mode,
                                     bool glitch)
 {
-       u32 rwimsc;
-       u32 fwimsc;
+       u32 rwimsc = readl(nmk_chip->addr + NMK_GPIO_RWIMSC);
+       u32 fwimsc = readl(nmk_chip->addr + NMK_GPIO_FWIMSC);
 
        if (glitch && nmk_chip->set_ioforce) {
                u32 bit = BIT(offset);
 
-               rwimsc = readl(nmk_chip->addr + NMK_GPIO_RWIMSC);
-               fwimsc = readl(nmk_chip->addr + NMK_GPIO_FWIMSC);
-
                /* Prevent spurious wakeups */
                writel(rwimsc & ~bit, nmk_chip->addr + NMK_GPIO_RWIMSC);
                writel(fwimsc & ~bit, nmk_chip->addr + NMK_GPIO_FWIMSC);