| Message ID | 20230807124723.382899-1-felix.engelhardt@eidu.com |
|---|---|
| State | Accepted |
| Commit | 07b4c950f27bef0362dc6ad7ee713aab61d58149 |
| Headers | show |
| Series | drivers/input/touchscreen/goodix.c: Add support for ACPI ID GDX9110 | expand |
Hi, On 8/7/23 14:47, Felix Engelhardt wrote: > The Goodix touchscreen controller with ACPI ID GDX9110 was not recognized > by the goodix driver. This patch adds this ID to the list of supported IDs, > allowing the driver to be used with this device. The change will allow > Linux to be used on ~1 million tablet devices used in Kenyan primary > schools. > > Signed-off-by: Felix Engelhardt <felix.engelhardt@eidu.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 4f53d3c57e69..9cdc01eb00c9 100644 > --- a/drivers/input/touchscreen/goodix.c > +++ b/drivers/input/touchscreen/goodix.c > @@ -1379,6 +1379,7 @@ MODULE_DEVICE_TABLE(i2c, goodix_ts_id); > static const struct acpi_device_id goodix_acpi_match[] = { > { "GDIX1001", 0 }, > { "GDIX1002", 0 }, > + { "GDX9110", 0 }, > { } > }; > MODULE_DEVICE_TABLE(acpi, goodix_acpi_match); Hmm, GDX without the 'I' does look like an official ACPI vendor prefix. But if this is used in the wild, then I guess we'll just need to live with it: Reviewed-by: Hans de Goede <hdegoede@redhat.com> Regards, Hans
On Mon, Aug 07, 2023 at 02:47:23PM +0200, Felix Engelhardt wrote: > The Goodix touchscreen controller with ACPI ID GDX9110 was not recognized > by the goodix driver. This patch adds this ID to the list of supported IDs, > allowing the driver to be used with this device. The change will allow > Linux to be used on ~1 million tablet devices used in Kenyan primary > schools. > > Signed-off-by: Felix Engelhardt <felix.engelhardt@eidu.com> Applied, thank you.
diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c index 4f53d3c57e69..9cdc01eb00c9 100644 --- a/drivers/input/touchscreen/goodix.c +++ b/drivers/input/touchscreen/goodix.c @@ -1379,6 +1379,7 @@ MODULE_DEVICE_TABLE(i2c, goodix_ts_id); static const struct acpi_device_id goodix_acpi_match[] = { { "GDIX1001", 0 }, { "GDIX1002", 0 }, + { "GDX9110", 0 }, { } }; MODULE_DEVICE_TABLE(acpi, goodix_acpi_match);
The Goodix touchscreen controller with ACPI ID GDX9110 was not recognized by the goodix driver. This patch adds this ID to the list of supported IDs, allowing the driver to be used with this device. The change will allow Linux to be used on ~1 million tablet devices used in Kenyan primary schools. Signed-off-by: Felix Engelhardt <felix.engelhardt@eidu.com> --- drivers/input/touchscreen/goodix.c | 1 + 1 file changed, 1 insertion(+)