ACPI / video: Run _BCL before deciding registering backlight
authorAaron Lu <aaron.lu@intel.com>
Tue, 28 Oct 2014 06:35:59 +0000 (14:35 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 30 Oct 2014 00:30:24 +0000 (01:30 +0100)
commitdce4ec2e452fddb7542b5fc15d0e6b8531f6d5eb
tree12590b7ace0d4f59b8cb6f8e6a820204b2be4aa7
parentcac7f2429872d3733dc3f9915857b1691da2eb2f
ACPI / video: Run _BCL before deciding registering backlight

The ASUS K53SM's ACPI table queries _OSI("Windows 2012") in the video
output device's _BCL and _BCM control method instead of the usual _INI
functions of the _SB or PCI host bridge PCI0 devices. This made our
video module thought this is a pre-Win8 system when deciding if we
should register a backlight interface for it and the end result is that
a non-working acpi_video interface is registered and user is unable to
control backlight from GUI. Solve this problem by evaluating _BCL control
method before doing the decision.

Note that for some Thinkpad systems, the _BCL is also required to be
evaluated for the hotkey event to be generated no matter if we will
register an ACPI video backlight interface for it or not. Since the
thinkpad_acpi module will do this anyway we didn't add such a thing in
the video module previously. But now with this change here, the
thinkpad_acpi module is no more necessary for those systems regarding
backlight functionality.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=85051
Reported-and-tested-by: Ralf Jung <post+kernel@ralfj.de>
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/video.c