linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Aaron Lu <aaron.lu@intel.com>
To: Hans de Goede <hdegoede@redhat.com>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Zhang Rui <rui.zhang@intel.com>, linux-acpi@vger.kernel.org
Subject: Re: [RFC PATCH 0/2] acpi: video: Allow forcing native backlight on non win8 laptops
Date: Tue, 03 Mar 2015 16:25:48 +0800	[thread overview]
Message-ID: <54F5700C.8050200@intel.com> (raw)
In-Reply-To: <1425368350-4028-1-git-send-email-hdegoede@redhat.com>

On 03/03/2015 03:39 PM, Hans de Goede wrote:
> Hi All,
> 
> I've long wanted to have a way for users to force native-backlight behavior as
> that makes debugging backlight problems a bit easier.

I agree.

> 
> And now a laptop has shown up which needs native backlight behavior for things
> to work, has a non win8 ready bios, and I cannot use a quirk in the vendor
> backlight driver as it implements both the acer-wmi and ideapad-laptop vendor
> interfaces and putting the same quirk in 2 places is just wrong.
> 
> So I've finally written a small patch to allow forcing native-backlight
> behavior as well as a second patch using this to add a quirk for said laptop.
> 
> For now this is just a RFC series as I'm waiting for confirmation from the
> user that this actually fixes things, but I wanted to get the discussion going
> beforehand in case anyone has any objections against the solution I've come
> up with.

See my comments for patch 1.

Thanks,
Aaron

> 
> Regards,
> 
> Hans
> 


      parent reply	other threads:[~2015-03-03  8:27 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-03  7:39 [RFC PATCH 0/2] acpi: video: Allow forcing native backlight on non win8 laptops Hans de Goede
2015-03-03  7:39 ` [RFC PATCH 1/2] acpi: video: Allow forcing native backlight on non win8 machines Hans de Goede
2015-03-03  8:26   ` Aaron Lu
2015-03-03 13:11     ` Hans de Goede
2015-03-04  2:50       ` Aaron Lu
2015-03-10 22:25         ` Rafael J. Wysocki
2015-03-10 22:42           ` Hans de Goede
2015-03-10 23:10             ` Rafael J. Wysocki
2015-03-11  6:14               ` [PATCH update] acpi: video: Allow forcing native backlight on non win8, machines Aaron Lu
2015-03-11 11:14                 ` Hans de Goede
2015-03-11 13:05                   ` Aaron Lu
2015-03-12 22:40                     ` Rafael J. Wysocki
2015-03-03  7:39 ` [RFC PATCH 2/2] acpi: video: Add force native backlight quirk for Lenovo Ideapad Z570 Hans de Goede
2015-03-10 22:26   ` Rafael J. Wysocki
2015-03-10 22:44     ` Hans de Goede
2015-03-03  8:25 ` Aaron Lu [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54F5700C.8050200@intel.com \
    --to=aaron.lu@intel.com \
    --cc=hdegoede@redhat.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=rjw@rjwysocki.net \
    --cc=rui.zhang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).