diff options
author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-06-28 17:34:38 -0400 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2014-07-09 09:58:22 -0400 |
commit | a3dd01e17756e85badd587de774cec783abe1b19 (patch) | |
tree | 6f288648fb44c55b3a4af5b47f62f20142956b96 /drivers/mfd/ab8500-debugfs.c | |
parent | 005d16b6022676ba8c6ae93e2d38cb0e70e0ef92 (diff) |
mfd: ab8500-debugfs: Cleaning up unnecessary to test, unsigned can't be negative.
Unsigned variable can't be negative so it is unnecessary to test it
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd/ab8500-debugfs.c')
-rw-r--r-- | drivers/mfd/ab8500-debugfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c index 8452c15bf4d4..e23fcbad79d6 100644 --- a/drivers/mfd/ab8500-debugfs.c +++ b/drivers/mfd/ab8500-debugfs.c | |||
@@ -2503,7 +2503,7 @@ static ssize_t ab8500_gpadc_trig_timer_write(struct file *file, | |||
2503 | if (err) | 2503 | if (err) |
2504 | return err; | 2504 | return err; |
2505 | 2505 | ||
2506 | if ((user_trig_timer >= 0) && (user_trig_timer <= 255)) { | 2506 | if (user_trig_timer <= 255) { |
2507 | trig_timer = (u8) user_trig_timer; | 2507 | trig_timer = (u8) user_trig_timer; |
2508 | } else { | 2508 | } else { |
2509 | dev_err(dev, "debugfs error input: " | 2509 | dev_err(dev, "debugfs error input: " |