regulator: mc13xxx: Fix voltage checking for mc13xxx_fixed_regulator_set_voltage
authorAxel Lin <axel.lin@gmail.com>
Fri, 8 Jun 2012 07:39:34 +0000 (15:39 +0800)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Sun, 17 Jun 2012 20:01:21 +0000 (21:01 +0100)
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>
drivers/regulator/mc13xxx-regulator-core.c

index 4fa9704739bc41bb764ece50e58071783501966a..3e191715fb25eb55a24ca17604acad575e94c690 100644 (file)
@@ -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;