diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-03-24 17:13:50 +0000 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2017-03-24 18:41:02 +0000 |
| commit | c635df496a5c397fd078b4e72a42e1be79632d7d (patch) | |
| tree | dd111ced36a03fb19a94919d4f095b88bc7d47bd /drivers/regulator/twl6030-regulator.c | |
| parent | d00b74613fb18dfd0a5aa99270ee2e72d5c808d7 (diff) | |
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; |