* [PATCH] platform/x86: peaq-wmi: Fix peaq_ignore_events_counter handling off by 1
@ 2017-07-11 11:41 Hans de Goede
2017-07-11 16:13 ` Darren Hart
0 siblings, 1 reply; 2+ messages in thread
From: Hans de Goede @ 2017-07-11 11:41 UTC (permalink / raw)
To: Darren Hart, Andy Shevchenko
Cc: Hans de Goede, platform-driver-x86, linux-kernel
If peaq_ignore_events_counter gets set to 1 we should skip polling 1
time, rather then ignoring it.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/platform/x86/peaq-wmi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/peaq-wmi.c b/drivers/platform/x86/peaq-wmi.c
index 23942d5b3525..6e6a3e650e1b 100644
--- a/drivers/platform/x86/peaq-wmi.c
+++ b/drivers/platform/x86/peaq-wmi.c
@@ -50,7 +50,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 >= 0)
return;
if (obj.integer.value) {
--
2.13.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] platform/x86: peaq-wmi: Fix peaq_ignore_events_counter handling off by 1
2017-07-11 11:41 [PATCH] platform/x86: peaq-wmi: Fix peaq_ignore_events_counter handling off by 1 Hans de Goede
@ 2017-07-11 16:13 ` Darren Hart
0 siblings, 0 replies; 2+ messages in thread
From: Darren Hart @ 2017-07-11 16:13 UTC (permalink / raw)
To: Hans de Goede; +Cc: Andy Shevchenko, platform-driver-x86, linux-kernel
On Tue, Jul 11, 2017 at 01:41:34PM +0200, Hans de Goede wrote:
> If peaq_ignore_events_counter gets set to 1 we should skip polling 1
> time, rather then ignoring it.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Thanks Hans, queued to testing.
--
Darren Hart
VMware Open Source Technology Center
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-11 16:13 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-11 11:41 [PATCH] platform/x86: peaq-wmi: Fix peaq_ignore_events_counter handling off by 1 Hans de Goede
2017-07-11 16:13 ` Darren Hart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox