Bug 45663

Summary: Make Apple Magic Trackpad work with touch events
Product: xorg Reporter: Benjamin Otte <otte>
Component: Input/synapticsAssignee: Peter Hutterer <peter.hutterer>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: chase.douglas, peter.hutterer
Version: git   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
eventcomm: Fix initialization code
none
eventcomm: Make code work without custom valuator none

Description Benjamin Otte 2012-02-05 18:34:21 UTC
Here's the patching I did to make my trackpad work. Without them, the mouse and touch events would randomly report wrong coordinates and make everything unusable.
Comment 1 Benjamin Otte 2012-02-05 18:37:51 UTC
Created attachment 56637 [details] [review]
eventcomm: Fix initialization code
Comment 2 Benjamin Otte 2012-02-05 18:37:55 UTC
Created attachment 56638 [details] [review]
eventcomm: Make code work without custom valuator

Instead, always use the valuator for the current touchpoint directly.

Fixes crazy pointer jumps with Apple Magic Trackpad.
Comment 3 Peter Hutterer 2012-02-06 06:28:23 UTC
First patch looks good, though i've expanded the commit message a bit. Second patch I'll defer for Chase for now.
Comment 4 Chase Douglas 2012-02-06 08:22:07 UTC
I agree on the first patch.

The second patch removes the functionality that ensures unchanged values are not sent in XI 2.x events. It is likely just papering over bad behavior in the server or client.
Comment 5 Peter Hutterer 2012-02-08 06:56:27 UTC
Comment on attachment 56637 [details] [review]
eventcomm: Fix initialization code

Review of attachment 56637 [details] [review]:
-----------------------------------------------------------------

commit da0d20d0fd5cce3707b54146e54e2df0c697e1b4
Author: Benjamin Otte <otte@redhat.com>
Date:   Mon Feb 6 00:45:23 2012 +0100

    eventcomm: Fix initialization code
Comment 6 Chase Douglas 2012-02-08 07:43:21 UTC
Comment on attachment 56637 [details] [review]
eventcomm: Fix initialization code

Review of attachment 56637 [details] [review]:
-----------------------------------------------------------------

I'm not sure if I need to make a review here, but just to be sure:

Reviewed-by: Chase Douglas <chase.douglas@canonical.com>
Comment 7 Peter Hutterer 2012-03-13 18:50:32 UTC
commit da0d20d0fd5cce3707b54146e54e2df0c697e1b4
Author: Benjamin Otte <otte@redhat.com>
Date:   Mon Feb 6 00:45:23 2012 +0100

    eventcomm: Fix initialization code

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.