diff options
author | Stephen Boyd <sboyd@codeaurora.org> | 2015-06-16 14:11:22 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-06-17 03:54:19 -0400 |
commit | 9b2dfee3954e0b38a6a1b5305edd375defa8ad97 (patch) | |
tree | 7ed50765931b1af90d10ed72daea9dfed61994e4 | |
parent | 0f94bffad401faa045c2062f1652d21e3e89f439 (diff) |
regulator: qcom_spmi: Add missing braces for aligned code
drivers/regulator/qcom_spmi-regulator.c:751:3-50: code aligned
with following code on line 753
drivers/regulator/qcom_spmi-regulator.c:584:3-41: code aligned
with following code on line 587
These lines where missing braces causing the break to always
be executed even when it shouldn't be. Fix it.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Reviewed-by: Andy Gross <agross@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/regulator/qcom_spmi-regulator.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/regulator/qcom_spmi-regulator.c b/drivers/regulator/qcom_spmi-regulator.c index 15bba2b4ef70..16a5c1885153 100644 --- a/drivers/regulator/qcom_spmi-regulator.c +++ b/drivers/regulator/qcom_spmi-regulator.c | |||
@@ -580,11 +580,12 @@ static int spmi_regulator_select_voltage_same_range(struct spmi_regulator *vreg, | |||
580 | *selector = 0; | 580 | *selector = 0; |
581 | for (i = 0; i < vreg->set_points->count; i++) { | 581 | for (i = 0; i < vreg->set_points->count; i++) { |
582 | if (uV >= vreg->set_points->range[i].set_point_min_uV | 582 | if (uV >= vreg->set_points->range[i].set_point_min_uV |
583 | && uV <= vreg->set_points->range[i].set_point_max_uV) | 583 | && uV <= vreg->set_points->range[i].set_point_max_uV) { |
584 | *selector += | 584 | *selector += |
585 | (uV - vreg->set_points->range[i].set_point_min_uV) | 585 | (uV - vreg->set_points->range[i].set_point_min_uV) |
586 | / vreg->set_points->range[i].step_uV; | 586 | / vreg->set_points->range[i].step_uV; |
587 | break; | 587 | break; |
588 | } | ||
588 | 589 | ||
589 | *selector += vreg->set_points->range[i].n_voltages; | 590 | *selector += vreg->set_points->range[i].n_voltages; |
590 | } | 591 | } |
@@ -743,10 +744,11 @@ static int spmi_regulator_common_list_voltage(struct regulator_dev *rdev, | |||
743 | return 0; | 744 | return 0; |
744 | 745 | ||
745 | for (i = 0; i < vreg->set_points->count; i++) { | 746 | for (i = 0; i < vreg->set_points->count; i++) { |
746 | if (selector < vreg->set_points->range[i].n_voltages) | 747 | if (selector < vreg->set_points->range[i].n_voltages) { |
747 | uV = selector * vreg->set_points->range[i].step_uV | 748 | uV = selector * vreg->set_points->range[i].step_uV |
748 | + vreg->set_points->range[i].set_point_min_uV; | 749 | + vreg->set_points->range[i].set_point_min_uV; |
749 | break; | 750 | break; |
751 | } | ||
750 | 752 | ||
751 | selector -= vreg->set_points->range[i].n_voltages; | 753 | selector -= vreg->set_points->range[i].n_voltages; |
752 | } | 754 | } |