Make code paths a little easier to follow, and don't needlessly continue
list iteration.

Signed-off-by: Danny Baumann <dannybaum...@web.de>
---
 drivers/acpi/video.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index 9c33871..5ef329a 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
@@ -736,16 +736,17 @@ acpi_video_init_brightness(struct acpi_video_device 
*device)
                 */
                if (use_bios_initial_backlight) {
                        for (i = 2; i < br->count; i++)
-                               if (level_old == br->levels[i])
+                               if (level_old == br->levels[i]) {
                                        level = level_old;
+                                       break;
+                               }
                }
-               goto set_level;
+       } else {
+               if (br->flags._BCL_reversed)
+                       level_old = (br->count - 1) - level_old;
+               level = br->levels[level_old];
        }
 
-       if (br->flags._BCL_reversed)
-               level_old = (br->count - 1) - level_old;
-       level = br->levels[level_old];
-
 set_level:
        result = acpi_video_device_lcd_set_level(device, level);
        if (result)
-- 
1.8.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to