Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8

All of lore.kernel.org
 help / color / mirror / Atom feed

From: Matthew Garrett <matthew.garrett@nebula.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>, Aaron Lu <aaron.lu@intel.com>,
	ACPI Devel Mailing List <linux-acpi@vger.kernel.org>,
	Seth Forshee <seth.forshee@canonical.com>,
	"Lee, Chun-Yi" <joeyli.kernel@gmail.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Len Brown <lenb@kernel.org>,
	Igor Gnatenko <i.gnatenko.brain@gmail.com>,
	Yves-Alexis Perez <corsac@debian.org>,
	Felipe Contreras <felipe.contreras@gmail.com>,
	Lee Chun-Yi <jlee@novell.com>,
	Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Subject: Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8
Date: Mon, 9 Sep 2013 15:38:46 +0000	[thread overview]
Message-ID: <1378741125.17982.1.camel@x230.lan> (raw)
In-Reply-To: <20130909152118.GG27291@phenom.ffwll.local>

On Mon, 2013-09-09 at 17:21 +0200, Daniel Vetter wrote:

> If Win8 is as broken as we are I'm ok with the module option. It just
> sounded to me like right now we don't know of a way to make all machines
> somewhat happy, combined with the other pile of random backlight issues
> the assumption that we do something (maybe something a bit racy) that
> windows doesn't do isn't too far-fetched. So I'm not wary of the machines
> where the aml is busted for acpi_os=win8, but for the others where this
> broke stuff.

Windows 8 isn't broken because (as far as we can tell) the Intel drivers
under Windows 8 never use the ACPI backlight set function. Of course, it
would be nice to have that confirmed by Intel.

-- 
Matthew Garrett <matthew.garrett@nebula.com>

WARNING: multiple messages have this Message-ID (diff)

From: Matthew Garrett <matthew.garrett@nebula.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>, Aaron Lu <aaron.lu@intel.com>,
	"ACPI Devel Mailing List" <linux-acpi@vger.kernel.org>,
	Seth Forshee <seth.forshee@canonical.com>,
	"Lee, Chun-Yi" <joeyli.kernel@gmail.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Len Brown <lenb@kernel.org>,
	Igor Gnatenko <i.gnatenko.brain@gmail.com>,
	Yves-Alexis Perez <corsac@debian.org>,
	"Felipe Contreras" <felipe.contreras@gmail.com>,
	Lee Chun-Yi <jlee@novell.com>,
	Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Subject: Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8
Date: Mon, 9 Sep 2013 15:38:46 +0000	[thread overview]
Message-ID: <1378741125.17982.1.camel@x230.lan> (raw)
In-Reply-To: <20130909152118.GG27291@phenom.ffwll.local>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 919 bytes --]

On Mon, 2013-09-09 at 17:21 +0200, Daniel Vetter wrote:

> If Win8 is as broken as we are I'm ok with the module option. It just
> sounded to me like right now we don't know of a way to make all machines
> somewhat happy, combined with the other pile of random backlight issues
> the assumption that we do something (maybe something a bit racy) that
> windows doesn't do isn't too far-fetched. So I'm not wary of the machines
> where the aml is busted for acpi_os=win8, but for the others where this
> broke stuff.

Windows 8 isn't broken because (as far as we can tell) the Intel drivers
under Windows 8 never use the ACPI backlight set function. Of course, it
would be nice to have that confirmed by Intel.

-- 
Matthew Garrett <matthew.garrett@nebula.com>
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

next prev parent reply	other threads:[~2013-09-09 15:38 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-09  8:37 [PATCH 0/2] Rework ACPI video driver Aaron Lu
2013-09-09  8:40 ` [PATCH 1/2] ACPI / video: seperate backlight control and event interface Aaron Lu
2013-09-10  5:23   ` Igor Gnatenko
2013-09-09  8:42 ` [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8 Aaron Lu
2013-09-09  8:42   ` Aaron Lu
2013-09-09  9:32   ` Daniel Vetter
2013-09-09  9:32     ` Daniel Vetter
2013-09-09 12:16     ` Rafael J. Wysocki
2013-09-09 15:21       ` Daniel Vetter
2013-09-09 15:38         ` Matthew Garrett [this message]
2013-09-09 15:38           ` Matthew Garrett
2013-09-09 20:23         ` Rafael J. Wysocki
2013-09-10 13:53           ` Jani Nikula
2013-09-10 13:53             ` Jani Nikula
2013-09-10 13:56             ` Matthew Garrett
2013-09-10 13:56               ` Matthew Garrett
2013-09-10 14:21               ` Jani Nikula
2013-09-10 14:21                 ` Jani Nikula
2013-09-10 14:21                 ` Matthew Garrett
2013-09-10 14:21                   ` Matthew Garrett
2013-09-10 19:23             ` Rafael J. Wysocki
2013-09-11  1:32               ` Aaron Lu
2013-09-11  8:45                 ` Jani Nikula
2013-09-11  8:45                   ` Jani Nikula
2013-09-11  8:45                   ` Matthew Garrett
2013-09-11  8:45                     ` Matthew Garrett
2013-09-11  9:09                     ` Yves-Alexis Perez
2013-09-11  9:09                       ` Yves-Alexis Perez
2013-09-11 10:29                     ` Jani Nikula
2013-09-11 10:29                       ` Jani Nikula
2013-09-11 10:30                       ` Matthew Garrett
2013-09-11 10:30                         ` Matthew Garrett
2013-09-12  2:26                   ` Aaron Lu
2013-09-10  6:30     ` Aaron Lu
2013-09-09 11:44   ` Igor Gnatenko
2013-09-10  3:27     ` Aaron Lu
2013-09-10  5:13       ` Igor Gnatenko
2013-09-10  5:16         ` Aaron Lu
2013-09-10  5:22           ` Igor Gnatenko
2013-09-10  5:42             ` Aaron Lu
2013-09-10  5:23   ` Igor Gnatenko

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=1378741125.17982.1.camel@x230.lan \
    --to=matthew.garrett@nebula.com \
    --cc=aaron.lu@intel.com \
    --cc=corsac@debian.org \
    --cc=daniel.vetter@ffwll.ch \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=felipe.contreras@gmail.com \
    --cc=hmh@hmh.eng.br \
    --cc=i.gnatenko.brain@gmail.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jlee@novell.com \
    --cc=joeyli.kernel@gmail.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rjw@sisk.pl \
    --cc=seth.forshee@canonical.com \
    /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.