regulator: hi6421: style fix, else with a single return is not required
authorGuodong Xu <guodong.xu@linaro.org>
Tue, 19 Aug 2014 10:07:42 +0000 (18:07 +0800)
committerMark Brown <broonie@linaro.org>
Tue, 19 Aug 2014 15:37:20 +0000 (10:37 -0500)
style fix for warnings. 'else' with a single 'return' is usually not
required.

Signed-off-by: Guodong Xu <guodong.xu@linaro.org>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/hi6421-regulator.c

index b40851eb54662800fb864d89a45fdadc75c0c86d..b0de92bee4a29dd01176dd7abc89a5acc9f08aea 100644 (file)
@@ -435,8 +435,8 @@ static unsigned int hi6421_regulator_ldo_get_mode(struct regulator_dev *rdev)
        regmap_read(rdev->regmap, rdev->desc->enable_reg, &reg_val);
        if (reg_val & info->mode_mask)
                return REGULATOR_MODE_IDLE;
-       else
-               return REGULATOR_MODE_NORMAL;
+
+       return REGULATOR_MODE_NORMAL;
 }
 
 static unsigned int hi6421_regulator_buck_get_mode(struct regulator_dev *rdev)
@@ -447,8 +447,8 @@ static unsigned int hi6421_regulator_buck_get_mode(struct regulator_dev *rdev)
        regmap_read(rdev->regmap, rdev->desc->enable_reg, &reg_val);
        if (reg_val & info->mode_mask)
                return REGULATOR_MODE_STANDBY;
-       else
-               return REGULATOR_MODE_NORMAL;
+
+       return REGULATOR_MODE_NORMAL;
 }
 
 static int hi6421_regulator_ldo_set_mode(struct regulator_dev *rdev,
@@ -506,8 +506,8 @@ unsigned int hi6421_regulator_ldo_get_optimum_mode(struct regulator_dev *rdev,
 
        if (load_uA > info->eco_microamp)
                return REGULATOR_MODE_NORMAL;
-       else
-               return REGULATOR_MODE_IDLE;
+
+       return REGULATOR_MODE_IDLE;
 }
 
 static const struct regulator_ops hi6421_ldo_ops = {
This page took 0.025258 seconds and 5 git commands to generate.