* [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet
@ 2020-04-21 13:25 Maksim Karasev
2020-04-21 13:50 ` Hans de Goede
2020-04-21 14:39 ` Hans de Goede
0 siblings, 2 replies; 5+ messages in thread
From: Maksim Karasev @ 2020-04-21 13:25 UTC (permalink / raw)
Cc: hdegoede, dvhart, andy, linux-input, platform-driver-x86,
Maksim Karasev
Add touchscreen info for the ONDA V891 v5 tablet.
Signed-off-by: Maksim Karasev <karasevm98@gmail.com>
---
drivers/platform/x86/touchscreen_dmi.c | 27 ++++++++++++++++++++++++++
1 file changed, 27 insertions(+)
diff --git a/drivers/platform/x86/touchscreen_dmi.c b/drivers/platform/x86/touchscreen_dmi.c
index 6ec8923dec1a..cc2a2e0a0585 100644
--- a/drivers/platform/x86/touchscreen_dmi.c
+++ b/drivers/platform/x86/touchscreen_dmi.c
@@ -448,6 +448,24 @@ static const struct ts_dmi_data onda_v820w_32g_data = {
.properties = onda_v820w_32g_props,
};
+static const struct property_entry onda_v891_v5_props[] = {
+ PROPERTY_ENTRY_U32("touchscreen-size-x", 1715),
+ PROPERTY_ENTRY_U32("touchscreen-size-y", 1140),
+ PROPERTY_ENTRY_BOOL("touchscreen-inverted-x"),
+ PROPERTY_ENTRY_BOOL("touchscreen-inverted-y"),
+ PROPERTY_ENTRY_BOOL("touchscreen-swapped-x-y"),
+ PROPERTY_ENTRY_STRING("firmware-name",
+ "gsl3676-onda-v891-v5.fw"),
+ PROPERTY_ENTRY_U32("silead,max-fingers", 10),
+ PROPERTY_ENTRY_BOOL("silead,home-button"),
+ { }
+};
+
+static const struct ts_dmi_data onda_v891_v5_data = {
+ .acpi_name = "MSSL1680:00",
+ .properties = onda_v891_v5_props,
+};
+
static const struct property_entry onda_v891w_v1_props[] = {
PROPERTY_ENTRY_U32("touchscreen-min-x", 46),
PROPERTY_ENTRY_U32("touchscreen-min-y", 8),
@@ -940,6 +958,15 @@ const struct dmi_system_id touchscreen_dmi_table[] = {
DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "V820w DualOS")
},
},
+ {
+ /* ONDA V891 v5 */
+ .driver_data = (void *)&onda_v891_v5_data,
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "ONDA"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "ONDA Tablet"),
+ DMI_MATCH(DMI_BIOS_VERSION, "ONDA.D869CJABNRBA06"),
+ },
+ },
{
/* ONDA V891w revision P891WBEBV1B00 aka v1 */
.driver_data = (void *)&onda_v891w_v1_data,
--
2.26.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet
2020-04-21 13:25 [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet Maksim Karasev
@ 2020-04-21 13:50 ` Hans de Goede
2020-04-21 14:24 ` Maksim Karasev
2020-04-21 14:39 ` Hans de Goede
1 sibling, 1 reply; 5+ messages in thread
From: Hans de Goede @ 2020-04-21 13:50 UTC (permalink / raw)
To: Maksim Karasev; +Cc: dvhart, andy, linux-input, platform-driver-x86
Hi,
On 4/21/20 3:25 PM, Maksim Karasev wrote:
> Add touchscreen info for the ONDA V891 v5 tablet.
Thank you for the patch.
Can you please replace v891 with v891w everywhere to
be consistent with the existing v891 entries; and ...
>
> Signed-off-by: Maksim Karasev <karasevm98@gmail.com>
> ---
> drivers/platform/x86/touchscreen_dmi.c | 27 ++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/drivers/platform/x86/touchscreen_dmi.c b/drivers/platform/x86/touchscreen_dmi.c
> index 6ec8923dec1a..cc2a2e0a0585 100644
> --- a/drivers/platform/x86/touchscreen_dmi.c
> +++ b/drivers/platform/x86/touchscreen_dmi.c
> @@ -448,6 +448,24 @@ static const struct ts_dmi_data onda_v820w_32g_data = {
> .properties = onda_v820w_32g_props,
> };
>
> +static const struct property_entry onda_v891_v5_props[] = {
> + PROPERTY_ENTRY_U32("touchscreen-size-x", 1715),
> + PROPERTY_ENTRY_U32("touchscreen-size-y", 1140),
> + PROPERTY_ENTRY_BOOL("touchscreen-inverted-x"),
> + PROPERTY_ENTRY_BOOL("touchscreen-inverted-y"),
> + PROPERTY_ENTRY_BOOL("touchscreen-swapped-x-y"),
> + PROPERTY_ENTRY_STRING("firmware-name",
> + "gsl3676-onda-v891-v5.fw"),
> + PROPERTY_ENTRY_U32("silead,max-fingers", 10),
> + PROPERTY_ENTRY_BOOL("silead,home-button"),
> + { }
> +};
> +
> +static const struct ts_dmi_data onda_v891_v5_data = {
> + .acpi_name = "MSSL1680:00",
> + .properties = onda_v891_v5_props,
> +};
> +
Put this after the v891w_v3 entry please so that we
have in order v1, v3 and then v5.
Thanks,
Hans
> static const struct property_entry onda_v891w_v1_props[] = {
> PROPERTY_ENTRY_U32("touchscreen-min-x", 46),
> PROPERTY_ENTRY_U32("touchscreen-min-y", 8),
> @@ -940,6 +958,15 @@ const struct dmi_system_id touchscreen_dmi_table[] = {
> DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "V820w DualOS")
> },
> },
> + {
> + /* ONDA V891 v5 */
> + .driver_data = (void *)&onda_v891_v5_data,
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "ONDA"),
> + DMI_MATCH(DMI_PRODUCT_NAME, "ONDA Tablet"),
> + DMI_MATCH(DMI_BIOS_VERSION, "ONDA.D869CJABNRBA06"),
> + },
> + },
> {
> /* ONDA V891w revision P891WBEBV1B00 aka v1 */
> .driver_data = (void *)&onda_v891w_v1_data,
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet
2020-04-21 13:50 ` Hans de Goede
@ 2020-04-21 14:24 ` Maksim Karasev
2020-04-21 14:39 ` Hans de Goede
0 siblings, 1 reply; 5+ messages in thread
From: Maksim Karasev @ 2020-04-21 14:24 UTC (permalink / raw)
To: Hans de Goede; +Cc: dvhart, andy, linux-input, platform-driver-x86
Hello,
Sorry, but I think you are mistaken,
the v891 and v891w are 2 different tablets (different screen, battery,
camera, and internal storage).
My patch is for the v891.
Thanks.
-------------------------
Maksim Karasev
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet
2020-04-21 14:24 ` Maksim Karasev
@ 2020-04-21 14:39 ` Hans de Goede
0 siblings, 0 replies; 5+ messages in thread
From: Hans de Goede @ 2020-04-21 14:39 UTC (permalink / raw)
To: Maksim Karasev; +Cc: dvhart, andy, linux-input, platform-driver-x86
Hi,
On 4/21/20 4:24 PM, Maksim Karasev wrote:
> Hello,
> Sorry, but I think you are mistaken,
> the v891 and v891w are 2 different tablets (different screen, battery,
> camera, and internal storage).
> My patch is for the v891.
Ah I see, so the v891w has a 1920x1200 screen and the
v891 has a 1280x800 screen, ok.
Regards,
Hans
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet
2020-04-21 13:25 [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet Maksim Karasev
2020-04-21 13:50 ` Hans de Goede
@ 2020-04-21 14:39 ` Hans de Goede
1 sibling, 0 replies; 5+ messages in thread
From: Hans de Goede @ 2020-04-21 14:39 UTC (permalink / raw)
To: Maksim Karasev; +Cc: dvhart, andy, linux-input, platform-driver-x86
Hi,
On 4/21/20 3:25 PM, Maksim Karasev wrote:
> Add touchscreen info for the ONDA V891 v5 tablet.
>
> Signed-off-by: Maksim Karasev <karasevm98@gmail.com>
Patch looks good to me:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Regards,
Hans
> ---
> drivers/platform/x86/touchscreen_dmi.c | 27 ++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/drivers/platform/x86/touchscreen_dmi.c b/drivers/platform/x86/touchscreen_dmi.c
> index 6ec8923dec1a..cc2a2e0a0585 100644
> --- a/drivers/platform/x86/touchscreen_dmi.c
> +++ b/drivers/platform/x86/touchscreen_dmi.c
> @@ -448,6 +448,24 @@ static const struct ts_dmi_data onda_v820w_32g_data = {
> .properties = onda_v820w_32g_props,
> };
>
> +static const struct property_entry onda_v891_v5_props[] = {
> + PROPERTY_ENTRY_U32("touchscreen-size-x", 1715),
> + PROPERTY_ENTRY_U32("touchscreen-size-y", 1140),
> + PROPERTY_ENTRY_BOOL("touchscreen-inverted-x"),
> + PROPERTY_ENTRY_BOOL("touchscreen-inverted-y"),
> + PROPERTY_ENTRY_BOOL("touchscreen-swapped-x-y"),
> + PROPERTY_ENTRY_STRING("firmware-name",
> + "gsl3676-onda-v891-v5.fw"),
> + PROPERTY_ENTRY_U32("silead,max-fingers", 10),
> + PROPERTY_ENTRY_BOOL("silead,home-button"),
> + { }
> +};
> +
> +static const struct ts_dmi_data onda_v891_v5_data = {
> + .acpi_name = "MSSL1680:00",
> + .properties = onda_v891_v5_props,
> +};
> +
> static const struct property_entry onda_v891w_v1_props[] = {
> PROPERTY_ENTRY_U32("touchscreen-min-x", 46),
> PROPERTY_ENTRY_U32("touchscreen-min-y", 8),
> @@ -940,6 +958,15 @@ const struct dmi_system_id touchscreen_dmi_table[] = {
> DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "V820w DualOS")
> },
> },
> + {
> + /* ONDA V891 v5 */
> + .driver_data = (void *)&onda_v891_v5_data,
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "ONDA"),
> + DMI_MATCH(DMI_PRODUCT_NAME, "ONDA Tablet"),
> + DMI_MATCH(DMI_BIOS_VERSION, "ONDA.D869CJABNRBA06"),
> + },
> + },
> {
> /* ONDA V891w revision P891WBEBV1B00 aka v1 */
> .driver_data = (void *)&onda_v891w_v1_data,
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-04-21 14:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-21 13:25 [PATCH] platform/x86: touchscreen_dmi: Add info for the ONDA V891 v5 tablet Maksim Karasev
2020-04-21 13:50 ` Hans de Goede
2020-04-21 14:24 ` Maksim Karasev
2020-04-21 14:39 ` Hans de Goede
2020-04-21 14:39 ` Hans de Goede
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox