diff options
author | Ping Cheng <pinglinux@gmail.com> | 2016-06-23 13:55:11 -0400 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2016-06-23 16:38:45 -0400 |
commit | 9e72ac7492149a229ce9039c680849cb682d7092 (patch) | |
tree | 9cb12623238ce9b1ae7cdee435f2aa3aeaed1594 | |
parent | 12afb34400eb2b301f06b2aa3535497d14faee59 (diff) |
Input: wacom_w8001 - ignore invalid pen data packets
ThinkPad X60 Tablet PC (pen only device) sometime posts
packets that are larger than W8001_PKTLEN_TPCPEN.
Reported-by: Chris J Arges <christopherarges@gmail.com>
Tested-by: Chris J Arges <christopherarges@gmail.com>
Signed-off-by: Ping Cheng <pingc@wacom.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Cc: stable@vger.kernel.org
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
-rw-r--r-- | drivers/input/touchscreen/wacom_w8001.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/input/touchscreen/wacom_w8001.c b/drivers/input/touchscreen/wacom_w8001.c index b1b412766da8..0c9191cf324d 100644 --- a/drivers/input/touchscreen/wacom_w8001.c +++ b/drivers/input/touchscreen/wacom_w8001.c | |||
@@ -339,6 +339,15 @@ static irqreturn_t w8001_interrupt(struct serio *serio, | |||
339 | w8001->idx = 0; | 339 | w8001->idx = 0; |
340 | parse_multi_touch(w8001); | 340 | parse_multi_touch(w8001); |
341 | break; | 341 | break; |
342 | |||
343 | default: | ||
344 | /* | ||
345 | * ThinkPad X60 Tablet PC (pen only device) sometimes | ||
346 | * sends invalid data packets that are larger than | ||
347 | * W8001_PKTLEN_TPCPEN. Let's start over again. | ||
348 | */ | ||
349 | if (!w8001->touch_dev && w8001->idx > W8001_PKTLEN_TPCPEN - 1) | ||
350 | w8001->idx = 0; | ||
342 | } | 351 | } |
343 | 352 | ||
344 | return IRQ_HANDLED; | 353 | return IRQ_HANDLED; |