diff options
| author | Andrew Shadura <andrew.shadura@collabora.co.uk> | 2015-10-29 15:09:49 (GMT) |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2016-03-10 16:17:26 (GMT) |
| commit | 5f66872cbd9c2d8e5230da571aed52d9333a04ae (patch) | |
| tree | d7188c14cda49906ae842ea13bceabac2801c2b1 | |
| parent | c2848f2eef4dd08b0fd2a8eba1694fd8e77ddb67 (diff) | |
| download | linux-5f66872cbd9c2d8e5230da571aed52d9333a04ae.tar.xz | |
HID: penmount: report only one button for PenMount 6000 USB touchscreen controller
PenMount 6000 USB resistive touchscreen controller reports it has three
buttons, while in reality it doesn't have any and doesn't support active
styli, and only generates touch events.
In penmount_input_mapping(), map only the first button (with code 0),
ignore the rest.
Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
Tested-by: Christian Gmeiner <c.gmeiner@bachmann.info
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/hid/hid-penmount.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/hid/hid-penmount.c b/drivers/hid/hid-penmount.c index c11dce8..d90383f 100644 --- a/drivers/hid/hid-penmount.c +++ b/drivers/hid/hid-penmount.c @@ -23,8 +23,12 @@ static int penmount_input_mapping(struct hid_device *hdev, struct hid_usage *usage, unsigned long **bit, int *max) { if ((usage->hid & HID_USAGE_PAGE) == HID_UP_BUTTON) { - hid_map_usage(hi, usage, bit, max, EV_KEY, BTN_TOUCH); - return 1; + if (((usage->hid - 1) & HID_USAGE) == 0) { + hid_map_usage(hi, usage, bit, max, EV_KEY, BTN_TOUCH); + return 1; + } else { + return -1; + } } return 0; |