]> git.hungrycats.org Git - linux/commit
Input: cypress_ps2 - don't report as a button pads
authorHans de Goede <hdegoede@redhat.com>
Wed, 26 Mar 2014 20:30:52 +0000 (13:30 -0700)
committerJiri Slaby <jslaby@suse.cz>
Thu, 3 Apr 2014 08:32:25 +0000 (10:32 +0200)
commit6ed1a01c0d5512d66c04326cd4963ee1d871fa9c
tree857919b4a4fdca649d1149ef8c005edb3b2d0a6f
parent45b50b25b36538728f55f0a930b312af7fbb6e6f
Input: cypress_ps2 - don't report as a button pads

commit 6797b39e6f6f34c74177736e146406e894b9482b upstream.

The cypress PS/2 trackpad models supported by the cypress_ps2 driver
emulate BTN_RIGHT events in firmware based on the finger position, as part
of this no motion events are sent when the finger is in the button area.

The INPUT_PROP_BUTTONPAD property is there to indicate to userspace that
BTN_RIGHT events should be emulated in userspace, which is not necessary
in this case.

When INPUT_PROP_BUTTONPAD is advertised userspace will wait for a motion
event before propagating the button event higher up the stack, as it needs
current abs x + y data for its BTN_RIGHT emulation. Since in the
cypress_ps2 pads don't report motion events in the button area, this means
that clicks in the button area end up being ignored, so
INPUT_PROP_BUTTONPAD actually causes problems for these touchpads, and
removing it fixes:

https://bugs.freedesktop.org/show_bug.cgi?id=76341

Reported-by: Adam Williamson <awilliam@redhat.com>
Tested-by: Adam Williamson <awilliam@redhat.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
drivers/input/mouse/cypress_ps2.c