RE: [PATCH 4/9] input: goodix: add ACPI IDs for GT911 and GT9271

Tirdea, Irina
2015-06-05T16:36:30Z

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1695 bytes --]

> -----Original Message-----
> From: Bastien Nocera [mailto:hadess@hadess.net]
> Sent: 04 June, 2015 15:51
> To: Tirdea, Irina
> Cc: Dmitry Torokhov; linux-input@vger.kernel.org; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 4/9] input: goodix: add ACPI IDs for GT911 and GT9271
>
> On Thu, 2015-05-28 at 15:47 +0300, Irina Tirdea wrote:
> > Add ACPI IDs to support Goodix GT911 and GT9271
> > touchscreens.
>
> Those devices are present on which systems which you would have tested
> this on?
>

I have been using a custom setup by connecting the touchscreen directly
to a virtual Linux environment through an USB-I2C adapter (https://diolan.com/dln-2).
So these ACPI IDs do not correspond to any device publicly available.
However, once in upstream we can use this IDs for our future platforms.

> > Signed-off-by: Irina Tirdea <irina.tirdea@intel.com>
> > ---
> > drivers/input/touchscreen/goodix.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/input/touchscreen/goodix.c
> > b/drivers/input/touchscreen/goodix.c
> > index 9561396..9e7d215 100644
> > --- a/drivers/input/touchscreen/goodix.c
> > +++ b/drivers/input/touchscreen/goodix.c
> > @@ -392,6 +392,8 @@ static const struct i2c_device_id goodix_ts_id[]
> > = {
> > #ifdef CONFIG_ACPI
> > static const struct acpi_device_id goodix_acpi_match[] = {
> > { "GDIX1001", 0 },
> > + { "GDIX0911", 0 },
> > + { "GDIX9271", 0 },
> > { }
> > };
> > MODULE_DEVICE_TABLE(acpi, goodix_acpi_match);
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