On Wed, Jul 29, 2015 at 04:45:26PM -0400, [email protected] wrote:
> From: Stephen Chandler Paul <[email protected]>
> 
> The data concerning which buttons on the touchpad are held down or not
> are in the fourth packet we receive from the mouse, not the first.
> 
> Signed-off-by: Stephen Chandler Paul <[email protected]>

Let's also make sure it hoes not break Hans' (de Bruin) touchpad.

> ---
>  drivers/input/mouse/alps.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c
> index 113d6f1..e2f9b25 100644
> --- a/drivers/input/mouse/alps.c
> +++ b/drivers/input/mouse/alps.c
> @@ -254,9 +254,9 @@ static void alps_process_packet_v1_v2(struct psmouse 
> *psmouse)
>       /* Non interleaved V2 dualpoint has separate stick button bits */
>       if (priv->proto_version == ALPS_PROTO_V2 &&
>           priv->flags == (ALPS_PASS | ALPS_DUALPOINT)) {
> -             left |= packet[0] & 1;
> -             right |= packet[0] & 2;
> -             middle |= packet[0] & 4;
> +             left |= packet[3] & 1;
> +             right |= packet[3] & 2;
> +             middle |= packet[3] & 4;
>       }
>  
>       alps_report_buttons(dev, dev2, left, right, middle);
> -- 
> 2.4.3
> 

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to