Messages in this thread | ![/](/images/icornerl.gif) | | Date | Mon, 22 Jul 2013 14:21:37 +0100 | Subject | Re: Linux 3.11-rc2 | From | James Hogan <> |
| |
On 22 July 2013 14:02, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Monday, July 22, 2013 12:08:40 AM James Hogan wrote: >> On 21 July 2013 20:53, Linus Torvalds <torvalds@linux-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
| ![\](/images/icornerr.gif) |