diff options
author | Det | 2017-12-25 19:10:42 +0200 |
---|---|---|
committer | Det | 2017-12-25 19:10:42 +0200 |
commit | 02bc6ad4938114fa8b632800eb7912f34bd37e2a (patch) | |
tree | 7ee3d1152e2041484efb3a9b483a0772bd0f040a /udev-changes.diff | |
parent | d93dd0aaa9f26df38becc5945abd1d44a013da6b (diff) | |
download | aur-02bc6ad4938114fa8b632800eb7912f34bd37e2a.tar.gz |
Sync with [testing]
Diffstat (limited to 'udev-changes.diff')
-rw-r--r-- | udev-changes.diff | 105 |
1 files changed, 105 insertions, 0 deletions
diff --git a/udev-changes.diff b/udev-changes.diff new file mode 100644 index 000000000000..28490f90c2f7 --- /dev/null +++ b/udev-changes.diff @@ -0,0 +1,105 @@ +From 5a5b6d6cca469521daa6ac9087f3589b7489ab55 Mon Sep 17 00:00:00 2001 +From: Peter Hutterer <peter.hutterer@who-t.net> +Date: Tue, 26 Sep 2017 15:21:59 +1000 +Subject: config/udev: consider ID_INPUT_FOO=0 as 'unset' + +Historically we didn't need to care about this case but more devices are +having invalid types set and they cannot be unset with a hwdb entry (which +doesn't handle the empty string). Allow for "0" to mean "unset" because +anything else would be crazy anyway. + +Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> +(cherry picked from commit 5aad81445c8c3d6b7b30d503cfe26027fa482870) +--- + config/udev.c | 65 +++++++++++++++++++++++++++++------------------------------ + 1 file changed, 32 insertions(+), 33 deletions(-) + +(limited to 'config/udev.c') + +diff --git a/config/udev.c b/config/udev.c +index 932f230..e198e86 100644 +--- a/config/udev.c ++++ b/config/udev.c +@@ -134,7 +134,8 @@ device_added(struct udev_device *udev_device) + } + #endif + +- if (!udev_device_get_property_value(udev_device, "ID_INPUT")) { ++ value = udev_device_get_property_value(udev_device, "ID_INPUT"); ++ if (value && !strcmp(value, "0")) { + LogMessageVerb(X_INFO, 10, + "config/udev: ignoring device %s without " + "property ID_INPUT set\n", path); +@@ -237,38 +238,36 @@ device_added(struct udev_device *udev_device) + else if (!strcmp(key, "ID_VENDOR")) { + LOG_PROPERTY(path, key, value); + attrs.vendor = strdup(value); +- } +- else if (!strcmp(key, "ID_INPUT_KEY")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_KEY; +- } +- else if (!strcmp(key, "ID_INPUT_KEYBOARD")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_KEYBOARD; +- } +- else if (!strcmp(key, "ID_INPUT_MOUSE")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_POINTER; +- } +- else if (!strcmp(key, "ID_INPUT_JOYSTICK")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_JOYSTICK; +- } +- else if (!strcmp(key, "ID_INPUT_TABLET")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_TABLET; +- } +- else if (!strcmp(key, "ID_INPUT_TABLET_PAD")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_TABLET_PAD; +- } +- else if (!strcmp(key, "ID_INPUT_TOUCHPAD")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_TOUCHPAD; +- } +- else if (!strcmp(key, "ID_INPUT_TOUCHSCREEN")) { +- LOG_PROPERTY(path, key, value); +- attrs.flags |= ATTR_TOUCHSCREEN; ++ } else if (!strncmp(key, "ID_INPUT_", 9)) { ++ const struct pfmap { ++ const char *property; ++ unsigned int flag; ++ } map[] = { ++ { "ID_INPUT_KEY", ATTR_KEY }, ++ { "ID_INPUT_KEYBOARD", ATTR_KEYBOARD }, ++ { "ID_INPUT_MOUSE", ATTR_POINTER }, ++ { "ID_INPUT_JOYSTICK", ATTR_JOYSTICK }, ++ { "ID_INPUT_TABLET", ATTR_TABLET }, ++ { "ID_INPUT_TABLET_PAD", ATTR_TABLET_PAD }, ++ { "ID_INPUT_TOUCHPAD", ATTR_TOUCHPAD }, ++ { "ID_INPUT_TOUCHSCREEN", ATTR_TOUCHSCREEN }, ++ { NULL, 0 }, ++ }; ++ ++ /* Anything but the literal string "0" is considered a ++ * boolean true. The empty string isn't a thing with udev ++ * properties anyway */ ++ if (value && strcmp(value, "0")) { ++ const struct pfmap *m = map; ++ ++ while (m->property != NULL) { ++ if (!strcmp(m->property, key)) { ++ LOG_PROPERTY(path, key, value); ++ attrs.flags |= m->flag; ++ } ++ m++; ++ } ++ } + } + } + +-- +cgit v1.1 + + |