On Wed, May 30, 2018 at 10:19:46AM +0200, Bastien Nocera wrote: > On Tue, 2018-05-29 at 15:44 -0700, Dmitry Torokhov wrote: > > On Tue, May 29, 2018 at 11:48 AM, <flibitijibibo@xxxxxxxxx> wrote: > > > From: Ethan Lee <flibitijibibo@xxxxxxxxx> > > > > > > GPD Win 2 Website: http://www.gpd.hk/gpdwin2.asp > > > > > > Tested on a unit from the first production run sent to Indiegogo > > > backers > > > > > > Signed-off-by: Ethan Lee <flibitijibibo@xxxxxxxxx> > > > --- > > > 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 9736c83dd418..6a4ffe800194 100644 > > > --- a/drivers/input/touchscreen/goodix.c > > > +++ b/drivers/input/touchscreen/goodix.c > > > @@ -926,6 +926,7 @@ static SIMPLE_DEV_PM_OPS(goodix_pm_ops, > > > goodix_suspend, goodix_resume); > > > > > > static const struct i2c_device_id goodix_ts_id[] = { > > > { "GDIX1001:00", 0 }, > > > + { "GDIX1002:00", 0 }, > > > > Do we really need this? And actually the first one as well? > > It was in the original driver submission, which isn't that old. I don't > have a way to test this right now though. I am pretty sure we changed the way we generate modaliases for I2C devices (from ACPI/OF) since then. Ethan, can you please try building the driver without the new entry in goodix_ts_id, make it a module and make sure it gets autoloaded for you. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
