diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-03-24 13:13:50 -0400 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2017-03-24 14:41:02 -0400 |
| commit | c635df496a5c397fd078b4e72a42e1be79632d7d (patch) | |
| tree | dd111ced36a03fb19a94919d4f095b88bc7d47bd /drivers/regulator | |
| 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')
| -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, | |||
| 456 | vsel = 60; | 456 | vsel = 60; |
| 457 | else if ((min_uV > 1350000) && (min_uV <= 1500000)) | 457 | else if ((min_uV > 1350000) && (min_uV <= 1500000)) |
| 458 | vsel = 59; | 458 | vsel = 59; |
| 459 | else if ((min_uV > 1300000) && (min_uV <= 1350000)) | ||
| 460 | vsel = 58; | ||
| 461 | else | 459 | else |
| 462 | return -EINVAL; | 460 | return -EINVAL; |
| 463 | break; | 461 | break; |
