All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list
@ 2020-04-20 16:37 Kevin Locke
  2020-04-20 17:02 ` Dmitry Torokhov
  0 siblings, 1 reply; 7+ messages in thread
From: Kevin Locke @ 2020-04-20 16:37 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: linux-input, stable

On the Lenovo ThinkPad Twist S230u (3347-4HU) with BIOS version
"GDETC1WW (1.81 ) 06/27/2019", whether booted in UEFI or Legacy/CSM mode
the keyboard, Synaptics TouchPad, and TrackPoint either do not function
or stop functioning a few minutes after boot.  This problem has been
noted before, perhaps only occurring on BIOS 1.57 and
later.[1][2][3][4][5]

This model does not have an external PS/2 port, so mux does not appear
to be useful.

Odds of a BIOS fix appear to be low: 1.57 was released over 6 years ago
and although the [BIOS changelog] notes "Fixed an issue of UEFI
touchpad/trackpoint/keyboard/touchscreen" in 1.58, it appears to be
insufficient.

Adding 33474HU to the nomux list avoids the issue on my system.

[1]: https://bugs.launchpad.net/bugs/1210748
[2]: https://bbs.archlinux.org/viewtopic.php?pid=1360425
[3]: https://forums.linuxmint.com/viewtopic.php?f=46&t=41200
[4]: https://forums.linuxmint.com/viewtopic.php?f=49&t=157115
[5]: https://forums.lenovo.com/topic/findpost/27/1337119
[BIOS changelog]: https://download.lenovo.com/pccbbs/mobiles/gduj33uc.txt

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Cc: stable@vger.kernel.org
---
 drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index 08e919dbeb5d..5bbc9152731d 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -541,6 +541,13 @@ static const struct dmi_system_id __initconst i8042_dmi_nomux_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5738"),
 		},
 	},
+	{
+		/* Lenovo ThinkPad Twist S230u */
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "33474HU"),
+		},
+	},
 	{ }
 };
 
-- 
2.26.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list
  2020-04-20 16:37 [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list Kevin Locke
@ 2020-04-20 17:02 ` Dmitry Torokhov
  2020-04-24  4:25   ` Kevin Locke
  0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Torokhov @ 2020-04-20 17:02 UTC (permalink / raw)
  To: Kevin Locke; +Cc: linux-input, stable

On Mon, Apr 20, 2020 at 10:37:15AM -0600, Kevin Locke wrote:
> On the Lenovo ThinkPad Twist S230u (3347-4HU) with BIOS version
> "GDETC1WW (1.81 ) 06/27/2019", whether booted in UEFI or Legacy/CSM mode
> the keyboard, Synaptics TouchPad, and TrackPoint either do not function
> or stop functioning a few minutes after boot.  This problem has been
> noted before, perhaps only occurring on BIOS 1.57 and
> later.[1][2][3][4][5]
> 
> This model does not have an external PS/2 port, so mux does not appear
> to be useful.
> 
> Odds of a BIOS fix appear to be low: 1.57 was released over 6 years ago
> and although the [BIOS changelog] notes "Fixed an issue of UEFI
> touchpad/trackpoint/keyboard/touchscreen" in 1.58, it appears to be
> insufficient.
> 
> Adding 33474HU to the nomux list avoids the issue on my system.
> 
> [1]: https://bugs.launchpad.net/bugs/1210748
> [2]: https://bbs.archlinux.org/viewtopic.php?pid=1360425
> [3]: https://forums.linuxmint.com/viewtopic.php?f=46&t=41200
> [4]: https://forums.linuxmint.com/viewtopic.php?f=49&t=157115
> [5]: https://forums.lenovo.com/topic/findpost/27/1337119
> [BIOS changelog]: https://download.lenovo.com/pccbbs/mobiles/gduj33uc.txt
> 
> Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
> Cc: stable@vger.kernel.org

Applied, thank you.

> ---
>  drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
> index 08e919dbeb5d..5bbc9152731d 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -541,6 +541,13 @@ static const struct dmi_system_id __initconst i8042_dmi_nomux_table[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5738"),
>  		},
>  	},
> +	{
> +		/* Lenovo ThinkPad Twist S230u */
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "33474HU"),
> +		},
> +	},
>  	{ }
>  };
>  
> -- 
> 2.26.1
> 

-- 
Dmitry

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list
  2020-04-20 17:02 ` Dmitry Torokhov
