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 Matthew Garrett
On Wed, Mar 27, 2013 at 12:56:37PM +0100, Danny Baumann wrote: > OK, I see. And there is user space depending on that behaviour? And > again - how is user space supposed to know about the behavioral > differences? Is it something like 'if type is raw, don't expect > anything'? "Do not rely upon 0

[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

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

2013-03-27 Thread Alex Deucher
On Wed, Mar 27, 2013 at 8:56 AM, Danny Baumann wrote: > 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 usefu

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

2013-03-27 Thread Alex Deucher
On Wed, Mar 27, 2013 at 7:56 AM, Danny Baumann wrote: > 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 cas

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

2013-03-27 Thread Matthew Garrett
On Wed, Mar 27, 2013 at 12:56:37PM +0100, Danny Baumann wrote: > OK, I see. And there is user space depending on that behaviour? And > again - how is user space supposed to know about the behavioral > differences? Is it something like 'if type is raw, don't expect > anything'? "Do not rely upon 0

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

2013-03-27 Thread Alex Deucher
On Wed, Mar 27, 2013 at 8:56 AM, Danny Baumann wrote: > 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 usefu

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

2013-03-27 Thread Alex Deucher
On Wed, Mar 27, 2013 at 7:56 AM, Danny Baumann wrote: > 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 cas

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 >> makes the panel content ba

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

2013-03-26 Thread Matthew Garrett
On Tue, Mar 26, 2013 at 06:10:30PM +0100, Danny Baumann wrote: > Am 26.03.2013 18:02, schrieb Matthew Garrett: > >I'm not quite clear what you mean here. The behaviour of "0" isn't well > >defined for the ACPI backlight driver - it's perfectly reasonable for it > >to turn the backlight off entirely

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

2013-03-26 Thread 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 readable instead of turning the backlight off. > Th

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

2013-03-26 Thread Danny Baumann
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 readable instead of turning the backlight off. This matches the expectations of user space (e.g. kde-workspace or the In

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 rea

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

2013-03-26 Thread 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 readable instead of turning the backlight off. > Th

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

2013-03-26 Thread Matthew Garrett
On Tue, Mar 26, 2013 at 06:10:30PM +0100, Danny Baumann wrote: > Am 26.03.2013 18:02, schrieb Matthew Garrett: > >I'm not quite clear what you mean here. The behaviour of "0" isn't well > >defined for the ACPI backlight driver - it's perfectly reasonable for it > >to turn the backlight off entirely

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

2013-03-26 Thread Danny Baumann
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 readable instead of turning the backlight off. This matches the expectations of user space (e.g. kde-workspace or the In