[PATCH] drm/radeon: add new AMD ACPI header and update relevant code

[Alex Deucher]

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