Bug 43728 - Regression on powerpc between 1.4.x and 1.5.0
Summary: Regression on powerpc between 1.4.x and 1.5.0
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Input/synaptics (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Peter Hutterer
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-11 16:46 UTC by Cyril Brulebois
Modified: 2011-12-11 18:00 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Proposed patch, reverting the guilty commit. (4.44 KB, patch)
2011-12-11 16:46 UTC, Cyril Brulebois
no flags Details | Splinter Review

Description Cyril Brulebois 2011-12-11 16:46:21 UTC
Created attachment 54331 [details] [review]
Proposed patch, reverting the guilty commit.

(Love it when that happens…)

The following commit broke synaptics on powerpc (e.g. on iBook G4 with appletouch):
---
commit 13543b156d78bc4d01a19844a5ee8f283269621b
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Sun Mar 20 09:28:45 2011 +1000

    eventcomm: replace synaptics-custom TEST_BIT with server's BitIsOn.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Cyril Brulebois <kibi@debian.org>
---

This was reported in: http://bugs.debian.org/648488

I was also able to reproduce it on such a machine, and got that after this commit:
---
$ grep appletouch /var/log/Xorg.0.log
[   828.530] (II) config/udev: Adding input device appletouch (/dev/input/event5)
[   828.530] (**) appletouch: Applying InputClass "evdev touchpad catchall"
[   828.530] (**) appletouch: Applying InputClass "touchpad catchall"
[   828.531] (II) Using input driver 'synaptics' for 'appletouch'
[   828.531] (**) appletouch: always reports core events
[   828.532] (EE) PreInit returned 11 for "appletouch"
[   828.532] (II) config/udev: Adding input device appletouch (/dev/input/mouse1)
[   828.532] (**) appletouch: Applying InputClass "touchpad catchall"
[   828.533] (II) Using input driver 'synaptics' for 'appletouch'
[   828.533] (**) appletouch: always reports core events
[   828.533] (EE) PreInit returned 11 for "appletouch"
---

as opposed to:
---
$ grep appletouch /var/log/Xorg.0.log
[   730.391] (II) config/udev: Adding input device appletouch (/dev/input/event5)
[   730.391] (**) appletouch: Applying InputClass "evdev touchpad catchall"
[   730.391] (**) appletouch: Applying InputClass "touchpad catchall"
[   730.392] (II) Using input driver 'synaptics' for 'appletouch'
[   730.392] (**) appletouch: always reports core events
[   730.392] (--) appletouch: x-axis range 0 - 959
[   730.392] (--) appletouch: y-axis range 0 - 644
[   730.392] (--) appletouch: pressure range 0 - 300
[   730.392] (--) appletouch: buttons: left double triple
[   730.392] (--) appletouch: invalid finger width range.  defaulting to 0 - 16
[   730.392] (--) appletouch: touchpad found
[   730.392] (**) appletouch: always reports core events
[   730.392] (II) XINPUT: Adding extended input device "appletouch" (type: TOUCHPAD, id 8)
[   730.393] (**) appletouch: (accel) MinSpeed is now constant deceleration 2.5
[   730.393] (**) appletouch: MaxSpeed is now 1.75
[   730.393] (**) appletouch: AccelFactor is now 0.173
[   730.393] (**) appletouch: (accel) keeping acceleration scheme 1
[   730.393] (**) appletouch: (accel) acceleration profile 1
[   730.393] (**) appletouch: (accel) acceleration factor: 2.000
[   730.393] (**) appletouch: (accel) acceleration threshold: 4
[   730.393] (--) appletouch: touchpad found
[   730.394] (II) config/udev: Adding input device appletouch (/dev/input/mouse1)
[   730.394] (**) appletouch: Applying InputClass "touchpad catchall"
[   730.394] (II) Using input driver 'synaptics' for 'appletouch'
[   730.394] (**) appletouch: always reports core events
[   730.394] (--) appletouch: invalid x-axis range.  defaulting to 1615 - 5685
[   730.394] (--) appletouch: invalid y-axis range.  defaulting to 1729 - 4171
[   730.394] (--) appletouch: invalid pressure range.  defaulting to 0 - 256
[   730.394] (--) appletouch: invalid finger width range.  defaulting to 0 - 16
[   730.415] (--) appletouch: no supported touchpad found
[   730.415] (EE) appletouch Unable to query/initialize Synaptics hardware.
[   730.415] (EE) PreInit returned 11 for "appletouch"
---

Going to that commit's parent makes it work again, as well as reverting it on top of the 1.5.0 tag. I haven't looked at architecture-specific fun yet (endianness comes to mind), but it might be a good idea to just revert this patch for now and issue a 1.5.1 release, until we have figured this out.

I'm attaching the patch I crafted on top of 1.5.0. I will also send it to the list with an extra bugzilla reference pointing to this report.
Comment 1 Peter Hutterer 2011-12-11 17:21:52 UTC
commit b7e65f04f5f0c17ac8a26393134cc7e8418ccdec
Refs: xf86-input-synaptics-1.4.99.1-35-gb7e65f0

thanks
Comment 2 Cyril Brulebois 2011-12-11 18:00:25 UTC
Sorry for the URL mix-up. I meant to point at Debian #649002 (http://bugs.debian.org/649002), aka. “after upgrading to latest version, iBook touchpad stopped working”.

Feel free to fix that up if you're cherry-picking it somewhere.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.