diff options
author | Danny Baumann <dannybaumann@web.de> | 2013-03-19 12:22:51 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-03-24 19:26:55 -0400 |
commit | 2d4128a25206685aaccaf14220c8436b11c6dc01 (patch) | |
tree | eae830a429a2602423592245fad57db097249c44 /drivers/acpi | |
parent | a89803df9c9b2d023227c48503066a728f1fed36 (diff) |
ACPI / video: Make logic a little easier to understand.
Make code paths a little easier to follow, and don't needlessly continue
list iteration.
Signed-off-by: Danny Baumann <dannybaumann@web.de>
Reviewed-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi')
-rw-r--r-- | drivers/acpi/video.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c index 387b3705279a..8522d14972cf 100644 --- a/drivers/acpi/video.c +++ b/drivers/acpi/video.c | |||
@@ -745,16 +745,17 @@ acpi_video_init_brightness(struct acpi_video_device *device) | |||
745 | */ | 745 | */ |
746 | if (use_bios_initial_backlight) { | 746 | if (use_bios_initial_backlight) { |
747 | for (i = 2; i < br->count; i++) | 747 | for (i = 2; i < br->count; i++) |
748 | if (level_old == br->levels[i]) | 748 | if (level_old == br->levels[i]) { |
749 | level = level_old; | 749 | level = level_old; |
750 | break; | ||
751 | } | ||
750 | } | 752 | } |
751 | goto set_level; | 753 | } else { |
754 | if (br->flags._BCL_reversed) | ||
755 | level_old = (br->count - 1) - level_old; | ||
756 | level = br->levels[level_old]; | ||
752 | } | 757 | } |
753 | 758 | ||
754 | if (br->flags._BCL_reversed) | ||
755 | level_old = (br->count - 1) - level_old; | ||
756 | level = br->levels[level_old]; | ||
757 | |||
758 | set_level: | 759 | set_level: |
759 | result = acpi_video_device_lcd_set_level(device, level); | 760 | result = acpi_video_device_lcd_set_level(device, level); |
760 | if (result) | 761 | if (result) |