linux-kernel - Re: [PATCH] Input: synaptic - reverting dcb00fc799dc03fd320e123e4c81b3278c763ea5 because it breaks the touchpad for one guy on Reddit.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 7 Feb 2021 16:41:20 +0100
From:   "Jason A. Donenfeld" <Jason@...c4.com>
To:     colton@...thsoftware.ca
Cc:     Dmitry Torokhov <dmitry.torokhov@...il.com>, colton@...thlinux.ca,
        Lyude Paul <lyude@...hat.com>,
        Hans de Goede <hdegoede@...hat.com>,
        Lee Jones <lee.jones@...aro.org>,
        Vincent Huang <vincent.huang@...synaptics.com>,
        Dennis Kadioglu <denk@...ipso.email>,
        Yussuf Khalil <dev@...345.net>, Ilya Katsnelson <me@...ti.me>,
        Gaurav Agrawal <agrawalgaurav@...me.org>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        linux-input@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Input: synaptic - reverting dcb00fc799dc03fd320e123e4c81b3278c763ea5
 because it breaks the touchpad for one guy on Reddit.

On Sun, Feb 7, 2021 at 5:00 AM Colton Booth <colton@...thsoftware.ca> wrote:
>
> I can't test myself since I don't have the correct hardware, BUT this change seems to work for him. I'm thinking he has an early version of the X1E which may use slightly different trackpad revision.
>
> Signed-off-by: Colton Booth <colton@...thsoftware.ca>
> ---
>  drivers/input/mouse/synaptics.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
> index ffad142801b3..2d3f03921dbc 100644
> --- a/drivers/input/mouse/synaptics.c
> +++ b/drivers/input/mouse/synaptics.c
> @@ -179,9 +179,7 @@ static const char * const smbus_pnp_ids[] = {
>         "LEN0093", /* T480 */
>         "LEN0096", /* X280 */
>         "LEN0097", /* X280 -> ALPS trackpoint */
> -       "LEN0099", /* X1 Extreme Gen 1 / P1 Gen 1 */
>         "LEN009b", /* T580 */
> -       "LEN0402", /* X1 Extreme Gen 2 / P1 Gen 2 */
>         "LEN200f", /* T450s */
>         "LEN2044", /* L470  */
>         "LEN2054", /* E480 */

This removes two totally separate models for "one guy on Reddit". Does
he have two computers that coincidentally have the same problem? Any
details about what that problem is so that we can assess it?

Jason