diff options
| author | Alexey Starikovskiy <astarikovskiy@suse.de> | 2007-09-03 12:29:58 (GMT) |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2007-09-05 18:24:28 (GMT) |
| commit | 4500ca8e221e72cc38e7c239b9b1a041031ee450 (patch) | |
| tree | 1e44b5134b6b715f30a9240d57a463049bedf28d | |
| parent | 40ffbfad6bb79a99cc7627bdaca0ee22dec526f6 (diff) | |
| download | linux-4500ca8e221e72cc38e7c239b9b1a041031ee450.tar.xz | |
ACPI: video: Don't call absent methods
Signed-off-by: Ryan May <rmay@ou.edu>
Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
Signed-off-by: Len Brown <len.brown@intel.com>
| -rw-r--r-- | drivers/acpi/video.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index 3c9bb85..83aa41c 100644 --- a/drivers/acpi/video.c +++ b/drivers/acpi/video.c @@ -409,14 +409,16 @@ acpi_video_device_lcd_query_levels(struct acpi_video_device *device, static int acpi_video_device_lcd_set_level(struct acpi_video_device *device, int level) { - int status; + int status = AE_OK; union acpi_object arg0 = { ACPI_TYPE_INTEGER }; struct acpi_object_list args = { 1, &arg0 }; arg0.integer.value = level; - status = acpi_evaluate_object(device->dev->handle, "_BCM", &args, NULL); - + if (device->cap._BCM) + status = acpi_evaluate_object(device->dev->handle, "_BCM", + &args, NULL); + device->brightness->curr = level; printk(KERN_DEBUG "set_level status: %x\n", status); return status; } @@ -425,11 +427,11 @@ static int acpi_video_device_lcd_get_level_current(struct acpi_video_device *device, unsigned long *level) { - int status; - - status = acpi_evaluate_integer(device->dev->handle, "_BQC", NULL, level); - - return status; + if (device->cap._BQC) + return acpi_evaluate_integer(device->dev->handle, "_BQC", NULL, + level); + *level = device->brightness->curr; + return AE_OK; } static int |