summaryrefslogtreecommitdiffstats
path: root/drivers/regulator/mc13xxx-regulator-core.c
diff options
context:
space:
mode:
authorAxel Lin <axel.lin@gmail.com>2012-06-08 09:39:34 +0200
committerMark Brown <broonie@opensource.wolfsonmicro.com>2012-06-17 22:01:21 +0200
commitee5e6253ac22210da892435711890fa2272daa41 (patch)
tree067bec34854e22272acd182b597d3fc03ac03251 /drivers/regulator/mc13xxx-regulator-core.c
parentregulator: tps62360: Simplify tps62360_set_voltage_time_sel implementation (diff)
downloadlinux-ee5e6253ac22210da892435711890fa2272daa41.tar.xz
linux-ee5e6253ac22210da892435711890fa2272daa41.zip
regulator: mc13xxx: Fix voltage checking for mc13xxx_fixed_regulator_set_voltage
The voltage range checking should be to ensure mc13xxx_regulators[id].voltages[0] falls with min_uV and max_uV. Signed-off-by: Axel Lin <axel.lin@gmail.com> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/regulator/mc13xxx-regulator-core.c')
-rw-r--r--drivers/regulator/mc13xxx-regulator-core.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/mc13xxx-regulator-core.c b/drivers/regulator/mc13xxx-regulator-core.c
index 4fa9704739bc..3e191715fb25 100644
--- a/drivers/regulator/mc13xxx-regulator-core.c
+++ b/drivers/regulator/mc13xxx-regulator-core.c
@@ -158,8 +158,8 @@ int mc13xxx_fixed_regulator_set_voltage(struct regulator_dev *rdev, int min_uV,
dev_dbg(rdev_get_dev(rdev), "%s id: %d min_uV: %d max_uV: %d\n",
__func__, id, min_uV, max_uV);
- if (min_uV >= mc13xxx_regulators[id].voltages[0] &&
- max_uV <= mc13xxx_regulators[id].voltages[0])
+ if (min_uV <= mc13xxx_regulators[id].voltages[0] &&
+ mc13xxx_regulators[id].voltages[0] <= max_uV)
return 0;
else
return -EINVAL;