All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Dmitry Tunin <hanipouspilot@gmail.com>
Cc: Josh Boyer <jwboyer@fedoraproject.org>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>,
	Aaron Lu <aaron.lu@intel.com>
Subject: Re: [PATCH v3 1/2] acpi: Add "acpi_osi=" for ASUS X200MA to enable, brightness keys
Date: Thu, 18 Dec 2014 14:36:23 +0000	[thread overview]
Message-ID: <20141218143623.GA7335@srcf.ucam.org> (raw)
In-Reply-To: <5492E5F9.2000504@gmail.com>

That's odd - it looks like the default behaviour in the absence of any 
OSI strings is to just default to an OSYS value that matches Windows 
2012, so disabling the 2013 value should be equivalent. If it's not, 
it's definitely not safe to turn off all values - there are several 
devices that change behaviour as a result.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

  reply	other threads:[~2014-12-18 14:36 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-18 12:16 [PATCH v3 1/2] acpi: Add "acpi_osi=" for ASUS X200MA to enable, brightness keys Dmitry Tunin
2014-12-18 12:19 ` [PATCH 2/2] acpi: Add "acpi_osi=" for ASUS UX302LA " Dmitry Tunin
2014-12-18 13:25 ` [PATCH v3 1/2] acpi: Add "acpi_osi=" for ASUS X200MA " Josh Boyer
2014-12-18 13:27   ` Matthew Garrett
2014-12-18 13:28   ` Dmitry Tunin
2014-12-18 13:28     ` Dmitry Tunin
2014-12-18 13:40   ` Dmitry Tunin
2014-12-18 13:57     ` Matthew Garrett
2014-12-18 14:03       ` Dmitry Tunin
2014-12-18 14:03         ` Dmitry Tunin
2014-12-18 14:04       ` Dmitry Tunin
2014-12-18 14:04         ` Dmitry Tunin
2014-12-18 14:09         ` Matthew Garrett
2014-12-18 14:22           ` Dmitry Tunin
2014-12-18 14:22             ` Dmitry Tunin
2014-12-18 14:26             ` Matthew Garrett
2014-12-18 14:34               ` Dmitry Tunin
2014-12-18 14:36                 ` Matthew Garrett [this message]
2014-12-18 14:40                   ` Dmitry Tunin
2014-12-18 14:43                     ` Matthew Garrett
2014-12-18 14:50                       ` Matthew Garrett
2014-12-18 18:09                         ` Len Brown
2014-12-18 18:20                           ` Dmitry Tunin
2014-12-18 20:04                         ` Matthew Garrett
2014-12-18 21:20                           ` Dmitry Tunin
2014-12-18 14:06       ` Dmitry Tunin
2014-12-18 14:06         ` Dmitry Tunin
2014-12-19  1:46   ` Aaron Lu
2014-12-19  3:23     ` Aaron Lu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141218143623.GA7335@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=aaron.lu@intel.com \
    --cc=hanipouspilot@gmail.com \
    --cc=jwboyer@fedoraproject.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.