linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Input: goodix - Add "GDIX1002" ACPI HID
@ 2018-06-07  9:36 Hans de Goede
  2018-06-07  9:55 ` Bastien Nocera
  0 siblings, 1 reply; 2+ messages in thread
From: Hans de Goede @ 2018-06-07  9:36 UTC (permalink / raw)
  To: Dmitry Torokhov, Bastien Nocera; +Cc: Hans de Goede, linux-input, stable

Some newer devices with a goodix touchscreen use GDIX1002 as ACPI HID
rather then GDIX1001. These devices work fine with the goodix driver
as is.

This commit adds the new "GDIX1002" ACPI HID to make the touchscreen
work on these devices.

Buglink: https://www.reddit.com/r/gpdwin/comments/8lwy8n/gpd_win_2_linux_touchscreen_fix/
Cc: stable@vger.kernel.org
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/input/touchscreen/goodix.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 9736c83dd418..f2d9c2c41885 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -933,6 +933,7 @@ MODULE_DEVICE_TABLE(i2c, goodix_ts_id);
 #ifdef CONFIG_ACPI
 static const struct acpi_device_id goodix_acpi_match[] = {
 	{ "GDIX1001", 0 },
+	{ "GDIX1002", 0 },
 	{ }
 };
 MODULE_DEVICE_TABLE(acpi, goodix_acpi_match);
-- 
2.17.1

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

* Re: [PATCH] Input: goodix - Add "GDIX1002" ACPI HID
  2018-06-07  9:36 [PATCH] Input: goodix - Add "GDIX1002" ACPI HID Hans de Goede
@ 2018-06-07  9:55 ` Bastien Nocera
  0 siblings, 0 replies; 2+ messages in thread
From: Bastien Nocera @ 2018-06-07  9:55 UTC (permalink / raw)
  To: Hans de Goede, Dmitry Torokhov; +Cc: linux-input, stable

On Thu, 2018-06-07 at 11:36 +0200, Hans de Goede wrote:
> Some newer devices with a goodix touchscreen use GDIX1002 as ACPI HID
> rather then GDIX1001. These devices work fine with the goodix driver
> as is.
> 
> This commit adds the new "GDIX1002" ACPI HID to make the touchscreen
> work on these devices.

Ethan Lee sent this one about a week ago, along with a fix for the
joysticks.

If you want, you can however follow-up on that patch to remove the i2c
struct above the acpi match tablet, that'd be appreciated :)

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

end of thread, other threads:[~2018-06-07  9:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-07  9:36 [PATCH] Input: goodix - Add "GDIX1002" ACPI HID Hans de Goede
2018-06-07  9:55 ` Bastien Nocera

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).