[Intel-gfx] [RFC PATCH 0/2] i915 brightness control
mjg59 at srcf.ucam.org
Mon Sep 13 10:48:48 PDT 2010
On Wed, Jun 02, 2010 at 03:11:40PM -0700, Kamal Mostafa wrote:
> In order to avoid the often-dysfunctional native acpi brightness control
> methods, a new acpi_brightness_hook interface is made available.
> The i915 driver uses the new hook to take over brightness control.
> Boot with i915.brightness=0 to disable.
I've looked into this issue more closely and think I've worked out the
underlying problem. The system in question appears to have two GPUs and
exposes two ACPI backlight devices. Both of these are associated with
existing PCI devices, so we don't ignore either of them because of that.
Further, one of them (the AMD one) implements the spec properly and
should work. We don't seem to perform a more fine-grained check to
identify whether every ACPI backlight has all the required methods, and
so as a result we provide both the working one and the non-working one.
Having thought about this some more, I don't think this is the right
approach. We should be ensuring that every backlight ahs all the
required methods and then dropping the one that doesn't. This should be
replaced with a native i915 backlight, and I sent patches to do that
Matthew Garrett | mjg59 at srcf.ucam.org
More information about the Intel-gfx