diff mbox series

[1/1,SRU,Zesty] Input: i8042 - add Gigabyte P57 to the keyboard reset table

Message ID 20170920011633.7306-2-kai.heng.feng@canonical.com
State New
Headers show
Series [1/1,SRU,Zesty] Input: i8042 - add Gigabyte P57 to the keyboard reset table | expand

Commit Message

Kai-Heng Feng Sept. 20, 2017, 1:16 a.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/1594214

Similar to other Gigabyte laptops, the touchpad on P57 requires a
keyboard reset to detect Elantech touchpad correctly.

BugLink: https://bugs.launchpad.net/bugs/1594214
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Cc: stable@vger.kernel.org
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
(cherry picked from commit 697c5d8a36768b36729533fb44622b35d56d6ad0)
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
---
 drivers/input/serio/i8042-x86ia64io.h | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Po-Hsu Lin Sept. 20, 2017, 3:26 a.m. UTC | #1 
Hi,
Is there any specific reason that this patch targets only Zesty?
Thanks

On Wed, Sep 20, 2017 at 9:16 AM, Kai-Heng Feng
<kai.heng.feng@canonical.com> wrote:
> BugLink: https://bugs.launchpad.net/bugs/1594214
>
> Similar to other Gigabyte laptops, the touchpad on P57 requires a
> keyboard reset to detect Elantech touchpad correctly.
>
> BugLink: https://bugs.launchpad.net/bugs/1594214
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> Cc: stable@vger.kernel.org
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> (cherry picked from commit 697c5d8a36768b36729533fb44622b35d56d6ad0)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> ---
>  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 ecd075fd5754..3c5da84a37ba 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -825,6 +825,13 @@ static const struct dmi_system_id __initconst i8042_dmi_kbdreset_table[] = {
>                         DMI_MATCH(DMI_PRODUCT_NAME, "P34"),
>                 },
>         },
> +       {
> +               /* Gigabyte P57 - Elantech touchpad */
> +               .matches = {
> +                       DMI_MATCH(DMI_SYS_VENDOR, "GIGABYTE"),
> +                       DMI_MATCH(DMI_PRODUCT_NAME, "P57"),
> +               },
> +       },
>         {
>                 /* Schenker XMG C504 - Elantech touchpad */
>                 .matches = {
> --
> 2.14.1
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Seth Forshee Sept. 20, 2017, 1:31 p.m. UTC | #2 
On Wed, Sep 20, 2017 at 09:16:33AM +0800, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1594214
> 
> Similar to other Gigabyte laptops, the touchpad on P57 requires a
> keyboard reset to detect Elantech touchpad correctly.
> 
> BugLink: https://bugs.launchpad.net/bugs/1594214
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> Cc: stable@vger.kernel.org
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> (cherry picked from commit 697c5d8a36768b36729533fb44622b35d56d6ad0)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>

Clean cherry pick, only adds a new DMI match so minimal regression
potential.

Acked-by: Seth Forshee <seth.forshee@canonical.com>

Applied to artful/master-next, thanks.
Stefan Bader Sept. 26, 2017, 3:50 p.m. UTC | #3 
On 19.09.2017 21:16, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1594214
> 
> Similar to other Gigabyte laptops, the touchpad on P57 requires a
> keyboard reset to detect Elantech touchpad correctly.
> 
> BugLink: https://bugs.launchpad.net/bugs/1594214
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> Cc: stable@vger.kernel.org
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> (cherry picked from commit 697c5d8a36768b36729533fb44622b35d56d6ad0)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>

> ---
>  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 ecd075fd5754..3c5da84a37ba 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -825,6 +825,13 @@ static const struct dmi_system_id __initconst i8042_dmi_kbdreset_table[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "P34"),
>  		},
>  	},
> +	{
> +		/* Gigabyte P57 - Elantech touchpad */
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "GIGABYTE"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "P57"),
> +		},
> +	},
>  	{
>  		/* Schenker XMG C504 - Elantech touchpad */
>  		.matches = {
>
diff mbox series

Patch

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index ecd075fd5754..3c5da84a37ba 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -825,6 +825,13 @@  static const struct dmi_system_id __initconst i8042_dmi_kbdreset_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "P34"),
 		},
 	},
+	{
+		/* Gigabyte P57 - Elantech touchpad */
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "GIGABYTE"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "P57"),
+		},
+	},
 	{
 		/* Schenker XMG C504 - Elantech touchpad */
 		.matches = {