Gitweb:
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=fb87a811a4c232e2af8d746dc75330cbe5b0780c
Commit: fb87a811a4c232e2af8d746dc75330cbe5b0780c
Parent: 2df910b4c3edcce9a0c12394db6f5f4a6e69c712
Author: Henrique de Moraes Holschuh <[EMAIL PROTECTED]>
AuthorDate: Sat Nov 25 16:35:09 2006 -0200
Committer: Len Brown <[EMAIL PROTECTED]>
CommitDate: Thu Dec 7 01:38:45 2006 -0500
ACPI: ibm-acpi: backlight device cleanup
This patch cleans up the recently added backlight device support by Holger
Macht <[EMAIL PROTECTED]> to fit well with the rest of the code, using the
ibms struct as the other "subdrivers" in ibm-acpi.
Signed-off-by: Henrique de Moraes Holschuh <[EMAIL PROTECTED]>
---
drivers/acpi/ibm_acpi.c | 50 +++++++++++++++++++++++++++++-----------------
1 files changed, 31 insertions(+), 19 deletions(-)
diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c
index fbb4970..36e2667 100644
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -397,7 +397,7 @@ struct ibm_struct {
static struct proc_dir_entry *proc_dir = NULL;
-static struct backlight_device *ibm_backlight_device;
+static struct backlight_device *ibm_backlight_device = NULL;
#define onoff(status,bit) ((status) & (1 << (bit)) ? "on" : "off")
#define enabled(status,bit) ((status) & (1 << (bit)) ? "enabled" : "disabled")
@@ -1639,6 +1639,7 @@ static int brightness_get(struct backlight_device *bd)
return -EIO;
level &= 0x7;
+
return level;
}
@@ -1713,6 +1714,33 @@ static int brightness_update_status(struct
backlight_device *bd)
return brightness_set(bd->props->brightness);
}
+static struct backlight_properties ibm_backlight_data = {
+ .owner = THIS_MODULE,
+ .get_brightness = brightness_get,
+ .update_status = brightness_update_status,
+ .max_brightness = 7,
+};
+
+static int brightness_init(void)
+{
+ ibm_backlight_device = backlight_device_register("ibm", NULL,
+ &ibm_backlight_data);
+ if (IS_ERR(ibm_backlight_device)) {
+ printk(IBM_ERR "Could not register backlight device\n");
+ return PTR_ERR(ibm_backlight_device);
+ }
+
+ return 0;
+}
+
+static void brightness_exit(void)
+{
+ if (ibm_backlight_device) {
+ backlight_device_unregister(ibm_backlight_device);
+ ibm_backlight_device = NULL;
+ }
+}
+
static int volume_offset = 0x30;
static int volume_read(char *p)
@@ -2377,6 +2405,8 @@ static struct ibm_struct ibms[] = {
.name = "brightness",
.read = brightness_read,
.write = brightness_write,
+ .init = brightness_init,
+ .exit = brightness_exit,
},
{
.name = "volume",
@@ -2642,20 +2672,10 @@ IBM_PARAM(brightness);
IBM_PARAM(volume);
IBM_PARAM(fan);
-static struct backlight_properties ibm_backlight_data = {
- .owner = THIS_MODULE,
- .get_brightness = brightness_get,
- .update_status = brightness_update_status,
- .max_brightness = 7,
-};
-
static void acpi_ibm_exit(void)
{
int i;
- if (ibm_backlight_device)
- backlight_device_unregister(ibm_backlight_device);
-
for (i = ARRAY_SIZE(ibms) - 1; i >= 0; i--)
ibm_exit(&ibms[i]);
@@ -2758,14 +2778,6 @@ static int __init acpi_ibm_init(void)
}
}
- ibm_backlight_device = backlight_device_register("ibm", NULL,
- &ibm_backlight_data);
- if (IS_ERR(ibm_backlight_device)) {
- printk(IBM_ERR "Could not register ibm backlight device\n");
- ibm_backlight_device = NULL;
- acpi_ibm_exit();
- }
-
return 0;
}
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
