Re: [PATCH v2 3/3] acer-wmi: Add Aspire 5741 to video_vendor_dmi_table

2014-05-19 Thread joeyli
On Sat, May 17, 2014 at 10:48:03AM +0200, Hans de Goede wrote: > The Aspire 5741 has broken acpi-video backlight control, so add it to the > quirk table. > > https://bugzilla.redhat.com/show_bug.cgi?id=1012674 > > Signed-off-by: Hans de Goede Reviewed-by: Lee, Chun-Yi Thanks for your patch!

Re: [PATCH v2 2/3] acer-wmi: Switch to acpi_video_unregister_backlight

2014-05-19 Thread joeyli
On Sat, May 17, 2014 at 10:48:02AM +0200, Hans de Goede wrote: > Switch from acpi_video_unregister(), to acpi_video_unregister_backlight(), > so that the hotkeys handler registered by acpi-video stays in place. > > Since there are no mappings for the atkbd raw codes for the brightness > keys used

Re: [PATCH 1/4] acpi-video: Don't register acpi_video_resume notifier without backlight devices

2014-05-19 Thread Rafael J. Wysocki
On Friday, May 16, 2014 10:25:52 AM Aaron Lu wrote: > On 05/15/2014 07:22 PM, Hans de Goede wrote: > > If we're not going to be registering any backlight devices then > > acpi_video_resume is always nop, so don't register it in that case. > > > > Signed-off-by: Hans de Goede > > Reviewed-by: Aar

Re: RFC: Cleanup firmware backlight control method selection

2014-05-19 Thread Hans de Goede
Hi, On 05/19/2014 01:24 AM, Mattia Dongili wrote: > On Fri, May 16, 2014 at 10:06:15AM +0200, Hans de Goede wrote: >> This is only a problem with crazy PC's which have multiple firmware >> API's (both standardized and custom ACPI) to control the backlight, which >> also are sometimes all broken