Merge branch 'acpi-video'

* acpi-video:
  ACPI / video: clean up DMI table for initial black screen problem

1 files changed, 12 insertions, 75 deletions

diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index 18dbdff4656e..995e91bcb97b 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
@@ -82,13 +82,6 @@ static bool allow_duplicates;
 module_param(allow_duplicates, bool, 0644);
 
 /*
- * Some BIOSes claim they use minimum backlight at boot,
- * and this may bring dimming screen after boot
- */
-static bool use_bios_initial_backlight = 1;
-module_param(use_bios_initial_backlight, bool, 0644);
-
-/*
  * For Windows 8 systems: if set ture and the GPU driver has
  * registered a backlight interface, skip registering ACPI video's.
  */
@@ -406,12 +399,6 @@ static int __init video_set_bqc_offset(const struct dmi_system_id *d)
         return 0;
 }
 
-static int video_ignore_initial_backlight(const struct dmi_system_id *d)
-{
-        use_bios_initial_backlight = 0;
-        return 0;
-}
-
 static struct dmi_system_id video_dmi_table[] __initdata = {
         /*
          * Broken _BQC workaround http://bugzilla.kernel.org/show_bug.cgi?id=13121
@@ -456,54 +443,6 @@ static struct dmi_system_id video_dmi_table[] __initdata = {
                 DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 7720"),
                 },
         },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "HP Folio 13-2000",
-         .matches = {
-                DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-                DMI_MATCH(DMI_PRODUCT_NAME, "HP Folio 13 - 2000 Notebook PC"),
-                },
-        },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "Fujitsu E753",
-         .matches = {
-                DMI_MATCH(DMI_BOARD_VENDOR, "FUJITSU"),
-                DMI_MATCH(DMI_PRODUCT_NAME, "LIFEBOOK E753"),
-                },
-        },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "HP Pavilion dm4",
-         .matches = {
-                DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-                DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dm4 Notebook PC"),
-                },
-        },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "HP Pavilion g6 Notebook PC",
-         .matches = {
-                 DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-                 DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion g6 Notebook PC"),
-                },
-        },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "HP 1000 Notebook PC",
-         .matches = {
-                DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-                DMI_MATCH(DMI_PRODUCT_NAME, "HP 1000 Notebook PC"),
-                },
-        },
-        {
-         .callback = video_ignore_initial_backlight,
-         .ident = "HP Pavilion m4",
-         .matches = {
-                DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-                DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion m4 Notebook PC"),
-                },
-        },
         {}
 };
 
@@ -839,20 +778,18 @@ acpi_video_init_brightness(struct acpi_video_device *device)
         if (!device->cap._BQC)
                 goto set_level;
 
-        if (use_bios_initial_backlight) {
-                level = acpi_video_bqc_value_to_level(device, level_old);
-                /*
-                 * On some buggy laptops, _BQC returns an uninitialized
-                 * value when invoked for the first time, i.e.
-                 * level_old is invalid (no matter whether it's a level
-                 * or an index). Set the backlight to max_level in this case.
-                 */
-                for (i = 2; i < br->count; i++)
-                        if (level == br->levels[i])
-                                break;
-                if (i == br->count || !level)
-                        level = max_level;
-        }
+        level = acpi_video_bqc_value_to_level(device, level_old);
+        /*
+         * On some buggy laptops, _BQC returns an uninitialized
+         * value when invoked for the first time, i.e.
+         * level_old is invalid (no matter whether it's a level
+         * or an index). Set the backlight to max_level in this case.
+         */
+        for (i = 2; i < br->count; i++)
+                if (level == br->levels[i])
+                        break;
+        if (i == br->count || !level)
+                level = max_level;
 
 set_level:
         result = acpi_video_device_lcd_set_level(device, level);