summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Hutterer <peter.hutterer@who-t.net>2020-07-14 13:04:25 +1000
committerPeter Hutterer <peter.hutterer@who-t.net>2020-07-15 03:24:44 +0000
commitb9ec408872cc4764d404d7b9057802ae476266bc (patch)
treef5dfde54439318ae07df40c8dbb0fa184285143e
parenteededbeb7faa2b6182c02a84884f54c1cb1d8a2a (diff)
downloadlibinput-b9ec408872cc4764d404d7b9057802ae476266bc.tar.gz
evdev: drop the check for tablet vs joystick
libwacom has been unsetting ID_INPUT_JOYSTICK for known tablets since 2015 (libwacom 0.12) so this comment is outdated. And the input-id udev builtin never labels something as tablet *and* joystick. Which means: systemd sets either tablet or joystick. For tablets that are known to libwacom the joystick bit gets corrected and we only see the tablet bits. Tablets unknown to libwacom remain as joysticks and are ignored but that's the behavior we had anyway. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r--src/evdev.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/src/evdev.c b/src/evdev.c
index 560412aa..0d8e8766 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -1810,11 +1810,7 @@ evdev_configure_device(struct evdev_device *device)
evdev_disable_accelerometer_axes(device);
}
- /* libwacom *adds* TABLET, TOUCHPAD but leaves JOYSTICK in place, so
- make sure we only ignore real joystick devices */
- const unsigned int joystick_tag = EVDEV_UDEV_TAG_JOYSTICK;
- const unsigned int joystick_or_tablet_tag = EVDEV_UDEV_TAG_JOYSTICK|EVDEV_UDEV_TAG_TABLET;
- if ((udev_tags & joystick_or_tablet_tag) == joystick_tag) {
+ if (udev_tags & EVDEV_UDEV_TAG_JOYSTICK) {
evdev_log_info(device, "device is a joystick, ignoring\n");
return NULL;
}