regmap: Fix 'ret' would return an uninitialized value
This patch give a warning when calling regmap_register_patch with parameter num_regs <= 0. When the num_regs parameter is zero and krealloc doesn't fail, then the code would return an uninitialized value. However, calling this function with num_regs == 0, would be a waste as it essentially does nothing. Signed-off-by: Cai Zhiyong <caizhiyong@huawei.com> Reviewed-by: Geyslan G. Bem <geyslan@gmail.com> Signed-off-by: Mark Brown <broonie@linaro.org>
This commit is contained in:
parent
6ce4eac1f6
commit
bd60e381fa
|
@ -2173,6 +2173,10 @@ int regmap_register_patch(struct regmap *map, const struct reg_default *regs,
|
||||||
int i, ret;
|
int i, ret;
|
||||||
bool bypass;
|
bool bypass;
|
||||||
|
|
||||||
|
if (WARN_ONCE(num_regs <= 0, "invalid registers number (%d)\n",
|
||||||
|
num_regs))
|
||||||
|
return 0;
|
||||||
|
|
||||||
map->lock(map->lock_arg);
|
map->lock(map->lock_arg);
|
||||||
|
|
||||||
bypass = map->cache_bypass;
|
bypass = map->cache_bypass;
|
||||||
|
|
Loading…
Reference in a new issue