Re: [PATCH] acpi: Revert "ACPI / video: Add Dell Inspiron 5737 to the blacklist"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, January 22, 2016 11:41:05 AM Hans de Goede wrote:
> The quirk to get "acpi_backlight=vendor" behavior by default on the
> Dell Inspiron 5737 was added before we started doing
> "acpi_backlight=native" by default on Win8 ready machines.
> 
> Since we now avoid using acpi-video as backlight driver on these machines
> by default (using the native driver instead) we no longer need this quirk.
> 
> Moreover the vendor driver does not work after a suspend/resume where
> as the native driver does.
> 
> This reverts commit 08a56226d847 ("ACPI / video: Add Dell Inspiron 5737
> to the blacklist").
> 
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=111061
> Cc: stable@xxxxxxxxxxxxxxx

OK, but I think we need to limit the range of "stable" kernels this is
going to go into.

What's the least recent "stable" series one you'd like this to be applied to?

> Reported-and-tested-by: erusan@xxxxxxxxx
> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> ---
>  drivers/acpi/video_detect.c | 8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
> index 90e2d54..1316ddd 100644
> --- a/drivers/acpi/video_detect.c
> +++ b/drivers/acpi/video_detect.c
> @@ -135,14 +135,6 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
>  		DMI_MATCH(DMI_PRODUCT_NAME, "UL30A"),
>  		},
>  	},
> -	{
> -	.callback = video_detect_force_vendor,
> -	.ident = "Dell Inspiron 5737",
> -	.matches = {
> -		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> -		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5737"),
> -		},
> -	},
>  
>  	/*
>  	 * These models have a working acpi_video backlight control, and using
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]