Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752235AbaLRNZo (ORCPT ); Thu, 18 Dec 2014 08:25:44 -0500 Received: from mail-ob0-f172.google.com ([209.85.214.172]:48004 "EHLO mail-ob0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750983AbaLRNZm (ORCPT ); Thu, 18 Dec 2014 08:25:42 -0500 MIME-Version: 1.0 In-Reply-To: <5492C5A3.8090700@gmail.com> References: <5492C5A3.8090700@gmail.com> Date: Thu, 18 Dec 2014 08:25:42 -0500 X-Google-Sender-Auth: QjKZojSTtfqxRSLaues0e_XY7g8 Message-ID: Subject: Re: [PATCH v3 1/2] acpi: Add "acpi_osi=" for ASUS X200MA to enable, brightness keys From: Josh Boyer To: Dmitry Tunin Cc: "linux-acpi@vger.kernel.org" , linux-kernel , "stable@vger.kernel.org" , Aaron Lu , Matthew Garrett Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 18, 2014 at 7:16 AM, Dmitry Tunin wrote: > Add "acpi_osi=" quirk for ASUS X200MA > More information can be found in UX302LA bugreport > https://bugzilla.kernel.org/show_bug.cgi?id=70241 > > X200MA is affected by th e same bug and the solution has been tested > > I reported it to launchpad too > https://bugs.launchpad.net/ubuntu/bug/1400068 > > Cc: stable@vger.kernel.org > Signed-off-by: Dmitry Tunin > The UX301LA models have the same setup, with 16 devices instead of 8. I discussed this with Matthew Garrett a bit and a more ideal approach is to actually enumerate all 16 devices between the firmware and i915 driver. The problem we have is that the intel opregion spec that hasn't been updated since 2008 doesn't cover the additional 8 devices. Anyway, that is being tracked here: https://bugs.freedesktop.org/show_bug.cgi?id=81762 > --- > drivers/acpi/blacklist.c | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/drivers/acpi/blacklist.c b/drivers/acpi/blacklist.c > index 7556e7c..f78ed08 100644 > --- a/drivers/acpi/blacklist.c > +++ b/drivers/acpi/blacklist.c > @@ -162,6 +162,17 @@ static int __init dmi_disable_osi_win8(const struct > dmi_system_id *d) > acpi_osi_setup("!Windows 2012"); > return 0; > } > +/* > + * Some ASUS models firmware declares 16 devices instead of 8 > + * 'acpi_osi=' kernel parameter fixes it > + * Without this parameters brightness keys Fn+F5 and F6 do not work > + */ > +static int __init dmi_disable_osi_all(const struct dmi_system_id *d) > +{ > + printk(KERN_NOTICE PREFIX "DMI detected: %s\n", d->ident); > + acpi_osi_setup(""); > + return 0; > +} What does this do to the rest of the function keys? Does everything else still work as expected, including suspend and resume? > static struct dmi_system_id acpi_osi_dmi_table[] __initdata = { > { > @@ -371,6 +382,15 @@ static struct dmi_system_id acpi_osi_dmi_table[] > __initdata = { > DMI_MATCH(DMI_PRODUCT_NAME, "1015PX"), > }, > }, > + /* Without this brightness keys do not work */ > + { > + .callback = dmi_disable_osi_all, > + .ident = "ASUSTeK COMPUTER INC. X200MA", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."), > + DMI_MATCH(DMI_PRODUCT_NAME, "X200MA"), > + }, > + }, > {} > }; > > -- > 1.9.1 > -- > To unsubscribe from this list: send the line "unsubscribe stable" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/