Re: [PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-28 Thread Danny Baumann
Hi, Well, the ACPI spec says this (section B.5.2): " The OEM may define the number 0 as "Zero brightness" that can mean to turn off the lighting (e.g. LCD panel backlight) in the device. This may be useful in the case of an output device that can still be viewed using only ambient light, for ex

[PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-27 Thread Danny Baumann
Hi, Well, the ACPI spec says this (section B.5.2): " The OEM may define the number 0 as "Zero brightness" that can mean to turn off the lighting (e.g. LCD panel backlight) in the device. This may be useful in the case of an output device that can still be viewed

[PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-27 Thread Danny Baumann
Hi, >> Well, the ACPI spec says this (section B.5.2): >> >> " >> The OEM may define the number 0 as "Zero brightness" that can mean >> to turn off the lighting (e.g. LCD panel backlight) in the device. >> This may be useful in the case of an output device that can still be >> viewed using only amb

Re: [PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-27 Thread Danny Baumann
Hi, Well, the ACPI spec says this (section B.5.2): " The OEM may define the number 0 as "Zero brightness" that can mean to turn off the lighting (e.g. LCD panel backlight) in the device. This may be useful in the case of an output device that can still be viewed using only ambient light, for ex

[PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, Am 26.03.2013 18:02, schrieb Matthew Garrett: > On Tue, Mar 26, 2013 at 12:48:44PM +0100, Danny Baumann wrote: >> This patch makes the behaviour of the intel_backlight backlight device >> consistent to e.g. acpi_videoX: When writing the value 0, the set brightness >> ma

[PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, >> Thus far our assumption always was that the acpi backlight works better >> than the intel native backlight. So everything only uses the intel >> backlight if there's no other backlight driver by default. > > There are machines, such as the pnv netbook on my desk, on which > intel_backlight

[PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, > Thus far our assumption always was that the acpi backlight works better > than the intel native backlight. So everything only uses the intel > backlight if there's no other backlight driver by default. > > So if I should merge this as a general solution for Windows 8 machines not > working p

[PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
of the maximum level. The default is 5, which provides a backlight level that is barely readable. Setting it to 0 restores the old behaviour. Signed-off-by: Danny Baumann --- drivers/gpu/drm/i915/intel_panel.c | 48 -- 1 file changed, 41 insertions(+), 7 deleti

[PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
l.org/show_bug.cgi?id=55071 Danny Baumann (1): drm/i915: Allow specifying a minimum brightness level for sysfs control. drivers/gpu/drm/i915/intel_panel.c | 48 -- 1 file changed, 41 insertions(+), 7 deletions(-) -- 1.8.1.4

Re: [PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, Thus far our assumption always was that the acpi backlight works better than the intel native backlight. So everything only uses the intel backlight if there's no other backlight driver by default. So if I should merge this as a general solution for Windows 8 machines not working properly,

Re: [PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, Am 26.03.2013 18:02, schrieb Matthew Garrett: On Tue, Mar 26, 2013 at 12:48:44PM +0100, Danny Baumann wrote: This patch makes the behaviour of the intel_backlight backlight device consistent to e.g. acpi_videoX: When writing the value 0, the set brightness makes the panel content barely

Re: [PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
Hi, Thus far our assumption always was that the acpi backlight works better than the intel native backlight. So everything only uses the intel backlight if there's no other backlight driver by default. There are machines, such as the pnv netbook on my desk, on which intel_backlight does nothin

[PATCH 1/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
of the maximum level. The default is 5, which provides a backlight level that is barely readable. Setting it to 0 restores the old behaviour. Signed-off-by: Danny Baumann --- drivers/gpu/drm/i915/intel_panel.c | 48 -- 1 file changed, 41 insertions(+), 7 deleti

[PATCH 0/1] drm/i915: Allow specifying a minimum brightness level for sysfs control.

2013-03-26 Thread Danny Baumann
l.org/show_bug.cgi?id=55071 Danny Baumann (1): drm/i915: Allow specifying a minimum brightness level for sysfs control. drivers/gpu/drm/i915/intel_panel.c | 48 -- 1 file changed, 41 insertions(+), 7 deletions(-) -- 1.