aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2014-09-04 08:01:27 -0400
committerSebastian Reichel <sre@kernel.org>2014-09-16 05:01:30 -0400
commita0f93b4268d903430d7e21110f10be0919daf3b2 (patch)
tree04fb92d5e3e21c14c1fda11c32ed477baec4c4cd
parente80cf421432bab7327891e7e8afe027fafb4c27b (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.c4
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) {