Re: mbp_nvidia_bl: Remove DMI dependency (was: Re: [PATCH,RESEND]backlight: mbp_nvidia_bl - add support for MacBookPro7,1)

[Richard Hughes]

On 11 January 2011 16:48, Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote:
> On Tue, Jan 11, 2011 at 05:42:12PM +0100, Mourad De Clerck wrote:
>> A bit late, but I tested this patch series and it works for me.
>>
>> The only slight change is that gnome-power-manager needs to be updated
>> to use /sys/class/backlight/apple_backlight instead of mbp_backlight.
>
> gnome-power-manager needs not to have a hardcoded list of backlights.

It's only uses the PolicyKit-enabled utility if there is no XBACKLIGHT
driver support. The list is put there as for a long time people had
multiple backlights, where, for instance thinkpad_screen worked but
not acpi_video1.

If the named backlight file isn't present in the override list, then
we just use the first one in /sys/class/backlight

Mourad, how many devices do you have in /sys/class/backlight?

Richard.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/