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@gmail.com wrote:
On Wed, Aug 1, 2012 at 4:57 AM, Luca Tettamanti kronos.it@gmail.com 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