[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAAG0J98T2gY9A8sbDOjfR_jqy9xQOh1Y65uSRbOsf4Tk9ZgcHg@mail.gmail.com>
Date: Mon, 22 Jul 2013 14:21:37 +0100
From: James Hogan <james@...anarts.com>
To: "Rafael J. Wysocki" <rjw@...k.pl>
Cc: Linus Torvalds <torvalds@...ux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Kamal Mostafa <kamal@...onical.com>,
Matthew Garrett <matthew.garrett@...ula.com>,
"Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
Steven Newbury <steve@...wbury.org.uk>
Subject: Re: Linux 3.11-rc2
On 22 July 2013 14:02, Rafael J. Wysocki <rjw@...k.pl> wrote:
> On Monday, July 22, 2013 12:08:40 AM James Hogan wrote:
>> On 21 July 2013 20:53, Linus Torvalds <torvalds@...ux-foundation.org> wrote:
>> > (b) we had a late change to how ACPI backlight handling is done on
>> > certain machines, and while this kind of thing really shouldn't be
>> > done outside the merge window, I ended up pulling it anyway. But I'd
>> > *really* like to have people test this thing particularly on laptops
>> > with intel-based graphics. It should only matter (and hopefully
>> > improve things) for the newer ones with BIOSes designed for Windows 8,
>> > but hey, the more testing, the better. Backlight handling has beenin
>> > painful before, so I'm mentioning this explicitly.
>>
>> 8c5bd7a "ACPI / video / i915: No ACPI backlight if firmware expects
>> Windows 8" breaks backlight control for me because
>> /sys/class/backlight/acpi_video0 disappears, and
>> /sys/class/backlight/intel_backlight doesn't seem to have any effect.
>>
>> Note that acpi_video0 only worked because I was applying "[PATCH]
>> drm/i915: quirk no PCH_PWM_ENABLE for Dell XPS13 backlight" [1], so
>> strictly speaking mainline already didn't work.
>
> James, sorry for breaking things for you. The patch you're mentioning is going
> to hit the mainline at one point anyway I suppose.
>
> In the meantime I received a report from Steven Newbury that these changes
> broke things for him too, so we need to revert commits 8c5bd7a and efaa14c.
> The other two commits in the series should be benign.
Feel free to Cc me on updated versions of the patches if you'd like me to test.
Cheers
James
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists