iio: accel: mma9553: fix gpio bitnum init value
authorIrina Tirdea <irina.tirdea@intel.com>
Mon, 13 Apr 2015 15:40:59 +0000 (18:40 +0300)
committerJonathan Cameron <jic23@kernel.org>
Sun, 26 Apr 2015 18:52:35 +0000 (19:52 +0100)
Initial value of gpio bitnum is set to -1, but
the variable is declared as unsigned.

Use a positive invalid value for initial gpio
bitnum.

Signed-off-by: Irina Tirdea <irina.tirdea@intel.com>
Suggested-by: Hartmut Knaack <knaack.h@gmx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/accel/mma9553.c

index 032537fc2c561a7d6aebdd249f1eabfb71e3c43c..9d649c4a21fd0edb26a4007d2dc2afa86338a37c 100644 (file)
@@ -77,6 +77,7 @@
 
 /* Bitnum used for GPIO configuration = bit number in high status byte */
 #define MMA9553_STATUS_TO_BITNUM(bit)  (ffs(bit) - 9)
+#define MMA9553_MAX_BITNUM             MMA9553_STATUS_TO_BITNUM(BIT(16))
 
 #define MMA9553_DEFAULT_SAMPLE_RATE    30      /* Hz */
 
@@ -406,8 +407,8 @@ static int mma9553_init(struct mma9553_data *data)
        }
 
 
-       data->gpio_bitnum = -1;
        /* Reset GPIO */
+       data->gpio_bitnum = MMA9553_MAX_BITNUM;
        ret = mma9553_conf_gpio(data);
        if (ret < 0)
                return ret;