diff options
author | Axel Lin <axel.lin@gmail.com> | 2012-02-07 04:06:20 +0100 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-03-11 21:48:40 +0100 |
commit | 46783a046e13588f0459271ad6db9785fa8dcb8b (patch) | |
tree | f0944269fb2ccd3570d6f36b18e65ee41a5dbb0b /drivers/regulator/tps62360-regulator.c | |
parent | regulator: s5m8767: Fix unused variable warning in probe() (diff) | |
download | linux-46783a046e13588f0459271ad6db9785fa8dcb8b.tar.xz linux-46783a046e13588f0459271ad6db9785fa8dcb8b.zip |
regulator: tps62360: Remove pointless test for unsigned less than zero
The variable 'selector' is a 'unsigned int', so it can never be less than zero.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/regulator/tps62360-regulator.c')
-rw-r--r-- | drivers/regulator/tps62360-regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/tps62360-regulator.c b/drivers/regulator/tps62360-regulator.c index f778ef067696..e2ec73068ee2 100644 --- a/drivers/regulator/tps62360-regulator.c +++ b/drivers/regulator/tps62360-regulator.c @@ -191,7 +191,7 @@ static int tps62360_dcdc_list_voltage(struct regulator_dev *dev, { struct tps62360_chip *tps = rdev_get_drvdata(dev); - if ((selector < 0) || (selector >= tps->desc.n_voltages)) + if (selector >= tps->desc.n_voltages) return -EINVAL; return (tps->voltage_base + selector * 10) * 1000; } |