On Mon, Mar 20, 2017 at 1:35 PM, Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > Add an entry providing the necessary info to make the touchscreen work > on various tablets based on the Insyde 7W whitebox tablet. > > This has been tested on a DEXP Ursus 7W tablet. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> One minor below. Otherwise looks good to me (Darren or me will take care of pushing this later on), > --- > drivers/platform/x86/silead_dmi.c | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/drivers/platform/x86/silead_dmi.c b/drivers/platform/x86/silead_dmi.c > index 02e11fd..a4399e7 100644 > --- a/drivers/platform/x86/silead_dmi.c > +++ b/drivers/platform/x86/silead_dmi.c > @@ -53,6 +53,19 @@ static const struct silead_ts_dmi_data jumper_ezpad_mini3_data = { > .properties = jumper_ezpad_mini3_props, > }; > > +static struct property_entry dexp_ursus_7w_props[] = { const ? > + PROPERTY_ENTRY_U32("touchscreen-size-x", 890), > + PROPERTY_ENTRY_U32("touchscreen-size-y", 630), > + PROPERTY_ENTRY_STRING("firmware-name", "gsl1686-dexp-ursus-7w.fw"), > + PROPERTY_ENTRY_U32("silead,max-fingers", 10), > + { } > +}; > + > +static const struct silead_ts_dmi_data dexp_ursus_7w_data = { > + .acpi_name = "MSSL1680:00", > + .properties = dexp_ursus_7w_props, > +}; > + > static const struct dmi_system_id silead_ts_dmi_table[] = { > { > /* CUBE iwork8 Air */ > @@ -72,6 +85,14 @@ static const struct dmi_system_id silead_ts_dmi_table[] = { > DMI_MATCH(DMI_BIOS_VERSION, "jumperx.T87.KFBNEEA"), > }, > }, > + { > + /* DEXP Ursus 7W */ > + .driver_data = (void *)&dexp_ursus_7w_data, > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Insyde"), > + DMI_MATCH(DMI_PRODUCT_NAME, "7W"), > + }, > + }, > { }, > }; -- With Best Regards, Andy Shevchenko