Commit b8c77ff6 authored by Dmitry Torokhov's avatar Dmitry Torokhov Committed by Mark Brown

regulator: core: simplify regulator_bulk_force_disable()

There is no need to have two loops there, we can store error for subsequent
reporting.
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent a8bd42a9
...@@ -3687,21 +3687,17 @@ int regulator_bulk_force_disable(int num_consumers, ...@@ -3687,21 +3687,17 @@ int regulator_bulk_force_disable(int num_consumers,
struct regulator_bulk_data *consumers) struct regulator_bulk_data *consumers)
{ {
int i; int i;
int ret; int ret = 0;
for (i = 0; i < num_consumers; i++) for (i = 0; i < num_consumers; i++) {
consumers[i].ret = consumers[i].ret =
regulator_force_disable(consumers[i].consumer); regulator_force_disable(consumers[i].consumer);
for (i = 0; i < num_consumers; i++) { /* Store first error for reporting */
if (consumers[i].ret != 0) { if (consumers[i].ret && !ret)
ret = consumers[i].ret; ret = consumers[i].ret;
goto out;
}
} }
return 0;
out:
return ret; return ret;
} }
EXPORT_SYMBOL_GPL(regulator_bulk_force_disable); EXPORT_SYMBOL_GPL(regulator_bulk_force_disable);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment