Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1715394 > unrolled thread

[PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices should use RMI

Started byDmitry Torokhov <dmitry.torokhov@gmail.com>
First post2017-08-18 21:40 +0200
Last post2017-08-21 18:30 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices  should use RMI Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2017-08-18 21:40 +0200
    Re: [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices  should use RMI Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2017-08-21 09:20 +0200
    Re: [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices  should use RMI Damjan Georgievski <gdamjan@gmail.com> - 2017-08-21 11:00 +0200
      Re: [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices  should use RMI Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2017-08-21 17:50 +0200
        Re: [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices  should use RMI Damjan Georgievski <gdamjan@gmail.com> - 2017-08-21 18:30 +0200

#1715394 — [PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices should use RMI

FromDmitry Torokhov <dmitry.torokhov@gmail.com>
Date2017-08-18 21:40 +0200
Subject[PATCH] Input: synaptics - Lenovo Carbon X1 Gen5 (2017) devices should use RMI
Message-ID<ufVhE-82H-11@gated-at.bofh.it>
The touchpad on Lenovo Carbon X1 Gen 5 (2017 - Kabylake) is accessible over
SMBUS/RMI, so let's activate it by default.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
 drivers/input/mouse/synaptics.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
index 9a8d81efd329..248c7c9dcbaa 100644
--- a/drivers/input/mouse/synaptics.c
+++ b/drivers/input/mouse/synaptics.c
@@ -172,6 +172,7 @@ static const char * const smbus_pnp_ids[] = {
 	"LEN0048", /* X1 Carbon 3 */
 	"LEN0046", /* X250 */
 	"LEN004a", /* W541 */
+	"LEN0072", /* X1 Carbon Gen 5 (2017) */
 	"LEN200f", /* T450s */
 	NULL
 };
-- 
2.14.1.480.gb18f417b89-goog


-- 
Dmitry

[toc] | [next] | [standalone]


#1716122

FromBenjamin Tissoires <benjamin.tissoires@redhat.com>
Date2017-08-21 09:20 +0200
Message-ID<ugPaa-1f1-9@gated-at.bofh.it>
In reply to#1715394
On Aug 18 2017 or thereabouts, Dmitry Torokhov wrote:
> The touchpad on Lenovo Carbon X1 Gen 5 (2017 - Kabylake) is accessible over
> SMBUS/RMI, so let's activate it by default.
> 
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> ---

Reviewed-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>

Cheers,
Benjamin

>  drivers/input/mouse/synaptics.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
> index 9a8d81efd329..248c7c9dcbaa 100644
> --- a/drivers/input/mouse/synaptics.c
> +++ b/drivers/input/mouse/synaptics.c
> @@ -172,6 +172,7 @@ static const char * const smbus_pnp_ids[] = {
>  	"LEN0048", /* X1 Carbon 3 */
>  	"LEN0046", /* X250 */
>  	"LEN004a", /* W541 */
> +	"LEN0072", /* X1 Carbon Gen 5 (2017) */
>  	"LEN200f", /* T450s */
>  	NULL
>  };
> -- 
> 2.14.1.480.gb18f417b89-goog
> 
> 
> -- 
> Dmitry

[toc] | [prev] | [next] | [standalone]


#1716210

FromDamjan Georgievski <gdamjan@gmail.com>
Date2017-08-21 11:00 +0200
Message-ID<ugQIW-25q-17@gated-at.bofh.it>
In reply to#1715394
On 18 August 2017 at 21:34, Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> The touchpad on Lenovo Carbon X1 Gen 5 (2017 - Kabylake) is accessible over
> SMBUS/RMI, so let's activate it by default.

the patch over latest Linus 4.13-rc5 makes the trackpoint device to
not appear after boot.

Sometimes it  spontaneuosly appears 100-200 seconds after boot.

It does appear after suspend/resume. And it does appear after `rmmod
psmouse; modprobe psmouse`.


Second issues is, trackpoint is reported as "ImPS/2 Generic Wheel
Mouse". Middle button scrolling is not enabled by default.

hardware is:
Lenovo ThinkPad X1 Carbon 5th, model 20HQS0LV00
DMI: LENOVO 20HQS0LV00/20HQS0LV00, BIOS N1MET37W (1.22 ) 07/04/2017


-- 
damjan

[toc] | [prev] | [next] | [standalone]


#1716624

FromDmitry Torokhov <dmitry.torokhov@gmail.com>
Date2017-08-21 17:50 +0200
Message-ID<ugX7H-6aN-5@gated-at.bofh.it>
In reply to#1716210
On Mon, Aug 21, 2017 at 10:52:11AM +0200, Damjan Georgievski wrote:
> On 18 August 2017 at 21:34, Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> > The touchpad on Lenovo Carbon X1 Gen 5 (2017 - Kabylake) is accessible over
> > SMBUS/RMI, so let's activate it by default.
> 
> the patch over latest Linus 4.13-rc5 makes the trackpoint device to
> not appear after boot.
> 
> Sometimes it  spontaneuosly appears 100-200 seconds after boot.
> 
> It does appear after suspend/resume. And it does appear after `rmmod
> psmouse; modprobe psmouse`.

Hmm, I do not observe this on:

Handle 0x000B, DMI type 0, 24 bytes
BIOS Information
        Vendor: LENOVO
        Version: N1MET37W (1.22 )
        Release Date: 07/04/2017
...
        BIOS Revision: 1.22
        Firmware Revision: 1.15

Handle 0x000C, DMI type 1, 27 bytes
System Information
        Manufacturer: LENOVO
        Product Name: 20HQS0EG02
        Version: ThinkPad X1 Carbon 5th
...
        SKU Number: LENOVO_MT_20HQ_BU_Think_FM_ThinkPad X1 Carbon 5th
        Family: ThinkPad X1 Carbon 5th

I wonder if we have different configs...

> 
> 
> Second issues is, trackpoint is reported as "ImPS/2 Generic Wheel
> Mouse". Middle button scrolling is not enabled by default.

I think this will be fixed by:

https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/patch/?id=ec667683c532c93fb41e100e5d61a518971060e2

> 
> hardware is:
> Lenovo ThinkPad X1 Carbon 5th, model 20HQS0LV00
> DMI: LENOVO 20HQS0LV00/20HQS0LV00, BIOS N1MET37W (1.22 ) 07/04/2017
> 

Thanks.

-- 
Dmitry

[toc] | [prev] | [next] | [standalone]


#1716699

FromDamjan Georgievski <gdamjan@gmail.com>
Date2017-08-21 18:30 +0200
Message-ID<ugXKq-6Fz-19@gated-at.bofh.it>
In reply to#1716624
> I wonder if we have different configs...
>
>>
>>
>> Second issues is, trackpoint is reported as "ImPS/2 Generic Wheel
>> Mouse". Middle button scrolling is not enabled by default.
>
> I think this will be fixed by:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/patch/?id=ec667683c532c93fb41e100e5d61a518971060e2
>

Interesting, with this patch, the trackpoint appeared sooner, but
still late (at the 44 second mark):

$ dmesg | grep -i -E '(mouse|syna)'
[    5.338111] input: PixArt Dell MS116 USB Optical Mouse as
/devices/pci0000:00/0000:00:1d.0/0000:06:00.0/0000:07:01.0/0000:09:00.0/0000:0a:00.0/0000:0b:00.0/usb3/3-3/3-3.2/3-3.2:1.0/0003:413C:301A.0005/input/input9
[    5.338221] hid-generic 0003:413C:301A.0005: input,hidraw4: USB HID
v1.11 Mouse [PixArt Dell MS116 USB Optical Mouse] on
usb-0000:0b:00.0-3.2/input0
[   11.442141] mousedev: PS/2 mouse device common for all mice
[   12.715385] psmouse serio1: synaptics: queried max coordinates: x
[..5676], y [..4760]
[   12.744807] psmouse serio1: synaptics: queried min coordinates: x
[1266..], y [1094..]
[   12.744810] psmouse serio1: synaptics: Trying to set up SMBus access
[   12.808775] rmi4_f01 rmi4-00.fn01: found RMI device, manufacturer:
Synaptics, product: TM3289-002, fw id: 2492434
[   12.871445] input: Synaptics TM3289-002 as /devices/rmi4-00/input/input22
[   43.941741] psmouse serio2: trackpoint: failed to get extended
button data, assuming 3 buttons
[   44.564540] psmouse serio2: trackpoint: IBM TrackPoint firmware:
0x01, buttons: 3/3
[   50.834012] psmouse serio2: TrackPoint at
synaptics-rmi4-pt/serio1/input0 lost synchronization, throwing 1 bytes
away.


ps. there's a human induced delay between 5.5 and 10.5 while I'm
entering my LUKS passphrase


$ xinput list
⎡ Virtual core pointer                          id=2    [master pointer  (3)]
⎜   ↳ Virtual core XTEST pointer                id=4    [slave  pointer  (2)]
⎜   ↳ PixArt Dell MS116 USB Optical Mouse       id=13   [slave  pointer  (2)]
⎜   ↳ Synaptics TM3289-002                      id=16   [slave  pointer  (2)]
⎜   ↳ TPPS/2 IBM TrackPoint                     id=17   [slave  pointer  (2)]


-- 
damjan

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web