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>

diff --git a/src/evdev.c b/src/evdev.c
index 560412a..0d8e876 100644 (file)
--- 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;
        }