diff mbox

Input: goodix - Add "GDIX1002" ACPI HID

Message ID 20180607093640.8103-1-hdegoede@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hans de Goede June 7, 2018, 9:36 a.m. UTC
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(+)

Comments

Bastien Nocera June 7, 2018, 9:55 a.m. UTC | #1
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 :)
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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);