]> asedeno.scripts.mit.edu Git - linux.git/commit
HID: wacom: generic: Send BTN_STYLUS3 when both barrel switches are set
authorJason Gerecke <killertofu@gmail.com>
Tue, 7 Nov 2017 16:25:17 +0000 (08:25 -0800)
committerJiri Kosina <jkosina@suse.cz>
Thu, 9 Nov 2017 12:32:43 +0000 (13:32 +0100)
commit9e429d564926d3bca49907fa03031da705ad6f2c
tree591f242c7674ac2076d9e2a56139167ffbf1c5b1
parent2f84723de7cdd031f293b900fecd68ddbec3feaa
HID: wacom: generic: Send BTN_STYLUS3 when both barrel switches are set

The Wacom Pro Pen 3D includes a third barrel switch which is intended to
be particularly useful in applications where one frequency uses pan, zoom,
and rotate to navigate around a scene or model. The pen is compatible with
the MobileStudio Pro, 2nd-gen Intuos Pro, and Cintiq Pro. When the third
button is pressed, these devices set both the HID_DG_BARRELSWITCH and
HID_DG_BARRELSWITCH2 usages since their HID descriptors do not include a
usage specific to the button.

Rather than send both BTN_STYLUS and BTN_STYLUS2 when the third button is
pressed, userspace (libinput) has requested that we detect this condition
and report a newly-defined BTN_STYLUS3 event instead. We could define a
quirk specific to devices compatible with the Pro Pen 3D, but the liklihood
of seeing both barrel switch bits set with other pens/devices is low enough
to not worry about (pens mechanically prevent accidental activation of
multiple switches).

Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Acked-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/wacom_wac.c
drivers/hid/wacom_wac.h
include/uapi/linux/input-event-codes.h