Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.kernel > #89573 > unrolled thread

Re: Patch proposal for Goodix Touchscreen Driver (GDIX1003 - Debian 13.1 - kernel 6.12.43)

Back to article view | Back to linux.debian.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: Patch proposal for Goodix Touchscreen Driver (GDIX1003 - Debian  13.1 - kernel 6.12.43) Salvatore Bonaccorso <carnil@debian.org> - 2025-10-13 09:10 +0200
    Re: Patch proposal for Goodix Touchscreen Driver (GDIX1003 - Debian  13.1 - kernel 6.12.43) Salvatore Bonaccorso <carnil@debian.org> - 2025-12-01 17:10 +0100

#89573 — Re: Patch proposal for Goodix Touchscreen Driver (GDIX1003 - Debian 13.1 - kernel 6.12.43)

Hi,

On Fri, Oct 03, 2025 at 08:21:51PM +0200, Raphael La Greca wrote:
> Hi,
> 
> I have a new touchscreen tablet. I installed the Trixie version (13.1 -
> kernel 6.12.43), but unfortunately the touchscreen was not functional. The
> touchscreen uses a Goodix controller.
> 
> I patched the Goodix driver to make it work. The driver currently supports
> ACPI matches for devices GDIX1001 and GDIX1002, but not GDIX1003, which is
> the model used by my touchscreen.
> 
> I’m attaching a very simple patch to this message, do you think it would be
> possible to include it in a future Debian release?
> 
> Best regards,
> Raphaël La Greca

> --- /tmp/kernel/linux-source-6.12/drivers/input/touchscreen/goodix.c	2025-08-20 18:30:58.000000000 +0200
> +++ drivers/input/touchscreen/goodix.c	2025-10-03 12:50:16.723973276 +0200
> @@ -1519,6 +1519,7 @@
>  static const struct acpi_device_id goodix_acpi_match[] = {
>  	{ "GDIX1001", 0 },
>  	{ "GDIX1002", 0 },
> +	{ "GDIX1003", 0 },
>  	{ "GDX9110", 0 },
>  	{ }
>  };

This change has been proposed already upstream via
https://lore.kernel.org/linux-input/72619870-bf83-47f9-9b66-6678e245364c@redhat.com/
https://lore.kernel.org/linux-input/20250301113525.6997-1-hdegoede@redhat.com/

but TTBOMK it has not yet been accepted.

Regards,
Salvatore

[toc] | [next] | [standalone]

#90334

Hi,

On Mon, Oct 13, 2025 at 09:08:49AM +0200, Salvatore Bonaccorso wrote:
> Hi,
> 
> On Fri, Oct 03, 2025 at 08:21:51PM +0200, Raphael La Greca wrote:
> > Hi,
> > 
> > I have a new touchscreen tablet. I installed the Trixie version (13.1 -
> > kernel 6.12.43), but unfortunately the touchscreen was not functional. The
> > touchscreen uses a Goodix controller.
> > 
> > I patched the Goodix driver to make it work. The driver currently supports
> > ACPI matches for devices GDIX1001 and GDIX1002, but not GDIX1003, which is
> > the model used by my touchscreen.
> > 
> > I’m attaching a very simple patch to this message, do you think it would be
> > possible to include it in a future Debian release?
> > 
> > Best regards,
> > Raphaël La Greca
> 
> > --- /tmp/kernel/linux-source-6.12/drivers/input/touchscreen/goodix.c	2025-08-20 18:30:58.000000000 +0200
> > +++ drivers/input/touchscreen/goodix.c	2025-10-03 12:50:16.723973276 +0200
> > @@ -1519,6 +1519,7 @@
> >  static const struct acpi_device_id goodix_acpi_match[] = {
> >  	{ "GDIX1001", 0 },
> >  	{ "GDIX1002", 0 },
> > +	{ "GDIX1003", 0 },
> >  	{ "GDX9110", 0 },
> >  	{ }
> >  };
> 
> This change has been proposed already upstream via
> https://lore.kernel.org/linux-input/72619870-bf83-47f9-9b66-6678e245364c@redhat.com/
> https://lore.kernel.org/linux-input/20250301113525.6997-1-hdegoede@redhat.com/
> 
> but TTBOMK it has not yet been accepted.

This change did make it to 6.18-rc7 and got backported to 6.17.10 and
6.12.60.

Regards,
Salvatore

[toc] | [prev] | [standalone]

Back to top | Article view | linux.debian.kernel

csiph-web