]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
mfd: arizona: Correct type of gpio_defaults
authorCharles Keepax <ckeepax@opensource.wolfsonmicro.com>
Sun, 29 Mar 2015 11:45:42 +0000 (12:45 +0100)
committerLee Jones <lee.jones@linaro.org>
Mon, 30 Mar 2015 09:06:29 +0000 (10:06 +0100)
gpio_defaults needs to be specified as an unsigned int rather than an
int, because the intention of the DT binding is that all out of range
values for a 16-bit register will cause the defaults to be used,
however, if gpio_defaults is an int then values that are larger than
INT_MAX will become negative numbers and be written out directly to the
hardware. As no where in the code replies on gpio_defaults being an int,
the simplest fix is to just change it to unsigned.

Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
include/linux/mfd/arizona/pdata.h

index 48fe31356605fb42a9dce885a2acc3edd4715bd7..1789cb0f4f170171d4e37fc20192906a44a4eb44 100644 (file)
@@ -94,7 +94,7 @@ struct arizona_pdata {
        int gpio_base;
 
        /** Pin state for GPIO pins */
-       int gpio_defaults[ARIZONA_MAX_GPIO];
+       unsigned int gpio_defaults[ARIZONA_MAX_GPIO];
 
        /**
         * Maximum number of channels clocks will be generated for,