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

From: Mourad De Clerck
Date: Tue Jan 11 2011 - 11:42:26 EST


On 04/01/11 23:36, Andrew Morton wrote:
> I applied this, and now I'll unapply it because Matthew's patch series
> "mbp_nvidia_bl: Remove DMI dependency" appears to make your change
> obsolete.
>
> I'll cc you on those three patches - it will be great if you are able
> to find time to test them, thanks.

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.

Thanks,

Mourad

PS: something like this, maybe?
--
diff --git a/src/gpm-backlight-helper.c b/src/gpm-backlight-helper.c
index 27aed09..a8ef608 100644
--- a/src/gpm-backlight-helper.c
+++ b/src/gpm-backlight-helper.c
@@ -58,6 +58,7 @@ gcm_backlight_helper_get_best_backlight ()
"thinkpad_screen",
"acpi_video1",
"mbp_backlight",
+ "apple_backlight",
"acpi_video0",
"fujitsu-laptop",
"sony",
--
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/