|
From: Henrique de M. H. <hm...@hm...> - 2006-11-22 20:14:32
|
From: Henrique de Moraes Holschuh <hm...@hm...>
The current code names the backlight device that controls the Thinkpad
display brightness "ibm". This is not nearly descriptive enough.
The ThinkPad can connect to more than one flat pannel display using the
external VGA or DVI output, which we may be able to control over DCC
someday.
Also, Lenovo may introduce a ThinkPad with more than one internal LCD
screen in the future. Or we could have LCD gadgets from ibm with
backlights connected to the ThinkPad...
This patch renames the main backlight device from "ibm" to
"built-in_screen", which is far more descriptive and future-proof.
Maybe the backlight devices refering to the built-in screen of laptops
in other drivers should also be renamed to "built-in_screen" ?
Signed-off-by: Henrique de Moraes Holschuh <hm...@hm...>
Cc: Holger Macht <hm...@su...>
Cc: Andy Whitcroft <ap...@sh...>
Cc: Andrew Morton <ak...@os...>
Cc: Len Brown <len...@in...>
---
drivers/acpi/ibm_acpi.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c
index 96e945c..dab3fd1 100644
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -1718,8 +1718,9 @@ static struct backlight_properties ibm_b
=20
static int brightness_init(void)
{
- ibm_backlight_device =3D backlight_device_register("ibm", NULL,
- &ibm_backlight_data);
+ ibm_backlight_device =3D backlight_device_register(
+ "built-in_screen", 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);
|
