From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH] input: cypress_ps2: Don't report the cypress PS/2 trackpads as a button pad Date: Fri, 28 Mar 2014 01:01:27 -0700 Message-ID: <20140328080127.GF22093@core.coreip.homeip.net> References: <1395668975-10588-1-git-send-email-hdegoede@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pb0-f52.google.com ([209.85.160.52]:35743 "EHLO mail-pb0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751038AbaC1IBb (ORCPT ); Fri, 28 Mar 2014 04:01:31 -0400 Received: by mail-pb0-f52.google.com with SMTP id rr13so4662590pbb.39 for ; Fri, 28 Mar 2014 01:01:30 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1395668975-10588-1-git-send-email-hdegoede@redhat.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Hans de Goede Cc: Benjamin Tissoires , linux-input@vger.kernel.org, Adam Williamson , Peter Hutterer On Mon, Mar 24, 2014 at 02:49:35PM +0100, Hans de Goede wrote: > 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 > Tested-by: Adam Williamson > Reviewed-by: Peter Hutterer > Cc: Adam Williamson > Cc: Peter Hutterer > Signed-off-by: Hans de Goede Applied, thank you. > --- > drivers/input/mouse/cypress_ps2.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/input/mouse/cypress_ps2.c b/drivers/input/mouse/cypress_ps2.c > index 87095e2..8af34ff 100644 > --- a/drivers/input/mouse/cypress_ps2.c > +++ b/drivers/input/mouse/cypress_ps2.c > @@ -409,7 +409,6 @@ static int cypress_set_input_params(struct input_dev *input, > __clear_bit(REL_X, input->relbit); > __clear_bit(REL_Y, input->relbit); > > - __set_bit(INPUT_PROP_BUTTONPAD, input->propbit); > __set_bit(EV_KEY, input->evbit); > __set_bit(BTN_LEFT, input->keybit); > __set_bit(BTN_RIGHT, input->keybit); > -- > 1.9.0 > -- Dmitry