I admit I'm not really an ACPI expert, but thinking about this more, I'm wondering if maybe we should just send the appropriate brightness change, switch display, etc. event to userspace rather than handling it directly in the radeon driver, then let userspace callback down via the bl interface, etc. With backlight for example, does handling it in the kernel driver as per your patch prevent userspace from seeing the brightness up/down event? Wouldn't that break things like OSD brightness displays and such? Alex On Wed, Aug 1, 2012 at 9:56 AM, Alex Deucher <alexdeucher@xxxxxxxxx> wrote: > On Wed, Aug 1, 2012 at 4:57 AM, Luca Tettamanti <kronos.it@xxxxxxxxx> wrote: >> On Tue, Jul 31, 2012 at 05:33:16PM -0400, Alex Deucher wrote: >>> Patches look good. I picked them up and combined them with may >>> patches plus a few other small fixes. They are available here: >>> http://cgit.freedesktop.org/~agd5f/linux/log/?h=acpi_patches >>> Let me know what you think. >> >> Looks ok, I lost one fix along the road though, I'm attaching the patch. > > Thanks, I rolled it into your original patch. New tree: > http://cgit.freedesktop.org/~agd5f/linux/log/?h=acpi_patches > > Alex > >> >> Luca _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
