Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751197AbaLRSJ1 (ORCPT ); Thu, 18 Dec 2014 13:09:27 -0500 Received: from mail-yh0-f48.google.com ([209.85.213.48]:42836 "EHLO mail-yh0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751009AbaLRSJZ (ORCPT ); Thu, 18 Dec 2014 13:09:25 -0500 MIME-Version: 1.0 In-Reply-To: <20141218145054.GA7745@srcf.ucam.org> References: <5492D938.1020907@gmail.com> <20141218135705.GD4918@srcf.ucam.org> <5492DEE7.8010407@gmail.com> <20141218140930.GA6436@srcf.ucam.org> <5492E32E.2030602@gmail.com> <20141218142629.GA6924@srcf.ucam.org> <5492E5F9.2000504@gmail.com> <20141218143623.GA7335@srcf.ucam.org> <5492E753.7090505@gmail.com> <20141218144337.GA7613@srcf.ucam.org> <20141218145054.GA7745@srcf.ucam.org> Date: Thu, 18 Dec 2014 13:09:24 -0500 X-Google-Sender-Auth: XAOHyDu-pZJzNs-JHbBrHR7Y9PM Message-ID: Subject: Re: [PATCH v3 1/2] acpi: Add "acpi_osi=" for ASUS X200MA to enable, brightness keys From: Len Brown To: Matthew Garrett Cc: Dmitry Tunin , Josh Boyer , "linux-acpi@vger.kernel.org" , linux-kernel , "stable@vger.kernel.org" , Aaron Lu Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org NAK on addoing this acpi_osi= quirk to upstream Linux. The reason is that problem is being actively debugged. Quirks in upstream Linux are for when we give up. Sure, it is perfectly okay for individual users to invoke it. It is even fine for a distro to check in this workaround if they care about that box working *now* and they are not concerned about side effects. But the real fix is in development, and checking workarounds with possible other side-effects into up-stream Linux is just a distraction from getting it right for not just this machine, but other machines w/ the same issue. thanks, Len Brown, Intel Open Source Technology Center ps. lots of people are mis-using acpi_osi= in debugging, please consult Documentation/kernel-parameters.txt for a detailed description of how this cmdline option works. Note, in particular, that acpi_osi=XYZ by itself will _add_ XYZ to the list of recognized strings. If that string is already built into Linux (eg. "Windows 2012"), then adding it again has no effect. -- 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/