diff options
author | Guenter Roeck <linux@roeck-us.net> | 2012-09-19 14:21:51 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2012-09-24 00:08:35 -0400 |
commit | 3be8102fe30c09e9b264a2f08ae2f69da7e32eb4 (patch) | |
tree | 9b1ce5af8e456cd63ac53e9c2543fce639603425 /drivers/hwmon | |
parent | 860f37319e9a1141b041ca2e77ca121dccde4b93 (diff) |
hwmon: (lm93) Drop unnecessary compare statement
The following build warning is seen with -Wextra.
lm93.c: In function ‘store_fan_smart_tach’:
lm93.c:1833:2: warning: comparison of unsigned expression >= 0 is always true
Drop the unnecessary comparison.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/lm93.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/lm93.c b/drivers/hwmon/lm93.c index bf946187bd37..c3d4255ed154 100644 --- a/drivers/hwmon/lm93.c +++ b/drivers/hwmon/lm93.c | |||
@@ -1830,7 +1830,7 @@ static ssize_t store_fan_smart_tach(struct device *dev, | |||
1830 | 1830 | ||
1831 | mutex_lock(&data->update_lock); | 1831 | mutex_lock(&data->update_lock); |
1832 | /* sanity test, ignore the write otherwise */ | 1832 | /* sanity test, ignore the write otherwise */ |
1833 | if (0 <= val && val <= 2) { | 1833 | if (val <= 2) { |
1834 | /* can't enable if pwm freq is 22.5KHz */ | 1834 | /* can't enable if pwm freq is 22.5KHz */ |
1835 | if (val) { | 1835 | if (val) { |
1836 | u8 ctl4 = lm93_read_byte(client, | 1836 | u8 ctl4 = lm93_read_byte(client, |