[Nouveau] nouveau exposes backlight controls in presence of ACPI
aaron+nv at aeneby.se
Tue Nov 2 12:28:02 PDT 2010
> It was my understanding that the future direction the kernel was going
> to go with this is that all detected backlight devices would be
> exported, with a tag in sysfs describing the type of interface
> (platform, firmware, direct hardware, whichever). Then it would be up to
> userspace to pick the correct backlight device to use based on the tags.
> Nothing stopping a user from manipulating a different one directly, with
> possible interesting side-effects, of course.
Interesting. I guess this would be the best long-term solution, but short-term we can't proceed based on speculation. Nor can we really trust userspace to make the correct decisions at the moment, as the controls don't normally provide enough information to be able to do that reliably.
> Note that the thinkpad-acpi module at the moment is also checking for
> the ACPI video device to be loaded, and will disable itself in that
> case. So the case that causes problems right now is if you have e.g.
> both thinkpad-acpi and nouveau, but no ACPI video device?
The problem in my case is when both ACPI and nouveau controls are exposed - gnome-power-manager prioritises the nouveau control over ACPI (I've raised a bug for this too) and because my card isn't supported properly by the driver, I get a range of approximately useless brightness levels. As I see it, we have 3 related issues when it comes to backlight control:
0) Fixing NV50 backlight support in nouveau for those chipsets with max_brightness > 1025
1) Determining under which circumstances we should expose kernel-level controls (IMHO only when no better alternatives are available)
2) Determining how userspace should select an appropriate control when more than one is available
Matthew Garrett addresses some of those issues in a fairly decent rant here:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 489 bytes
Desc: Digital signature
More information about the Nouveau