regulator: max8998: Let regulator core handle the case selector == old_selector
authorAxel Lin <axel.lin@ingics.com>
Wed, 6 Feb 2013 03:10:51 +0000 (11:10 +0800)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 8 Feb 2013 11:23:43 +0000 (11:23 +0000)
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
drivers/regulator/max8998.c

index 1f0df4046b86d0a303c229379091d76407f61129..24e2d1dec8c309c9408a3530cf15624d0a2cfa63 100644 (file)
@@ -311,25 +311,13 @@ static int max8998_set_voltage_buck_sel(struct regulator_dev *rdev,
                dev_get_platdata(max8998->iodev->dev);
        struct i2c_client *i2c = max8998->iodev->i2c;
        int buck = rdev_get_id(rdev);
-       int reg, shift = 0, mask, ret;
-       int j, previous_sel;
+       int reg, shift = 0, mask, ret, j;
        static u8 buck1_last_val;
 
        ret = max8998_get_voltage_register(rdev, &reg, &shift, &mask);
        if (ret)
                return ret;
 
-       previous_sel = max8998_get_voltage_sel(rdev);
-
-       /* Check if voltage needs to be changed */
-       /* if previous_voltage equal new voltage, return */
-       if (previous_sel == selector) {
-               dev_dbg(max8998->dev, "No voltage change, old:%d, new:%d\n",
-                       regulator_list_voltage_linear(rdev, previous_sel),
-                       regulator_list_voltage_linear(rdev, selector));
-               return ret;
-       }
-
        switch (buck) {
        case MAX8998_BUCK1:
                dev_dbg(max8998->dev,