lkml.org 
[lkml]   [2016]   [Mar]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.4 243/342] ACPI: Revert "ACPI / video: Add Dell Inspiron 5737 to the blacklist"
    Date
    4.4-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Hans de Goede <hdegoede@redhat.com>

    commit b186b4dcb79b1914c3dadb27ac72dafaa4267998 upstream.

    The quirk to get "acpi_backlight=vendor" behavior by default on the
    Dell Inspiron 5737 was added before we started doing
    "acpi_backlight=native" by default on Win8 ready machines.

    Since we now avoid using acpi-video as backlight driver on these machines
    by default (using the native driver instead) we no longer need this quirk.

    Moreover the vendor driver does not work after a suspend/resume where
    as the native driver does.

    This reverts commit 08a56226d847 (ACPI / video: Add Dell Inspiron 5737
    to the blacklist).

    Link: https://bugzilla.kernel.org/show_bug.cgi?id=111061
    Reported-and-tested-by: erusan@gmail.com
    Signed-off-by: Hans de Goede <hdegoede@redhat.com>
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/acpi/video_detect.c | 8 --------
    1 file changed, 8 deletions(-)

    --- a/drivers/acpi/video_detect.c
    +++ b/drivers/acpi/video_detect.c
    @@ -135,14 +135,6 @@ static const struct dmi_system_id video_
    DMI_MATCH(DMI_PRODUCT_NAME, "UL30A"),
    },
    },
    - {
    - .callback = video_detect_force_vendor,
    - .ident = "Dell Inspiron 5737",
    - .matches = {
    - DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
    - DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5737"),
    - },
    - },

    /*
    * These models have a working acpi_video backlight control, and using
    \
     
     \ /
      Last update: 2016-03-02 02:21    [W:4.069 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site