[Nouveau] nouveau exposes backlight controls in presence of ACPI
Aaron Sowry
aaron+nv at aeneby.se
Tue Nov 2 07:51:53 PDT 2010
> However. There's also the platform-specific modules (thinkpad etc) that
> will provide their own backlight methods if the standard ACPI mechanism
> isn't available. I didn't see any immediately obvious way of knowing
> whether or not they were being provided.
>
> I'm not too certain the best way to deal with this, any ideas? :)
See attached patch, which prevents nouveau registering a backlight control if ACPI already provides one. If there are no glaring errors in the code/implementation then I would like to submit it.
Dealing with platform-specific modules is tricky because it looks like nouveau initialises itself quite early on in the boot process:
[aaron at jules ~]$ dmesg | grep ACPI
...
[ 3.359377] nouveau 0000:01:00.0: power state changed by ACPI to D0
[ 3.359560] nouveau 0000:01:00.0: power state changed by ACPI to D0
[ 4.860678] [drm] nouveau 0000:01:00.0: Detected ACPI backlight support, not registering control
[ 14.206997] ACPI: WMI: Skipping duplicate GUID 05901221-D566-11D1-B2F0-00A0C9062910
[ 14.207693] ACPI: WMI: Mapper loaded
[ 17.686907] thinkpad_acpi: ThinkPad ACPI Extras v0.24
[ 17.689517] thinkpad_acpi: This ThinkPad has standard ACPI backlight brightness control, supported by the ACPI video driver
[ 17.756039] thinkpad_acpi: Standard ACPI backlight interface available, not loading native one.
My suggestion would be that it is the responsibility of user-space to select an appropriate ACPI backlight control.
/Aaron
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nouveau_state.patch
Type: text/x-diff
Size: 813 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20101102/59839d7a/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 489 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20101102/59839d7a/attachment.pgp>
More information about the Nouveau
mailing list