diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2014-09-04 08:01:27 -0400 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2014-09-16 05:01:30 -0400 |
commit | a0f93b4268d903430d7e21110f10be0919daf3b2 (patch) | |
tree | 04fb92d5e3e21c14c1fda11c32ed477baec4c4cd | |
parent | e80cf421432bab7327891e7e8afe027fafb4c27b (diff) |
power-supply: Use 'break' instead of 'continue' to end loop
In few routines, we need to end the do-while loop when no more "power-supplies"
are available. Currently we are doing 'continue' which will make the
'while(np)' conditional statement run again.
Skip this by doing a 'break' instead.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
-rw-r--r-- | drivers/power/power_supply_core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/power/power_supply_core.c b/drivers/power/power_supply_core.c index 71d00ecca881..7657335ea7f9 100644 --- a/drivers/power/power_supply_core.c +++ b/drivers/power/power_supply_core.c | |||
@@ -127,7 +127,7 @@ static int __power_supply_populate_supplied_from(struct device *dev, | |||
127 | do { | 127 | do { |
128 | np = of_parse_phandle(psy->of_node, "power-supplies", i++); | 128 | np = of_parse_phandle(psy->of_node, "power-supplies", i++); |
129 | if (!np) | 129 | if (!np) |
130 | continue; | 130 | break; |
131 | 131 | ||
132 | if (np == epsy->of_node) { | 132 | if (np == epsy->of_node) { |
133 | dev_info(psy->dev, "%s: Found supply : %s\n", | 133 | dev_info(psy->dev, "%s: Found supply : %s\n", |
@@ -215,7 +215,7 @@ static int power_supply_check_supplies(struct power_supply *psy) | |||
215 | 215 | ||
216 | np = of_parse_phandle(psy->of_node, "power-supplies", cnt++); | 216 | np = of_parse_phandle(psy->of_node, "power-supplies", cnt++); |
217 | if (!np) | 217 | if (!np) |
218 | continue; | 218 | break; |
219 | 219 | ||
220 | ret = power_supply_find_supply_from_node(np); | 220 | ret = power_supply_find_supply_from_node(np); |
221 | if (ret) { | 221 | if (ret) { |