Re: [PATCH] nouveau: fix ambiguous backlight controls

2014-12-28 Thread Hans de Goede
Hi, On 28-12-14 15:30, Jeremiah Mahler wrote: Hans, On Sat, Dec 27, 2014 at 02:39:42PM +0100, Hans de Goede wrote: Hi, On 27-12-14 00:51, Jeremiah Mahler wrote: Ilia, On Fri, Dec 26, 2014 at 04:39:08PM -0500, Ilia Mirkin wrote: On Fri, Dec 26, 2014 at 4:26 PM, Jeremiah Mahler wrote: If a

Re: [PATCH] nouveau: fix ambiguous backlight controls

2014-12-28 Thread Jeremiah Mahler
Hans, On Sat, Dec 27, 2014 at 02:39:42PM +0100, Hans de Goede wrote: > Hi, > > On 27-12-14 00:51, Jeremiah Mahler wrote: > >Ilia, > > > >On Fri, Dec 26, 2014 at 04:39:08PM -0500, Ilia Mirkin wrote: > >>On Fri, Dec 26, 2014 at 4:26 PM, Jeremiah Mahler wrote: > >>>If a display supports backlight c

Re: [PATCH] nouveau: fix ambiguous backlight controls

2014-12-27 Thread Hans de Goede
Hi, On 27-12-14 00:51, Jeremiah Mahler wrote: Ilia, On Fri, Dec 26, 2014 at 04:39:08PM -0500, Ilia Mirkin wrote: On Fri, Dec 26, 2014 at 4:26 PM, Jeremiah Mahler wrote: If a display supports backlight control using the nouveau driver, and also supports standard ACPI backlight control, there

Re: [PATCH] nouveau: fix ambiguous backlight controls

2014-12-26 Thread Jeremiah Mahler
Ilia, On Fri, Dec 26, 2014 at 04:39:08PM -0500, Ilia Mirkin wrote: > On Fri, Dec 26, 2014 at 4:26 PM, Jeremiah Mahler wrote: > > If a display supports backlight control using the nouveau driver, and > > also supports standard ACPI backlight control, there will be two sets of > > controls. > > > >

Re: [PATCH] nouveau: fix ambiguous backlight controls

2014-12-26 Thread Ilia Mirkin
On Fri, Dec 26, 2014 at 4:26 PM, Jeremiah Mahler wrote: > If a display supports backlight control using the nouveau driver, and > also supports standard ACPI backlight control, there will be two sets of > controls. > > /sys/class/backlight/acpi_video0 > /sys/class/backlight/nv_backlight > > This c

[PATCH] nouveau: fix ambiguous backlight controls

2014-12-26 Thread Jeremiah Mahler
If a display supports backlight control using the nouveau driver, and also supports standard ACPI backlight control, there will be two sets of controls. /sys/class/backlight/acpi_video0 /sys/class/backlight/nv_backlight This creates ambiguity because these controls can be out of sync with each ot