diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-07-19 15:07:19 +0200 |
|---|---|---|
| committer | Darren Hart (VMware) <dvhart@infradead.org> | 2017-07-25 18:56:37 +0200 |
| commit | 890f658c101df5842ce8aaec1b0833cf1ba37eb3 (patch) | |
| tree | bc3743e6e2e423ff29be4a137d8347768412ccbb /drivers/platform | |
| parent | platform/x86: wmi: Fix error handling in acpi_wmi_init() (diff) | |
| download | linux-890f658c101df5842ce8aaec1b0833cf1ba37eb3.tar.xz linux-890f658c101df5842ce8aaec1b0833cf1ba37eb3.zip | |
platform/x86: peaq-wmi: silence a static checker warning
There is a harmless static checker warning here that unsigned values are
always >= 0. The code looks like:
if (peaq_ignore_events_counter && --peaq_ignore_events_counter >= 0)
The first part of the condition ensures that we never wrap around so the
code works as intended. I've tweaked it slightly to avoid the warning.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Diffstat (limited to 'drivers/platform')
| -rw-r--r-- | drivers/platform/x86/peaq-wmi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/platform/x86/peaq-wmi.c b/drivers/platform/x86/peaq-wmi.c index 77d1f90b0794..c68bd76954fd 100644 --- a/drivers/platform/x86/peaq-wmi.c +++ b/drivers/platform/x86/peaq-wmi.c @@ -51,7 +51,7 @@ static void peaq_wmi_poll(struct input_polled_dev *dev) return; } - if (peaq_ignore_events_counter && --peaq_ignore_events_counter >= 0) + if (peaq_ignore_events_counter && peaq_ignore_events_counter--) return; if (obj.integer.value) { |