diff options
| author | Mattia Dongili | 2007-01-13 23:04:41 +0100 |
|---|---|---|
| committer | Len Brown | 2007-02-13 09:06:44 +0100 |
| commit | 7df03b82ed081777d2393ff8a5fb9d4a3a560f26 (patch) | |
| tree | 908c83698ec370bef246396fcc9e0acc30fc2a21 /drivers/acpi/sony_acpi.c | |
| parent | sony_acpi: Allow multiple sony_acpi_values for the same .name (diff) | |
| download | kernel-qcow2-linux-7df03b82ed081777d2393ff8a5fb9d4a3a560f26.tar.gz kernel-qcow2-linux-7df03b82ed081777d2393ff8a5fb9d4a3a560f26.tar.xz kernel-qcow2-linux-7df03b82ed081777d2393ff8a5fb9d4a3a560f26.zip | |
sony_acpi: Fix sony_acpi backlight registration and unregistration
Initialize the current brightness if the driver registration
was successful and unregister the driver in the error exit path.
Signed-off-by: Mattia Dongili <malattia@linux.it>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/sony_acpi.c')
| -rw-r--r-- | drivers/acpi/sony_acpi.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/acpi/sony_acpi.c b/drivers/acpi/sony_acpi.c index c65f5893e0e2..c01d98db2fc1 100644 --- a/drivers/acpi/sony_acpi.c +++ b/drivers/acpi/sony_acpi.c @@ -354,9 +354,14 @@ static int sony_acpi_add(struct acpi_device *device) if (ACPI_SUCCESS(acpi_get_handle(sony_acpi_handle, "GBRT", &handle))) { sony_backlight_device = backlight_device_register("sony", NULL, NULL, &sony_backlight_properties); + if (IS_ERR(sony_backlight_device)) { printk(LOG_PFX "unable to register backlight device\n"); + sony_backlight_device = NULL; } + else + sony_backlight_properties.brightness = + sony_backlight_get_brightness(sony_backlight_device); } for (item = sony_acpi_values; item->name; ++item) { @@ -400,6 +405,9 @@ static int sony_acpi_add(struct acpi_device *device) return 0; outproc: + if (sony_backlight_device) + backlight_device_unregister(sony_backlight_device); + for (item = sony_acpi_values; item->name; ++item) if (item->proc) remove_proc_entry(item->name, acpi_device_dir(device)); |