diff options
| author | Ye.Li <B37916@freescale.com> | 2016-01-04 15:26:30 +0800 |
|---|---|---|
| committer | Stefano Babic <sbabic@denx.de> | 2016-01-07 17:48:25 +0100 |
| commit | 7ea191aa10c43b5497d171ab6f48e633e6779468 (patch) | |
| tree | d7541475e53f9cee87eb709f7a339ffdb9812b4e | |
| parent | 59a6ca54f5db0f3d99ca3bc581ccadc6019e5034 (diff) | |
| download | blackbird-obmc-uboot-7ea191aa10c43b5497d171ab6f48e633e6779468.tar.gz blackbird-obmc-uboot-7ea191aa10c43b5497d171ab6f48e633e6779468.zip | |
pfuze: Fix unsigned variable for less-than-zero comparison
According to the Coverity result, a unsigned int variable is used fo less-
than-zero comparison, the result is never true. Need to fix the variable
type to signed int.
Signed-off-by: Ye.Li <B37916@freescale.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Cc: Przemyslaw Marczak <p.marczak@samsung.com>
Cc: Stefano Babic <sbabic@denx.de>
Reviewed-by: Stefano Babic <sbabic@denx.de>
| -rw-r--r-- | board/freescale/common/pfuze.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/board/freescale/common/pfuze.c b/board/freescale/common/pfuze.c index 783c46d882..69afa83562 100644 --- a/board/freescale/common/pfuze.c +++ b/board/freescale/common/pfuze.c @@ -13,7 +13,8 @@ int pfuze_mode_init(struct pmic *p, u32 mode) { unsigned char offset, i, switch_num; - u32 id, ret; + u32 id; + int ret; pmic_reg_read(p, PFUZE100_DEVICEID, &id); id = id & 0xf; |
