diff options
| author | Viresh Kumar <viresh.kumar@linaro.org> | 2014-09-04 14:01:27 +0200 |
|---|---|---|
| committer | Sebastian Reichel <sre@kernel.org> | 2014-09-16 11:01:30 +0200 |
| commit | a0f93b4268d903430d7e21110f10be0919daf3b2 (patch) | |
| tree | 04fb92d5e3e21c14c1fda11c32ed477baec4c4cd /drivers/power | |
| parent | power-supply: Drop unnecessary typecasts (diff) | |
| download | linux-a0f93b4268d903430d7e21110f10be0919daf3b2.tar.xz linux-a0f93b4268d903430d7e21110f10be0919daf3b2.zip | |
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>
Diffstat (limited to 'drivers/power')
| -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, do { np = of_parse_phandle(psy->of_node, "power-supplies", i++); if (!np) - continue; + break; if (np == epsy->of_node) { dev_info(psy->dev, "%s: Found supply : %s\n", @@ -215,7 +215,7 @@ static int power_supply_check_supplies(struct power_supply *psy) np = of_parse_phandle(psy->of_node, "power-supplies", cnt++); if (!np) - continue; + break; ret = power_supply_find_supply_from_node(np); if (ret) { |