Hi, On 1/11/22 16:40, yuk7 wrote: > Add touchscreen info for RWC NANOTE P8 (AY07J) 2-in-1. > > Signed-off-by: Yuka Kawajiri <yukx00@xxxxxxxxx> Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/x86/touchscreen_dmi.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/drivers/platform/x86/touchscreen_dmi.c b/drivers/platform/x86/touchscreen_dmi.c > index 17dd54d4b..e318b4094 100644 > --- a/drivers/platform/x86/touchscreen_dmi.c > +++ b/drivers/platform/x86/touchscreen_dmi.c > @@ -773,6 +773,21 @@ static const struct ts_dmi_data predia_basic_data = { > .properties = predia_basic_props, > }; > > +static const struct property_entry rwc_nanote_p8_props[] = { > + PROPERTY_ENTRY_U32("touchscreen-min-y", 46), > + PROPERTY_ENTRY_U32("touchscreen-size-x", 1728), > + PROPERTY_ENTRY_U32("touchscreen-size-y", 1140), > + PROPERTY_ENTRY_BOOL("touchscreen-inverted-y"), > + PROPERTY_ENTRY_STRING("firmware-name", "gsl1680-rwc-nanote-p8.fw"), > + PROPERTY_ENTRY_U32("silead,max-fingers", 10), > + { } > +}; > + > +static const struct ts_dmi_data rwc_nanote_p8_data = { > + .acpi_name = "MSSL1680:00", > + .properties = rwc_nanote_p8_props, > +}; > + > static const struct property_entry schneider_sct101ctm_props[] = { > PROPERTY_ENTRY_U32("touchscreen-size-x", 1715), > PROPERTY_ENTRY_U32("touchscreen-size-y", 1140), > @@ -1406,6 +1421,15 @@ const struct dmi_system_id touchscreen_dmi_table[] = { > DMI_EXACT_MATCH(DMI_BOARD_NAME, "0E57"), > }, > }, > + { > + /* RWC NANOTE P8 */ > + .driver_data = (void *)&rwc_nanote_p8_data, > + .matches = { > + DMI_MATCH(DMI_BOARD_VENDOR, "Default string"), > + DMI_MATCH(DMI_PRODUCT_NAME, "AY07J"), > + DMI_MATCH(DMI_PRODUCT_SKU, "0001") > + }, > + }, > { > /* Schneider SCT101CTM */ > .driver_data = (void *)&schneider_sct101ctm_data, >