Date: Fri, 30 Jan 2015 21:20:05 -0800 From: Adrian Chadd <adrian@freebsd.org> To: Elizabeth Myers <elizabeth@interlinked.me>, Konstantin Belousov <kib@freebsd.org> Cc: freebsd-current <freebsd-current@freebsd.org>, Justin Hibbits <jrh29@alumni.cwru.edu> Subject: Re: Questions on adding backlight support for the i915 driver Message-ID: <CAJ-VmonuAC9YFXR8jai5%2BKdW5q2x%2BjVVEABP0zRXiwHyWAC8pA@mail.gmail.com> In-Reply-To: <54CC57CE.2080001@interlinked.me> References: <54C883E7.4000300@interlinked.me> <CAJ-Vmokw2FqLHMkc%2B0MDsDxChMC68KdkvO43qhFztTZf8F89Mg@mail.gmail.com> <CAHSQbTCrU-8kYpNoL_UiNd1PHtSG8sPnt8jRfGvEOZEWa_G4FA@mail.gmail.com> <54CC57CE.2080001@interlinked.me>
next in thread | previous in thread | raw e-mail | index | archive | help
On 30 January 2015 at 20:19, Elizabeth Myers <elizabeth@interlinked.me> wrote: > On 01/30/15 17:25, Justin Hibbits wrote: >> Would it make sense to have a generic 'backlight' driver framework >> that we plug into? I wrote a backlight driver (well, 2, but both show >> up as dev.backlight in sysctl) for powerpc, but if we want to have >> even more individual backlight drivers, I think it makes sense to make >> them all look the same, with similar configuration properties. >> >> - Justin > > I was thinking the same thing myself. There's a lot of different > backlights out there, and acpi_video(4) obviously can't cover them all, > which seems to define the de facto "standard" MIB for brightness sysctls > (which upower presently uses). > > I hate to bikeshed, but I would humbly suggest adding an MIB hierarchy > such as hw.backlight.<hw_type>.<backlight_no>.brightness (and maybe > brightness_raw for advanced users who want/need more control). Woo! A patch! I'm happy with this, but you'll want to cc: kib at least who looks after the i915 code. Hi kib! I think this is fine for the short term and lets us experiment without committing to a fully public API for this for the short term. Do you have brightness buttons anywhere? What happens when you set it through this API and then you use the backlight buttons? -adrian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmonuAC9YFXR8jai5%2BKdW5q2x%2BjVVEABP0zRXiwHyWAC8pA>