On September 29, 2015 6:46:56 AM PDT, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>3.10-stable review patch. If anyone has any objections, please let me
>know.
Please drop.
>
>------------------
>
>From: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
>
>commit e51e38494a8ecc18650efb0c840600637891de2c upstream.
>
>Bit 2 of the mode byte has dual meaning: it can disable reporting of
>gestures when touchpad works in Relative mode or normal Absolute mode,
>or it can enable so called Extended W-Mode when touchpad uses enhanced
>Absolute mode (W-mode). The extended W-Mode confuses our driver and
>causes missing button presses on some Thinkpads (x250, T450s), so let's
>make sure we do not enable it.
>
>Also, according to the spec W mode "... bit is defined only in Absolute
>mode on pads whose capExtended capability bit is set. In Relative mode
>and
>in TouchPads without this capability, the bit is reserved and should be
>left at 0.", so let's make sure we respect this requirement as well.
>
>Reported-by: Nick Bowler <nbowler@xxxxxxxxxx>
>Suggested-by: Gabor Balla <gaborwho@xxxxxxxxx>
>Tested-by: Gabor Balla <gaborwho@xxxxxxxxx>
>Tested-by: Nick Bowler <nbowler@xxxxxxxxxx>
>Signed-off-by: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
>Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>
>---
> drivers/input/mouse/synaptics.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
>--- a/drivers/input/mouse/synaptics.c
>+++ b/drivers/input/mouse/synaptics.c
>@@ -360,14 +360,18 @@ static int synaptics_set_mode(struct psm
> struct synaptics_data *priv = psmouse->private;
>
> priv->mode = 0;
>- if (priv->absolute_mode)
>+
>+ if (priv->absolute_mode) {
> priv->mode |= SYN_BIT_ABSOLUTE_MODE;
>- if (priv->disable_gesture)
>+ if (SYN_CAP_EXTENDED(priv->capabilities))
>+ priv->mode |= SYN_BIT_W_MODE;
>+ }
>+
>+ if (!SYN_MODE_WMODE(priv->mode) && priv->disable_gesture)
> priv->mode |= SYN_BIT_DISABLE_GESTURE;
>+
> if (psmouse->rate >= 80)
> priv->mode |= SYN_BIT_HIGH_RATE;
>- if (SYN_CAP_EXTENDED(priv->capabilities))
>- priv->mode |= SYN_BIT_W_MODE;
>
> if (synaptics_mode_cmd(psmouse, priv->mode))
> return -1;
Thanks.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html