From 46d9536c2f76a7d798e531f2a80988b5f13c191d Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Thu, 22 Jun 2017 09:19:32 +1000 Subject: [PATCH] udev: always consider devices with joystick axes/buttons as joystick Even when they don't have an x/y axis. https://github.com/systemd/systemd/issues/6137 Signed-off-by: Peter Hutterer --- src/udev/udev-builtin-input_id.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/udev/udev-builtin-input_id.c b/src/udev/udev-builtin-input_id.c index 7ea86b21b..47f41a660 100644 --- a/src/udev/udev-builtin-input_id.c +++ b/src/udev/udev-builtin-input_id.c @@ -204,7 +204,7 @@ static bool test_pointers(struct udev_device *dev, has_touch = test_bit(BTN_TOUCH, bitmask_key); /* joysticks don't necessarily have buttons; e. g. * rudders/pedals are joystick-like, but buttonless; they have - * other fancy axes */ + * other fancy axes. Others have buttons only but no axes. */ has_joystick_axes_or_buttons = test_bit(BTN_TRIGGER, bitmask_key) || test_bit(BTN_TRIGGER_HAPPY, bitmask_key) || test_bit(BTN_A, bitmask_key) || @@ -231,7 +231,10 @@ static bool test_pointers(struct udev_device *dev, is_touchscreen = true; else if (has_joystick_axes_or_buttons) is_joystick = true; + } else if (has_joystick_axes_or_buttons) { + is_joystick = true; } + if (has_mt_coordinates) { if (stylus_or_pen) is_tablet = true;