touchscreen_dmi.c Patch for Viglen Connect 10

touchscreen_dmi.c Patch for Viglen Connect 10

[Mark Stamp]

Hello. 

I previously spoke with you on a touchscreen issue

https://github.com/onitake/gsl-firmware/issues/178

With regard to that, I made a patch for touchscreen_dmi.c. It adds
parameters for the Viglen Connect 10 tablet to get its touchscreen and
home button working.

Signed-off-by: Mark Stamp <stamp497@googlemail.com>

--- touchscreen_dmi.c.orig	2021-10-27 01:26:47.842668712 +0100
+++ touchscreen_dmi.c	2021-10-27 01:27:06.426298127 +0100
@@ -938,6 +938,26 @@ static const struct ts_dmi_data trekstor
 	.properties	= trekstor_surftab_wintron70_props,
 };
 
+static const struct property_entry viglen_connect_10_props[] = {
+	//PROPERTY_ENTRY_U32("touchscreen-min-x", 20),
+	PROPERTY_ENTRY_U32("touchscreen-min-x", 0),
+	PROPERTY_ENTRY_U32("touchscreen-min-y", 0),
+	PROPERTY_ENTRY_U32("touchscreen-size-x", 1890),
+	PROPERTY_ENTRY_U32("touchscreen-size-y", 1280),
+	PROPERTY_ENTRY_U32("touchscrken-fuzz-x", 8),
+	PROPERTY_ENTRY_U32("touchscreen-fuzz-y", 6),
+	PROPERTY_ENTRY_BOOL("touchscreen-swapped-x-y"),
+	PROPERTY_ENTRY_STRING("firmware-name", "gsl3680-viglen-
connect-10.fw"),
+	PROPERTY_ENTRY_U32("silead,max-fingers", 10),
+	PROPERTY_ENTRY_BOOL("silead,home-button"),
+	{ }
+};
+
+static const struct ts_dmi_data viglen_connect_10_data = {
+	.acpi_name	= "MSSL1680:00",
+	.properties	= viglen_connect_10_props,
+};
+
 static const struct property_entry vinga_twizzle_j116_props[] = {
 	PROPERTY_ENTRY_U32("touchscreen-size-x", 1920),
 	PROPERTY_ENTRY_U32("touchscreen-size-y", 1280),
@@ -1522,6 +1542,14 @@ const struct dmi_system_id touchscreen_d
 		},
 	},
 	{
+		/* Viglen Connect 10 */
+		.driver_data = (void *)&viglen_connect_10_data,
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Viglen Ltd."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "Connect 10''
Tablet PC"),
+		},
+	},
+	{
 		/* Vinga Twizzle J116 */
 		.driver_data = (void *)&vinga_twizzle_j116_data,
 		.matches = {

Re: touchscreen_dmi.c Patch for Viglen Connect 10

[Hans de Goede]

Hi Mark,

On 10/27/21 04:11, Mark Stamp wrote:
> Hello. 
> 
> I previously spoke with you on a touchscreen issue
> 
> https://github.com/onitake/gsl-firmware/issues/178
> 
> With regard to that, I made a patch for touchscreen_dmi.c. It adds
> parameters for the Viglen Connect 10 tablet to get its touchscreen and
> home button working.
> 
> Signed-off-by: Mark Stamp <stamp497@googlemail.com>

Thank you for your patch.

Unfortunately your email program has wrapped some long lines:

> 
> --- touchscreen_dmi.c.orig	2021-10-27 01:26:47.842668712 +0100
> +++ touchscreen_dmi.c	2021-10-27 01:27:06.426298127 +0100
> @@ -938,6 +938,26 @@ static const struct ts_dmi_data trekstor
>  	.properties	= trekstor_surftab_wintron70_props,
>  };
>  
> +static const struct property_entry viglen_connect_10_props[] = {
> +	//PROPERTY_ENTRY_U32("touchscreen-min-x", 20),
> +	PROPERTY_ENTRY_U32("touchscreen-min-x", 0),
> +	PROPERTY_ENTRY_U32("touchscreen-min-y", 0),
> +	PROPERTY_ENTRY_U32("touchscreen-size-x", 1890),
> +	PROPERTY_ENTRY_U32("touchscreen-size-y", 1280),
> +	PROPERTY_ENTRY_U32("touchscrken-fuzz-x", 8),
> +	PROPERTY_ENTRY_U32("touchscreen-fuzz-y", 6),
> +	PROPERTY_ENTRY_BOOL("touchscreen-swapped-x-y"),
> +	PROPERTY_ENTRY_STRING("firmware-name", "gsl3680-viglen-
> connect-10.fw"),
> +	PROPERTY_ENTRY_U32("silead,max-fingers", 10),
> +	PROPERTY_ENTRY_BOOL("silead,home-button"),
> +	{ }
> +};
> +
> +static const struct ts_dmi_data viglen_connect_10_data = {
> +	.acpi_name	= "MSSL1680:00",
> +	.properties	= viglen_connect_10_props,
> +};
> +
>  static const struct property_entry vinga_twizzle_j116_props[] = {
>  	PROPERTY_ENTRY_U32("touchscreen-size-x", 1920),
>  	PROPERTY_ENTRY_U32("touchscreen-size-y", 1280),
> @@ -1522,6 +1542,14 @@ const struct dmi_system_id touchscreen_d
>  		},
>  	},
>  	{
> +		/* Viglen Connect 10 */
> +		.driver_data = (void *)&viglen_connect_10_data,
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Viglen Ltd."),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "Connect 10''
> Tablet PC"),

And at least for this line I'm not sure how to unwrap it (does there
need to be a space there or not ?).


> +		},
> +	},
> +	{
>  		/* Vinga Twizzle J116 */
>  		.driver_data = (void *)&vinga_twizzle_j116_data,
>  		.matches = {
> 
> 


Can you resend this using git send-email (preferred), or send the
diff as attachment instead of inlining it?

Regards,

Hans