@ 2020-04-24  4:25   ` Kevin Locke
  2020-04-24  4:30     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list Kevin Locke
  2020-04-28  1:08     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list Dmitry Torokhov
  0 siblings, 2 replies; 7+ messages in thread
From: Kevin Locke @ 2020-04-24  4:25 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: linux-input

On Mon, 2020-04-20 at 10:02 -0700, Dmitry Torokhov wrote:
> Applied, thank you.

I believe I was in error and nomux only appeared to fix the issue
because the controller continued working after warm reboots.  After
more thorough testing from both warm and cold start, I now believe the
entry should be added to i8042_dmi_reset_table rather than
i8042_dmi_nomux_table as i8042.reset=1 alone is sufficient to avoid
the issue from both states while i8042.nomux is not.

Please drop or revert the previous patch, if possible.
I will send a corrected and better-tested patch momentarily.

My apologies for the confusion,
Kevin

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list
  2020-04-24  4:25   ` Kevin Locke
@ 2020-04-24  4:30     ` Kevin Locke
  2020-04-28  1:08       ` Dmitry Torokhov
  2020-04-28  1:08     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list Dmitry Torokhov
  1 sibling, 1 reply; 7+ messages in thread
From: Kevin Locke @ 2020-04-24  4:30 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: linux-input

On the Lenovo ThinkPad Twist S230u (3347-4HU) with BIOS version
"GDETC1WW (1.81 ) 06/27/2019", the keyboard, Synaptics TouchPad, and
TrackPoint either do not function or stop functioning a few minutes
after boot.  This problem has been noted before, perhaps only occurring
with BIOS 1.57 and later.[1][2][3][4][5]

Odds of a BIOS fix appear to be low: 1.57 was released over 6 years ago
and although the [BIOS changelog] notes "Fixed an issue of UEFI
touchpad/trackpoint/keyboard/touchscreen" in 1.58, it appears to be
insufficient.

Setting i8042.reset=1 or adding 33474HU to the reset list avoids the
issue on my system from either warm or cold boot.

[1]: https://bugs.launchpad.net/bugs/1210748
[2]: https://bbs.archlinux.org/viewtopic.php?pid=1360425
[3]: https://forums.linuxmint.com/viewtopic.php?f=46&t=41200
[4]: https://forums.linuxmint.com/viewtopic.php?f=49&t=157115
[5]: https://forums.lenovo.com/topic/findpost/27/1337119
[BIOS changelog]: https://download.lenovo.com/pccbbs/mobiles/gduj33uc.txt

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Cc: stable@vger.kernel.org
---
 drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index 08e919dbeb5d..7e048b557462 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -662,6 +662,13 @@ static const struct dmi_system_id __initconst i8042_dmi_reset_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "P65xRP"),
 		},
 	},
+	{
+		/* Lenovo ThinkPad Twist S230u */
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "33474HU"),
+		},
+	},
 	{ }
 };
 
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list
  2020-04-24  4:25   ` Kevin Locke
  2020-04-24  4:30     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list Kevin Locke
@ 2020-04-28  1:08     ` Dmitry Torokhov
  1 sibling, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2020-04-28  1:08 UTC (permalink / raw)
  To: Kevin Locke, linux-input

On Thu, Apr 23, 2020 at 10:25:00PM -0600, Kevin Locke wrote:
> On Mon, 2020-04-20 at 10:02 -0700, Dmitry Torokhov wrote:
> > Applied, thank you.
> 
> I believe I was in error and nomux only appeared to fix the issue
> because the controller continued working after warm reboots.  After
> more thorough testing from both warm and cold start, I now believe the
> entry should be added to i8042_dmi_reset_table rather than
> i8042_dmi_nomux_table as i8042.reset=1 alone is sufficient to avoid
> the issue from both states while i8042.nomux is not.
> 
> Please drop or revert the previous patch, if possible.
> I will send a corrected and better-tested patch momentarily.

Reverted, thank you.

-- 
Dmitry

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list
  2020-04-24  4:30     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list Kevin Locke
@ 2020-04-28  1:08       ` Dmitry Torokhov
  0 siblings, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2020-04-28  1:08 UTC (permalink / raw)
  To: Kevin Locke; +Cc: linux-input

On Thu, Apr 23, 2020 at 10:30:36PM -0600, Kevin Locke wrote:
> On the Lenovo ThinkPad Twist S230u (3347-4HU) with BIOS version
> "GDETC1WW (1.81 ) 06/27/2019", the keyboard, Synaptics TouchPad, and
> TrackPoint either do not function or stop functioning a few minutes
> after boot.  This problem has been noted before, perhaps only occurring
> with BIOS 1.57 and later.[1][2][3][4][5]
> 
> Odds of a BIOS fix appear to be low: 1.57 was released over 6 years ago
> and although the [BIOS changelog] notes "Fixed an issue of UEFI
> touchpad/trackpoint/keyboard/touchscreen" in 1.58, it appears to be
> insufficient.
> 
> Setting i8042.reset=1 or adding 33474HU to the reset list avoids the
> issue on my system from either warm or cold boot.
> 
> [1]: https://bugs.launchpad.net/bugs/1210748
> [2]: https://bbs.archlinux.org/viewtopic.php?pid=1360425
> [3]: https://forums.linuxmint.com/viewtopic.php?f=46&t=41200
> [4]: https://forums.linuxmint.com/viewtopic.php?f=49&t=157115
> [5]: https://forums.lenovo.com/topic/findpost/27/1337119
> [BIOS changelog]: https://download.lenovo.com/pccbbs/mobiles/gduj33uc.txt
> 
> Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
> Cc: stable@vger.kernel.org

Applied, thank you.

> ---
>  drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
> index 08e919dbeb5d..7e048b557462 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -662,6 +662,13 @@ static const struct dmi_system_id __initconst i8042_dmi_reset_table[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "P65xRP"),
>  		},
>  	},
> +	{
> +		/* Lenovo ThinkPad Twist S230u */
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "33474HU"),
> +		},
> +	},
>  	{ }
>  };
>  
> -- 
> 2.26.2
> 

-- 
Dmitry

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list
@ 2020-04-20 16:35 Kevin Locke
  0 siblings, 0 replies; 7+ messages in thread
From: Kevin Locke @ 2020-04-20 16:35 UTC (permalink / raw)
  To: kevin; +Cc: stable

On the Lenovo ThinkPad Twist S230u (3347-4HU) with BIOS version
"GDETC1WW (1.81 ) 06/27/2019", whether booted in UEFI or Legacy/CSM mode
the keyboard, Synaptics TouchPad, and TrackPoint either do not function
or stop functioning a few minutes after boot.  This problem has been
noted before, perhaps only occurring on BIOS 1.57 and
later.[1][2][3][4][5]

This model does not have an external PS/2 port, so mux does not appear
to be useful.

Odds of a BIOS fix appear to be low: 1.57 was released over 6 years ago
and although the [BIOS changelog] notes "Fixed an issue of UEFI
touchpad/trackpoint/keyboard/touchscreen" in 1.58, it appears to be
insufficient.

Adding 33474HU to the nomux list avoids the issue on my system.

[1]: https://bugs.launchpad.net/bugs/1210748
[2]: https://bbs.archlinux.org/viewtopic.php?pid=1360425
[3]: https://forums.linuxmint.com/viewtopic.php?f=46&t=41200
[4]: https://forums.linuxmint.com/viewtopic.php?f=49&t=157115
[5]: https://forums.lenovo.com/topic/findpost/27/1337119
[BIOS changelog]: https://download.lenovo.com/pccbbs/mobiles/gduj33uc.txt

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Cc: stable@vger.kernel.org
---
 drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index 08e919dbeb5d..5bbc9152731d 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -541,6 +541,13 @@ static const struct dmi_system_id __initconst i8042_dmi_nomux_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5738"),
 		},
 	},
+	{
+		/* Lenovo ThinkPad Twist S230u */
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "33474HU"),
+		},
+	},
 	{ }
 };
 
-- 
2.26.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-04-28  1:08 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-20 16:37 [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list Kevin Locke
2020-04-20 17:02 ` Dmitry Torokhov
2020-04-24  4:25   ` Kevin Locke
2020-04-24  4:30     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 reset list Kevin Locke
2020-04-28  1:08       ` Dmitry Torokhov
2020-04-28  1:08     ` [PATCH] Input: i8042 - add ThinkPad S230u to i8042 nomux list Dmitry Torokhov
  -- strict thread matches above, loose matches on Subject: below --
2020-04-20 16:35 Kevin Locke

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.