From: Tom Rini Date: Tue, 12 Apr 2016 19:11:24 +0000 (-0400) Subject: drivers/power/pmic/pm8916.c: Make usid be uint32_t X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=3bfc8152b2c09d81fb00f4d3990e5dfd59682b96;p=project%2Fbcm63xx%2Fu-boot.git drivers/power/pmic/pm8916.c: Make usid be uint32_t If get_dev_addr fails it will return FDT_ADDR_T_NONE and: >>> "priv->usid == 4294967295U" is always false regardless of the values of its operands. This occurs as the logical operand of if. Cc: Mateusz Kulikowski Reported-by: Coverity (CID: 143914) Signed-off-by: Tom Rini --- diff --git a/drivers/power/pmic/pm8916.c b/drivers/power/pmic/pm8916.c index 9acf5f55f7..d4c7d4adff 100644 --- a/drivers/power/pmic/pm8916.c +++ b/drivers/power/pmic/pm8916.c @@ -18,7 +18,7 @@ DECLARE_GLOBAL_DATA_PTR; #define REG_MASK 0xFF struct pm8916_priv { - uint16_t usid; /* Slave ID on SPMI bus */ + uint32_t usid; /* Slave ID on SPMI bus */ }; static int pm8916_reg_count(struct udevice *dev)