diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-03-24 18:13:50 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2017-03-24 19:41:02 +0100 |
| commit | c635df496a5c397fd078b4e72a42e1be79632d7d (patch) | |
| tree | dd111ced36a03fb19a94919d4f095b88bc7d47bd /drivers/regulator/twl6030-regulator.c | |
| parent | Merge remote-tracking branches 'regulator/topic/tps65086' and 'regulator/topi... (diff) | |
| download | linux-c635df496a5c397fd078b4e72a42e1be79632d7d.tar.xz linux-c635df496a5c397fd078b4e72a42e1be79632d7d.zip | |
regulator: twl6030: remove redundant range check min_uV > 1300000 && min_uV <= 1350000
It has been pointed out to me that the range for vsel = 58 is actually
dead code as this is covered by an earlier check for (min_uV >= 700000)
&& (min_uV <= 1420000) so remove that check completely.
Reported-by: Alban Auzeill <alban.auzeill@sonarsource.com>
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/twl6030-regulator.c')
| -rw-r--r-- | drivers/regulator/twl6030-regulator.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/regulator/twl6030-regulator.c b/drivers/regulator/twl6030-regulator.c index 4864b9d742c0..edaf93cc7823 100644 --- a/drivers/regulator/twl6030-regulator.c +++ b/drivers/regulator/twl6030-regulator.c @@ -456,8 +456,6 @@ static int twl6030smps_map_voltage(struct regulator_dev *rdev, int min_uV, vsel = 60; else if ((min_uV > 1350000) && (min_uV <= 1500000)) vsel = 59; - else if ((min_uV > 1300000) && (min_uV <= 1350000)) - vsel = 58; else return -EINVAL; break; |