[SRU Z/A] [PATCH 1/1] ACPI / blacklist: add _REV quirk for Dell Inspiron 7537

Colin Ian King colin.king at canonical.com
Fri Jun 2 07:57:55 UTC 2017


On 02/06/17 06:21, Kai-Heng Feng wrote:
> From: Kai Heng Feng <kai.heng.feng at canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1678590
> 
> The battery can only be detected after AC power adapter event.
> 
> Adding the machine to acpi_rev_dmi_table[] can work around this
> issue.
> 
> Link: https://bugs.launchpad.net/bugs/1678590
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=105721
> Signed-off-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki at intel.com>
> (cherry picked from commit 2cff319e9666f534a7538aa1f4178d1f799b068a)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
> ---
>  drivers/acpi/blacklist.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/acpi/blacklist.c b/drivers/acpi/blacklist.c
> index 4421f7c9981c..bb542acc0574 100644
> --- a/drivers/acpi/blacklist.c
> +++ b/drivers/acpi/blacklist.c
> @@ -188,6 +188,14 @@ static struct dmi_system_id acpi_rev_dmi_table[] __initdata = {
>  		      DMI_MATCH(DMI_PRODUCT_NAME, "Latitude 3350"),
>  		},
>  	},
> +	{
> +	 .callback = dmi_enable_rev_override,
> +	 .ident = "DELL Inspiron 7537",
> +	 .matches = {
> +		      DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +		      DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 7537"),
> +		},
> +	},
>  #endif
>  	{}
>  };
> 

Thanks for the fix. It's upstream, tested and is constrained to just one
specific machine and works around the broken _REV.

Is there any reason it's not applicable for other releases too (Xenial,
Yaketty)?

Acked-by: Colin Ian King  <colin.king at canonical.com>




More information about the kernel-team mailing list