Package: udev Version: 175-7 Severity: normal Tags: patch Hi, in order to determine the orientation of the device it picks the wrong device. It picked the "Asus Laptop extra buttons" instead of the accelerometer. This is a straight backport from upstream commit 1d010426c01044350b2b32d8b3af5d064f7dfe27.
It would be awesome to have this in an updated version of udev. Cheers, -- Guido -- System Information: Debian Release: 7.0 APT prefers testing APT policy: (990, 'testing'), (50, 'unstable'), (1, 'experimental') Architecture: i386 (i686) Kernel: Linux 3.2.0-4-686-pae (SMP w/2 CPU cores) Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages udev depends on: ii debconf [debconf-2.0] 1.5.49 ii libc6 2.13-37 ii libselinux1 2.1.9-5 ii libudev0 175-7 ii lsb-base 4.1+Debian8 ii procps 1:3.3.3-2 ii util-linux 2.20.1-5.3 Versions of packages udev recommends: ii pciutils 1:3.1.9-6 ii usbutils 1:005-3 udev suggests no packages. -- debconf information excluded
>From 701b112723dabd7ffb9a71cacfd45d0545bffa2b Mon Sep 17 00:00:00 2001 From: Guido Günther <a...@sigxcpu.org> Date: Sat, 23 Feb 2013 09:05:23 +0100 Subject: [PATCH] Fix device naming in the accelerometer --- .../fix_device_matching_in_the_accelerometer | 37 ++++++++++++++++++++ debian/patches/series | 1 + 2 files changed, 38 insertions(+) create mode 100644 debian/patches/fix_device_matching_in_the_accelerometer diff --git a/debian/patches/fix_device_matching_in_the_accelerometer b/debian/patches/fix_device_matching_in_the_accelerometer new file mode 100644 index 0000000..1954afe --- /dev/null +++ b/debian/patches/fix_device_matching_in_the_accelerometer @@ -0,0 +1,37 @@ +From: Guido Günther <a...@sigxcpu.org> +Date: Sat, 23 Feb 2013 08:45:50 +0100 +Subject: Fix device matching in the accelerometer + +As we were searching by ID_PATH, it would have been possible +for us to find a sibling device instead of the device we were +looking for. + +This fixes device matching on the WeTab with the upstream kernel, +as it was trying to use the "Asus Laptop extra buttons" device +instead of the accelerometer. + +Backported version of 1d010426c01044350b2b32d8b3af5d064f7dfe27 +--- + extras/accelerometer/accelerometer.c | 7 ++----- + 1 file changed, 2 insertions(+), 5 deletions(-) + +diff --git a/extras/accelerometer/accelerometer.c b/extras/accelerometer/accelerometer.c +index 59c2a4e..b8de3e0 100644 +--- a/extras/accelerometer/accelerometer.c ++++ b/extras/accelerometer/accelerometer.c +@@ -307,13 +307,10 @@ int main (int argc, char** argv) + return 0; + } + +- /* Get the children devices and find the devnode +- * FIXME: use udev_enumerate_add_match_children() instead +- * when it's available */ ++ /* Get the children devices and find the devnode */ + devnode = NULL; + enumerate = udev_enumerate_new(udev); +- udev_enumerate_add_match_property(enumerate, "ID_PATH", id_path); +- udev_enumerate_add_match_subsystem(enumerate, "input"); ++ udev_enumerate_add_match_parent(enumerate, dev); + udev_enumerate_scan_devices(enumerate); + udev_list_entry_foreach(list_entry, udev_enumerate_get_list_entry(enumerate)) { + struct udev_device *device; diff --git a/debian/patches/series b/debian/patches/series index ddd85b2..dfe9f0d 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -23,3 +23,4 @@ use_run_tmpfs dev_root_rule udevd_in_sbin udev_conf_comments +fix_device_matching_in_the_accelerometer -- 1.7.10.4