summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorPing Cheng <pinglinux@gmail.com>2013-11-25 21:43:45 -0500
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2013-11-25 22:03:37 -0500
commit1d0d6df02750b4a6f466768cbfbf860e24f4c8d4 (patch)
tree5212ea26707c1914ed64bcc1311d5f863b8e3bbd /drivers
parent0b279da7af779fa515b70c0f4127001cab22ea86 (diff)
Input: wacom - make sure touch_max is set for touch devices
Old single touch Tablet PCs do not have touch_max set at wacom_features. Since touch device at lease supports one finger, assign touch_max to 1 when touch usage is defined in its HID Descriptor and touch_max is not pre-defined. Tested-by: Jason Gerecke <killertofu@gmail.com> Signed-off-by: Ping Cheng <pingc@wacom.com> Reviewed-by: Chris Bagwell <chris@cnpbagwell.com> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/input/tablet/wacom_sys.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/input/tablet/wacom_sys.c b/drivers/input/tablet/wacom_sys.c
index 8a90da11365f..3d71b608330e 100644
--- a/drivers/input/tablet/wacom_sys.c
+++ b/drivers/input/tablet/wacom_sys.c
@@ -304,7 +304,7 @@ static int wacom_parse_hid(struct usb_interface *intf,
304 struct usb_device *dev = interface_to_usbdev(intf); 304 struct usb_device *dev = interface_to_usbdev(intf);
305 char limit = 0; 305 char limit = 0;
306 /* result has to be defined as int for some devices */ 306 /* result has to be defined as int for some devices */
307 int result = 0; 307 int result = 0, touch_max = 0;
308 int i = 0, usage = WCM_UNDEFINED, finger = 0, pen = 0; 308 int i = 0, usage = WCM_UNDEFINED, finger = 0, pen = 0;
309 unsigned char *report; 309 unsigned char *report;
310 310
@@ -351,7 +351,8 @@ static int wacom_parse_hid(struct usb_interface *intf,
351 if (usage == WCM_DESKTOP) { 351 if (usage == WCM_DESKTOP) {
352 if (finger) { 352 if (finger) {
353 features->device_type = BTN_TOOL_FINGER; 353 features->device_type = BTN_TOOL_FINGER;
354 354 /* touch device at least supports one touch point */
355 touch_max = 1;
355 switch (features->type) { 356 switch (features->type) {
356 case TABLETPC2FG: 357 case TABLETPC2FG:
357 features->pktlen = WACOM_PKGLEN_TPC2FG; 358 features->pktlen = WACOM_PKGLEN_TPC2FG;
@@ -504,6 +505,8 @@ static int wacom_parse_hid(struct usb_interface *intf,
504 } 505 }
505 506
506 out: 507 out:
508 if (!features->touch_max && touch_max)
509 features->touch_max = touch_max;
507 result = 0; 510 result = 0;
508 kfree(report); 511 kfree(report);
509 return result; 512 return result;