diff mbox series

[v3,2/2] Input: synaptics - enable InterTouch for ThinkPad P1/X1E gen 2

Message ID 20200930225046.173190-3-Jason@zx2c4.com
State New
Headers show
Series Support trackpoint and touchpad in Thinkpad P1 gen 2 / X1E gen 2 | expand

Commit Message

Jason A. Donenfeld Sept. 30, 2020, 10:50 p.m. UTC
With the new RMI4 F3A support, we're now able to enable full RMI4
support for this model. We also tidy up the comments a bit, as the X1E
is essentially the same computer as the P1.

Acked-by: Lyude Paul <lyude@redhat.com>
Cc: Vincent Huang <vincent.huang@tw.synaptics.com>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
---
 drivers/input/mouse/synaptics.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Dmitry Torokhov Oct. 5, 2020, 2:51 a.m. UTC | #1
On Thu, Oct 01, 2020 at 12:50:46AM +0200, Jason A. Donenfeld wrote:
> With the new RMI4 F3A support, we're now able to enable full RMI4

> support for this model. We also tidy up the comments a bit, as the X1E

> is essentially the same computer as the P1.

> 

> Acked-by: Lyude Paul <lyude@redhat.com>

> Cc: Vincent Huang <vincent.huang@tw.synaptics.com>

> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

> ---

>  drivers/input/mouse/synaptics.c | 3 ++-

>  1 file changed, 2 insertions(+), 1 deletion(-)

> 

> diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c

> index 8a54efd6eb95..bf0f3fdf10d9 100644

> --- a/drivers/input/mouse/synaptics.c

> +++ b/drivers/input/mouse/synaptics.c

> @@ -179,7 +179,8 @@ static const char * const smbus_pnp_ids[] = {

>  	"LEN0093", /* T480 */

>  	"LEN0096", /* X280 */

>  	"LEN0097", /* X280 -> ALPS trackpoint */

> -	"LEN0099", /* X1 Extreme 1st */

> +	"LEN0099", /* X1 Extreme Gen 1 / P1 Gen 1 */

> +	"LEN0402", /* X1 Extreme Gen 2 / P1 Gen 2 */


Moved this entry one down to keep the list sorted, and applied, thank
you.

-- 
Dmitry
Jason A. Donenfeld Oct. 7, 2020, 3:55 p.m. UTC | #2
Oddly, CPU usage seems exceedingly high when using the trackpoint, in
the rmi4 irq handler. Running perf-top indicates a hotspot in
i801_access calling "in     (%dx),%al", which makes sense. I wonder:
is this a culprit of expensive accesses in general, which Sultan
(CC'd) has been working on? Or are we hitting something different
here? Improper IRQ masking in the i801 code?

Jason
diff mbox series

Patch

diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
index 8a54efd6eb95..bf0f3fdf10d9 100644
--- a/drivers/input/mouse/synaptics.c
+++ b/drivers/input/mouse/synaptics.c
@@ -179,7 +179,8 @@  static const char * const smbus_pnp_ids[] = {
 	"LEN0093", /* T480 */
 	"LEN0096", /* X280 */
 	"LEN0097", /* X280 -> ALPS trackpoint */
-	"LEN0099", /* X1 Extreme 1st */
+	"LEN0099", /* X1 Extreme Gen 1 / P1 Gen 1 */
+	"LEN0402", /* X1 Extreme Gen 2 / P1 Gen 2 */
 	"LEN009b", /* T580 */
 	"LEN200f", /* T450s */
 	"LEN2044", /* L470  */