Thread (4 messages) 4 messages, 2 authors, 2007-02-22

Re: ACPI: ibm-acpi: fix initial status of backlight device

From: Richard Purdie <hidden>
Date: 2007-02-22 10:03:19
Also in: linux-fbdev, lkml

On Wed, 2007-02-21 at 23:16 -0200, Henrique de Moraes Holschuh wrote:
NOTE: This patch needs an ACK from Richard Purdie before it can be merged,
as he might want to change the backlight class code instead.
As mentioned elsewhere, we can't do this in the class itself.
quoted hunk ↗ jump to hunk
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -1713,6 +1713,13 @@ static struct backlight_properties ibm_backlight_data = {
 
 static int brightness_init(void)
 {
+	int b;
+
+	b = brightness_get(NULL);
+	if (b < 0)
+		return b;
+	ibm_backlight_data.brightness = b;
+
 	ibm_backlight_device = backlight_device_register("ibm", NULL, NULL,
 							 &ibm_backlight_data);
This isn't against 2.6.21-rc1 which changed the backlight class a bit.
Basically, you need to set the brightness variable after
backlight_device_register(). It should be simple enough to do and fix
the problem the same way though.

Richard

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help