Re: [PATCH v3 2/2] acpi: Remove _OSI(Linux) for ThinkPads

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

 



On Fri, 17 Oct 2014, Andy Lutomirski wrote:
> AFAICT the only reason to set _OSI(Linux) on ThinkPads is to get
> sensible mute button behavior.  Now that the thinkpad_acpi driver
> can do this on is own, there is no reason to keep the ACPI
> quirk.
> 
> Cc: Len Brown <len.brown@xxxxxxxxx>
> Cc: Jerone Young <jerone.young@xxxxxxxxxxxxx>
> Signed-off-by: Andy Lutomirski <luto@xxxxxxxxxxxxxx>

Acked-by: Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>

> ---
>  drivers/acpi/blacklist.c | 54 ------------------------------------------------
>  1 file changed, 54 deletions(-)
> 
> diff --git a/drivers/acpi/blacklist.c b/drivers/acpi/blacklist.c
> index 36eb42e3b0bb..4a3cbb5d3c55 100644
> --- a/drivers/acpi/blacklist.c
> +++ b/drivers/acpi/blacklist.c
> @@ -265,60 +265,6 @@ static struct dmi_system_id acpi_osi_dmi_table[] __initdata = {
>  	 */
>  
>  	/*
> -	 * Lenovo has a mix of systems OSI(Linux) situations
> -	 * and thus we can not wildcard the vendor.
> -	 *
> -	 * _OSI(Linux) helps sound
> -	 * DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad R61"),
> -	 * DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T61"),
> -	 * T400, T500
> -	 * _OSI(Linux) has Linux specific hooks
> -	 * DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X61"),
> -	 * _OSI(Linux) is a NOP:
> -	 * DMI_MATCH(DMI_PRODUCT_VERSION, "3000 N100"),
> -	 * DMI_MATCH(DMI_PRODUCT_VERSION, "LENOVO3000 V100"),
> -	 */
> -	{
> -	.callback = dmi_enable_osi_linux,
> -	.ident = "Lenovo ThinkPad R61",
> -	.matches = {
> -		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> -		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad R61"),
> -		},
> -	},
> -	{
> -	.callback = dmi_enable_osi_linux,
> -	.ident = "Lenovo ThinkPad T61",
> -	.matches = {
> -		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> -		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T61"),
> -		},
> -	},
> -	{
> -	.callback = dmi_enable_osi_linux,
> -	.ident = "Lenovo ThinkPad X61",
> -	.matches = {
> -		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> -		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X61"),
> -		},
> -	},
> -	{
> -	.callback = dmi_enable_osi_linux,
> -	.ident = "Lenovo ThinkPad T400",
> -	.matches = {
> -		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> -		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T400"),
> -		},
> -	},
> -	{
> -	.callback = dmi_enable_osi_linux,
> -	.ident = "Lenovo ThinkPad T500",
> -	.matches = {
> -		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> -		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T500"),
> -		},
> -	},
> -	/*
>  	 * Without this this EEEpc exports a non working WMI interface, with
>  	 * this it exports a working "good old" eeepc_laptop interface, fixing
>  	 * both brightness control, and rfkill not working.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

------------------------------------------------------------------------------
_______________________________________________
ibm-acpi-devel mailing list
ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel




[Index of Archives]     [Linux ACPI]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Photo]     [Yosemite Photos]     [Yosemite Advice]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]

  Powered by Linux