drivers/gpio/pm8916_gpio.c: Make pid be uint32_t
authorTom Rini <trini@konsulko.com>
Tue, 12 Apr 2016 19:11:23 +0000 (15:11 -0400)
committerTom Rini <trini@konsulko.com>
Mon, 18 Apr 2016 21:11:42 +0000 (17:11 -0400)
If get_dev_addr fails it will return FDT_ADDR_T_NONE and:
>>>     "priv->pid == 4294967295U" is always false regardless of the values of its operands. This occurs as the logical operand of if.

Cc: Mateusz Kulikowski <mateusz.kulikowski@gmail.com>
Reported-by: Coverity (CID: 143913)
Signed-off-by: Tom Rini <trini@konsulko.com>
drivers/gpio/pm8916_gpio.c

index 1abab7fef1701928d597b89884ed8aff0cace3f2..0b619758438f856fb664b572a007579139f39784 100644 (file)
@@ -50,7 +50,7 @@ DECLARE_GLOBAL_DATA_PTR;
 #define REG_EN_CTL_ENABLE      (1 << 7)
 
 struct pm8916_gpio_bank {
-       uint16_t pid; /* Peripheral ID on SPMI bus */
+       uint32_t pid; /* Peripheral ID on SPMI bus */
 };
 
 static int pm8916_gpio_set_direction(struct udevice *dev, unsigned offset,