[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-30 Thread Rafael J. Wysocki
On Wednesday, October 30, 2013 12:40:13 AM Matthew Garrett wrote: > On Wed, 2013-10-16 at 01:33 +0200, Rafael J. Wysocki wrote: > > > Since the next step will be to introduce a list of systems that need > > video.use_native_backlight=1 *and* don't break in that configuration, I > > don't > > see

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-30 Thread Matthew Garrett
On Wed, 2013-10-16 at 01:33 +0200, Rafael J. Wysocki wrote: > Since the next step will be to introduce a list of systems that need > video.use_native_backlight=1 *and* don't break in that configuration, I don't > see much point adding another Kconfig option for the default. I'd still really prefe

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-29 Thread Aaron Lu
On 10/28/2013 04:09 PM, Jani Nikula wrote: > On Mon, 28 Oct 2013, Aaron Lu wrote: >> +static int __init video_set_use_native_backlight(const struct dmi_system_id >> *d) >> +{ >> +use_native_backlight = true; >> +return 0; >> +} > > Hi Aaron, it might be beneficial to make use_native_back

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-28 Thread Aaron Lu
On 10/25/2013 02:35 PM, Igor Gnatenko wrote: > Aaron, add this notebook to list. I've CC'ed owner. > And I've tested this patch on my TP X230 (add as Reported-and-Tested me > please) > + { > + .callback = video_set_use_native_backlight, > + .ident = "Dell Inspiron 7520", > + .mat

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-28 Thread Jani Nikula
On Mon, 28 Oct 2013, Aaron Lu wrote: > On 10/25/2013 02:35 PM, Igor Gnatenko wrote: >> Aaron, add this notebook to list. I've CC'ed owner. >> And I've tested this patch on my TP X230 (add as Reported-and-Tested me >> please) >> +{ >> + .callback = video_set_use_native_backlight, >> + .

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-25 Thread Igor Gnatenko
On Thu, 2013-10-24 at 16:13 +0800, Aaron Lu wrote: > On 10/16/2013 07:33 AM, Rafael J. Wysocki wrote: > > On Friday, October 11, 2013 09:27:42 PM Aaron Lu wrote: > >> v5: > >> 1 Introduce video.use_native_backlight module parameter and set its > >> value to false by default as suggested by Rafael

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-24 Thread Aaron Lu
On 10/16/2013 07:33 AM, Rafael J. Wysocki wrote: > On Friday, October 11, 2013 09:27:42 PM Aaron Lu wrote: >> v5: >> 1 Introduce video.use_native_backlight module parameter and set its >> value to false by default as suggested by Rafael. For Win8 systems >> which have broken ACPI video backligh

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-16 Thread Rafael J. Wysocki
On Friday, October 11, 2013 09:27:42 PM Aaron Lu wrote: > v5: > 1 Introduce video.use_native_backlight module parameter and set its > value to false by default as suggested by Rafael. For Win8 systems > which have broken ACPI video backlight control, the parameter can be > set to 1 in kernel

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Saturday, October 12, 2013 07:54:06 AM Yves-Alexis Perez wrote: > On sam., 2013-10-12 at 01:27 +0200, Rafael J. Wysocki wrote: > > If we are to use a Kconfig option, why don't we use one instead of rather > > than > > in addition to a command line option? Say, we have > > CONFIG_ACPI_VIDEO_WIN

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Saturday, October 12, 2013 08:34:32 AM Seth Forshee wrote: > On Sat, Oct 12, 2013 at 04:44:30AM -0700, Josh Boyer wrote: > > On Fri, Oct 11, 2013 at 4:27 PM, Rafael J. Wysocki > > wrote: > > > On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > > >> On Fri, Oct 11, 2013 at 6:10 PM, Raf

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Seth Forshee
On Sat, Oct 12, 2013 at 04:44:30AM -0700, Josh Boyer wrote: > On Fri, Oct 11, 2013 at 4:27 PM, Rafael J. Wysocki wrote: > > On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > >> On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki > >> wrote: > >> > On Friday, October 11, 2013 12:42:43 PM

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Josh Boyer
On Fri, Oct 11, 2013 at 4:27 PM, Rafael J. Wysocki wrote: > On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: >> On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki >> wrote: >> > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: >> >> On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wr

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Josh Boyer
On Fri, Oct 11, 2013 at 11:00 PM, Yves-Alexis Perez wrote: > On sam., 2013-10-12 at 00:10 +0200, Rafael J. Wysocki wrote: >> On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: >> > On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: >> > > v5: >> > > 1 Introduce video.use_native_backlight mod

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Yves-Alexis Perez
On sam., 2013-10-12 at 00:10 +0200, Rafael J. Wysocki wrote: > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: > > On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > > > v5: > > > 1 Introduce video.use_native_backlight module parameter and set its > > > value to false by default as sug

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Yves-Alexis Perez
On sam., 2013-10-12 at 01:27 +0200, Rafael J. Wysocki wrote: > If we are to use a Kconfig option, why don't we use one instead of rather than > in addition to a command line option? Say, we have > CONFIG_ACPI_VIDEO_WIN8_WORKAROUND and if that is set, the code will work like > the previous version

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki wrote: > > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: > >> On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > >> > v5: > >> > 1 Introduce video.use_native_backlight module

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Igor Gnatenko
On Sat, 2013-10-12 at 00:45 +0200, Rafael J. Wysocki wrote: > On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > > On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki > > wrote: > > > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: > > >> On Fri, Oct 11, 2013 at 9:27 AM, Aaron L

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Saturday, October 12, 2013 02:53:38 AM Igor Gnatenko wrote: > On Sat, 2013-10-12 at 00:45 +0200, Rafael J. Wysocki wrote: > > On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > > > On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki > > > wrote: > > > > On Friday, October 11, 2013 12:4

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Friday, October 11, 2013 06:01:43 PM Josh Boyer wrote: > On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki wrote: > > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: > >> On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > >> > v5: > >> > 1 Introduce video.use_native_backlight module

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Josh Boyer
On Fri, Oct 11, 2013 at 6:10 PM, Rafael J. Wysocki wrote: > On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: >> On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: >> > v5: >> > 1 Introduce video.use_native_backlight module parameter and set its >> > value to false by default as suggested

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Rafael J. Wysocki
On Friday, October 11, 2013 12:42:43 PM Josh Boyer wrote: > On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > > v5: > > 1 Introduce video.use_native_backlight module parameter and set its > > value to false by default as suggested by Rafael. For Win8 systems > > which have broken ACPI video b

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Igor Gnatenko
On Fri, 2013-10-11 at 12:42 -0400, Josh Boyer wrote: > On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > > v5: > > 1 Introduce video.use_native_backlight module parameter and set its > > value to false by default as suggested by Rafael. For Win8 systems > > which have broken ACPI video backli

Re: [PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Josh Boyer
On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu wrote: > v5: > 1 Introduce video.use_native_backlight module parameter and set its > value to false by default as suggested by Rafael. For Win8 systems > which have broken ACPI video backlight control, the parameter can be > set to 1 in kernel cmdlin

[PATCH v5 0/4] Fix Win8 backlight issue

2013-10-12 Thread Aaron Lu
v5: 1 Introduce video.use_native_backlight module parameter and set its value to false by default as suggested by Rafael. For Win8 systems which have broken ACPI video backlight control, the parameter can be set to 1 in kernel cmdline to skip registering ACPI video's backlight interface. Du