Re: [PATCH] acpi/x86: enable touchscreen on Dell Venue Pro 7139

[Andy Shevchenko]

On Sat, 2018-06-09 at 18:18 -0400, Tristian Celestin wrote:
> Permanently enable the SYNA7500 touchscreen device on the Dell
> Venue Pro 7139. The DSDT hides the touchscreen ACPI device on
> the 7139 in the same fashion as the 7130, and needs to
> be enabled in the same way.
> 

Cc: Mario.

> Signed-off-by: Tristian Celestin <tristiancelestin@xxxxxxxxxxxx>
> ---
>  drivers/acpi/x86/utils.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/acpi/x86/utils.c b/drivers/acpi/x86/utils.c
> index ec5b0f1..3a13b2a 100644
> --- a/drivers/acpi/x86/utils.c
> +++ b/drivers/acpi/x86/utils.c
> @@ -62,14 +62,20 @@ struct always_present_id {
>  	 */
>  	ENTRY("INT0002", "1", ICPU(INTEL_FAM6_ATOM_AIRMONT), {}),
>  	/*
> -	 * On the Dell Venue 11 Pro 7130 the DSDT hides the
> touchscreen ACPI
> -	 * device until a certain time after
> _SB.PCI0.GFX0.LCD.LCD1._ON gets
> -	 * called has passed *and* _STA has been called at least 3
> times since.
> +	 * On the Dell Venue 11 Pro 7130 and 7139, the DSDT hides
> +	 * the touchscreen ACPI device until a certain time
> +	 * after _SB.PCI0.GFX0.LCD.LCD1._ON gets called has passed
> +	 * *and* _STA has been called at least 3 times since.
>  	 */
>  	ENTRY("SYNA7500", "1", ICPU(INTEL_FAM6_HASWELL_ULT), {
>  		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
>  		DMI_MATCH(DMI_PRODUCT_NAME, "Venue 11 Pro 7130"),
>  	      }),
> +	ENTRY("SYNA7500", "1", ICPU(INTEL_FAM6_HASWELL_ULT), {
> +		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +		DMI_MATCH(DMI_PRODUCT_NAME, "Venue 11 Pro 7139"),
> +	      }),
> +
>  	/*
>  	 * The GPD win BIOS dated 20170221 has disabled the
> accelerometer, the
>  	 * drivers sometimes cause crashes under Windows and this is
> how the

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy