pinctrl: sunxi: fix theoretical uninitialized variable access
authorArnd Bergmann <arnd@arndb.de>
Wed, 16 Nov 2016 14:18:18 +0000 (15:18 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 16 Nov 2016 19:53:48 +0000 (20:53 +0100)
gcc warns about a  way that it could use an uninitialized variable:

drivers/pinctrl/sunxi/pinctrl-sunxi.c: In function 'sunxi_pinctrl_init':
drivers/pinctrl/sunxi/pinctrl-sunxi.c:1191:8: error: 'best_div' may be used uninitialized in this function [-Werror=maybe-uninitialized]

This cannot really happen except if 'freq' is UINT_MAX and 'clock' is
zero, and both of these are forbidden. To shut up the warning anyway,
this changes the logic to initialize the return code to the first
divider value before looking at the others.

Fixes: 7c926492d38a ("pinctrl: sunxi: Add support for interrupt debouncing")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/sunxi/pinctrl-sunxi.c

index 2339d4718b4dbfae9d14d36655cb716b04253a10..6b7953da4228f12471051c2502e59c270599b3e6 100644 (file)
@@ -1125,10 +1125,13 @@ static int sunxi_pinctrl_build_state(struct platform_device *pdev)
 static int sunxi_pinctrl_get_debounce_div(struct clk *clk, int freq, int *diff)
 {
        unsigned long clock = clk_get_rate(clk);
-       unsigned int best_diff = ~0, best_div;
+       unsigned int best_diff, best_div;
        int i;
 
-       for (i = 0; i < 8; i++) {
+       best_diff = abs(freq - clock);
+       best_div = 0;
+
+       for (i = 1; i < 8; i++) {
                int cur_diff = abs(freq - (clock >> i));
 
                if (cur_diff < best_diff) {