| Message ID | 1455183497-299798-1-git-send-email-arnd@arndb.de |
|---|---|
| State | New |
| Headers | show |
On Thu, Feb 11, 2016 at 10:38:08AM +0100, Arnd Bergmann wrote: > gcc cannot know that a for_each_possible_cpu() loop always executes > at least once, so it warns about the use of the "found" variable > in qcom_saw_regulator_probe: I'm going to drop this driver so this won't be needed.
diff --git a/drivers/regulator/qcom_saw-regulator.c b/drivers/regulator/qcom_saw-regulator.c index c800f16adaf0..30af1d380f9f 100644 --- a/drivers/regulator/qcom_saw-regulator.c +++ b/drivers/regulator/qcom_saw-regulator.c @@ -137,7 +137,7 @@ static struct saw_vreg *saw_get_drv(struct platform_device *pdev, struct saw_vreg *vreg = NULL; struct device_node *cpu_node, *saw_node; int cpu; - bool found; + bool found = 0; for_each_possible_cpu(cpu) { cpu_node = of_cpu_device_node_get(cpu);
gcc cannot know that a for_each_possible_cpu() loop always executes at least once, so it warns about the use of the "found" variable in qcom_saw_regulator_probe: drivers/regulator/qcom_saw-regulator.c: In function 'qcom_saw_regulator_probe': drivers/regulator/qcom_saw-regulator.c:154:5: error: 'found' is used uninitialized in this function [-Werror=uninitialized] This adds an explicit initialization to avoid the warning. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Fixes: 18bba3b50355 ("regulator: qcom-saw: Add support for SAW regulators") --- drivers/regulator/qcom_saw-regulator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.7.0