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

2012-08-02 Thread Luca Tettamanti
On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher wrote: > 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 driv

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

2012-08-02 Thread Alex Deucher
On Thu, Aug 2, 2012 at 12:33 PM, Alex Deucher wrote: > On Thu, Aug 2, 2012 at 12:31 PM, Luca Tettamanti > wrote: >> On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher >> wrote: >>> I admit I'm not really an ACPI expert, but thinking about this more, >>> I'm wondering if maybe we should just send the

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

2012-08-02 Thread Alex Deucher
On Thu, Aug 2, 2012 at 12:33 PM, Alex Deucher wrote: > On Thu, Aug 2, 2012 at 12:31 PM, Luca Tettamanti wrote: >> On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher wrote: >>> I admit I'm not really an ACPI expert, but thinking about this more, >>> I'm wondering if maybe we should just send the approp

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

2012-08-02 Thread Alex Deucher
On Thu, Aug 2, 2012 at 12:31 PM, Luca Tettamanti wrote: > On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher wrote: >> 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 u

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

2012-08-02 Thread 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, et

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

2012-08-02 Thread Alex Deucher
On Thu, Aug 2, 2012 at 12:31 PM, Luca Tettamanti wrote: > On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher wrote: >> 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 u

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

2012-08-02 Thread Luca Tettamanti
On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher wrote: > 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 driv

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

2012-08-02 Thread 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, et

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

2012-08-01 Thread Luca Tettamanti
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

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

2012-08-01 Thread Alex Deucher
On Wed, Aug 1, 2012 at 4:57 AM, Luca Tettamanti 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/l

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

2012-08-01 Thread Alex Deucher
On Wed, Aug 1, 2012 at 4:57 AM, Luca Tettamanti 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/l

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

2012-08-01 Thread Luca Tettamanti
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

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

2012-07-31 Thread Luca Tettamanti
On Tue, Jul 31, 2012 at 09:58:04AM -0400, Alex Deucher wrote: > On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti > wrote: > > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher > > wrote: > >> Regarding patches 3 and 4, it might be easier to just store a pointer > >> to the relevant encoder when yo

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

2012-07-31 Thread Alex Deucher
On Tue, Jul 31, 2012 at 4:05 PM, Luca Tettamanti wrote: > On Tue, Jul 31, 2012 at 09:58:04AM -0400, Alex Deucher wrote: >> On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti >> wrote: >> > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher >> > wrote: >> >> Regarding patches 3 and 4, it might be eas

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

2012-07-31 Thread Alex Deucher
On Tue, Jul 31, 2012 at 4:05 PM, Luca Tettamanti wrote: > On Tue, Jul 31, 2012 at 09:58:04AM -0400, Alex Deucher wrote: >> On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti wrote: >> > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher >> > wrote: >> >> Regarding patches 3 and 4, it might be easier

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

2012-07-31 Thread Luca Tettamanti
On Tue, Jul 31, 2012 at 09:58:04AM -0400, Alex Deucher wrote: > On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti wrote: > > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher > > wrote: > >> Regarding patches 3 and 4, it might be easier to just store a pointer > >> to the relevant encoder when you v

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

2012-07-31 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher wrote: > Regarding patches 3 and 4, it might be easier to just store a pointer > to the relevant encoder when you verify ATIF rather than walking the > encoder list every time. Makes sense, I was unsure about the lifetime of the encoders, but AFAICS

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

2012-07-31 Thread Alex Deucher
On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher > wrote: >> Regarding patches 3 and 4, it might be easier to just store a pointer >> to the relevant encoder when you verify ATIF rather than walking the >> encoder list every time. > > Makes

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

2012-07-31 Thread Alex Deucher
On Tue, Jul 31, 2012 at 5:16 AM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher wrote: >> Regarding patches 3 and 4, it might be easier to just store a pointer >> to the relevant encoder when you verify ATIF rather than walking the >> encoder list every time. > > Makes se

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

2012-07-31 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher wrote: > Regarding patches 3 and 4, it might be easier to just store a pointer > to the relevant encoder when you verify ATIF rather than walking the > encoder list every time. Makes sense, I was unsure about the lifetime of the encoders, but AFAICS

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:30 PM, Alex Deucher wrote: > On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti > wrote: >> On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >>> Supported DWORD Bit vector providing supported functions >>> information. Each bit marks >>> Functions Bit

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

2012-07-30 Thread joeyli
於 日,2012-07-29 於 15:10 +0200,Luca Tettamanti 提到: > On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > > Hi Luca, > > > > 於 六,2012-07-28 於 16:56 +0200,Luca Tettamanti 提到: > > > I just found the first problem (probably a BIOS bug): > > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: > On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti > wrote: > > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: > >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti > >> wrote: > >> > I just found the first problem (p

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

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >> On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti >> wrote: >> > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Te

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

2012-07-30 Thread joeyli
? ??2012-07-29 ? 15:10 +0200?Luca Tettamanti ??? > On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > > Hi Luca, > > > > ? ??2012-07-28 ? 16:56 +0200?Luca Tettamanti ??? > > > I just found the first problem (probably a BIOS bug): > > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in

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

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >> On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti >> wrote: >> > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Te

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 04:32:47PM +0800, joeyli wrote: > ? ??2012-07-29 ? 15:10 +0200?Luca Tettamanti ??? > > On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > > > Hi Luca, > > > > > > ? ??2012-07-28 ? 16:56 +0200?Luca Tettamanti ??? > > > > I just found the first problem (probably a BIO

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

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >> On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti wrote: >> > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettam

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:30 PM, Alex Deucher wrote: > On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti wrote: >> On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >>> Supported DWORD Bit vector providing supported functions >>> information. Each bit marks >>> Functions Bit

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

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti wrote: > On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: >> On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti wrote: >> > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettam

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: > On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti wrote: > > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: > >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti > >> wrote: > >> > I just found the first problem (prob

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

2012-07-30 Thread Alex Deucher
On Sun, Jul 29, 2012 at 3:33 PM, Luca Tettamanti wrote: > Hi, > I'm attaching a first draft of my work. The first 3 patches are > infrastructure work, the fourth wires the notification handler and > retrieves the requests from the system BIOS, but it does not actually > change brightness yet. > >

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

2012-07-30 Thread Alex Deucher
On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti wrote: > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti >> wrote: >> > I just found the first problem (probably a BIOS bug): >> > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemente

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

2012-07-30 Thread Alex Deucher
On Sun, Jul 29, 2012 at 3:33 PM, Luca Tettamanti wrote: > Hi, > I'm attaching a first draft of my work. The first 3 patches are > infrastructure work, the fourth wires the notification handler and > retrieves the requests from the system BIOS, but it does not actually > change brightness yet. > >

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

2012-07-30 Thread Alex Deucher
On Sun, Jul 29, 2012 at 9:06 AM, Luca Tettamanti wrote: > On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: >> On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti >> wrote: >> > I just found the first problem (probably a BIOS bug): >> > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemente

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

2012-07-30 Thread Luca Tettamanti
On Mon, Jul 30, 2012 at 04:32:47PM +0800, joeyli wrote: > 於 日,2012-07-29 於 15:10 +0200,Luca Tettamanti 提到: > > On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > > > Hi Luca, > > > > > > 於 六,2012-07-28 於 16:56 +0200,Luca Tettamanti 提到: > > > > I just found the first problem (probably a BIO

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

2012-07-29 Thread joeyli
Hi Luca, 於 六,2012-07-28 於 16:56 +0200,Luca Tettamanti 提到: > On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: > > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti > > wrote: > > > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > > >> On Thu, Jul 26, 2012 at 8:58 AM, L

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

2012-07-29 Thread joeyli
於 五,2012-07-27 於 23:32 +0800,joeyli 提到: > 於 五,2012-07-27 於 09:21 -0400,Alex Deucher 提到: > > On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > > > > > > + * flags > > > + * bits 1:0: > > > + * 0 - Notify(VGA, 0x81) is not used for notification > > > + * 1 - Notify(VGA, 0x81) is used for notificatio

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

2012-07-29 Thread joeyli
於 五,2012-07-27 於 09:21 -0400,Alex Deucher 提到: > On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > > > > + * flags > > + * bits 1:0: > > + * 0 - Notify(VGA, 0x81) is not used for notification > > + * 1 - Notify(VGA, 0x81) is used for notification > > > > Per the above flags, when we detect bit set

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

2012-07-29 Thread joeyli
於 四,2012-07-26 於 23:31 -0400,Alex Deucher 提到: > On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > > 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: > >> - again ACPI video module gets the nodification (in this case > >> ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE > >> -

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

2012-07-29 Thread joeyli
於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: > - again ACPI video module gets the nodification (in this case > ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE > - KDE seems this and muck with the screen configuration :( > - meanwhile the brightness notification is propaga

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

2012-07-29 Thread Luca Tettamanti
Hi, I'm attaching a first draft of my work. The first 3 patches are infrastructure work, the fourth wires the notification handler and retrieves the requests from the system BIOS, but it does not actually change brightness yet. The problem here is how to get the correct encoder: should I just scan

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

2012-07-29 Thread Luca Tettamanti
On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > Hi Luca, > > ? ??2012-07-28 ? 16:56 +0200?Luca Tettamanti ??? > > I just found the first problem (probably a BIOS bug): > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in the DSDT, but the > > corresponding bit ATIF_GET_SYSTEM_PARAM

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

2012-07-29 Thread Luca Tettamanti
On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: > On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti > wrote: > > I just found the first problem (probably a BIOS bug): > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in the DSDT, but the > > corresponding bit ATIF_GET_SYSTEM_PA

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

2012-07-29 Thread Luca Tettamanti
Hi, I'm attaching a first draft of my work. The first 3 patches are infrastructure work, the fourth wires the notification handler and retrieves the requests from the system BIOS, but it does not actually change brightness yet. The problem here is how to get the correct encoder: should I just scan

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

2012-07-29 Thread joeyli
Hi Luca, ? ??2012-07-28 ? 16:56 +0200?Luca Tettamanti ??? > On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: > > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti > > wrote: > > > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > > >> On Thu, Jul 26, 2012 at 8:58 AM, L

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

2012-07-29 Thread Luca Tettamanti
On Sun, Jul 29, 2012 at 11:51:48AM +0800, joeyli wrote: > Hi Luca, > > 於 六,2012-07-28 於 16:56 +0200,Luca Tettamanti 提到: > > I just found the first problem (probably a BIOS bug): > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in the DSDT, but the > > corresponding bit ATIF_GET_SYSTEM_PARAM

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

2012-07-29 Thread Luca Tettamanti
On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: > On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti wrote: > > I just found the first problem (probably a BIOS bug): > > ATIF_FUNCTION_GET_SYSTEM_PARAMETERS is implemented in the DSDT, but the > > corresponding bit ATIF_GET_SYSTEM_PARAM

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

2012-07-28 Thread Alex Deucher
On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti wrote: > On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: >> On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti >> wrote: >> > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >> >> On Thu, Jul 26, 2012 at 8:58 AM, Luca T

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

2012-07-28 Thread Luca Tettamanti
On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti > wrote: > > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > >> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti > >> wrote: > >> > The other missing bit is how to ac

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

2012-07-28 Thread Alex Deucher
On Sat, Jul 28, 2012 at 10:56 AM, Luca Tettamanti wrote: > On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: >> On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti wrote: >> > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >> >> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettam

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

2012-07-28 Thread Luca Tettamanti
On Thu, Jul 26, 2012 at 03:42:26PM -0400, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti wrote: > > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > >> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti > >> wrote: > >> > The other missing bit is how to actua

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

2012-07-27 Thread joeyli
? ??2012-07-27 ? 23:32 +0800?joeyli ??? > ? ??2012-07-27 ? 09:21 -0400?Alex Deucher ??? > > On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > > > > > > + * flags > > > + * bits 1:0: > > > + * 0 - Notify(VGA, 0x81) is not used for notification > > > + * 1 - Notify(VGA, 0x81) is used for notificatio

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

2012-07-27 Thread joeyli
? ??2012-07-27 ? 09:21 -0400?Alex Deucher ??? > On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > > > > + * flags > > + * bits 1:0: > > + * 0 - Notify(VGA, 0x81) is not used for notification > > + * 1 - Notify(VGA, 0x81) is used for notification > > > > Per the above flags, when we detect bit set

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

2012-07-27 Thread joeyli
? ??2012-07-26 ? 23:31 -0400?Alex Deucher ??? > On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > > ? ??2012-07-26 ? 14:58 +0200?Luca Tettamanti ??? > >> - again ACPI video module gets the nodification (in this case > >> ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE > >> -

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

2012-07-27 Thread Alex Deucher
On Fri, Jul 27, 2012 at 11:32 AM, joeyli wrote: > ? ??2012-07-27 ? 09:21 -0400?Alex Deucher ??? >> On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: >> > >> > + * flags >> > + * bits 1:0: >> > + * 0 - Notify(VGA, 0x81) is not used for notification >> > + * 1 - Notify(VGA, 0x81) is used for notifica

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

2012-07-27 Thread Luca Tettamanti
On Fri, Jul 27, 2012 at 12:46:50PM +0800, joeyli wrote: > ? ??2012-07-26 ? 23:31 -0400?Alex Deucher ??? > > On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > > > ? ??2012-07-26 ? 14:58 +0200?Luca Tettamanti ??? > > >> - again ACPI video module gets the nodification (in this case > > >> ACPI_VIDE

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

2012-07-27 Thread joeyli
? ??2012-07-26 ? 14:58 +0200?Luca Tettamanti ??? > - again ACPI video module gets the nodification (in this case > ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE > - KDE seems this and muck with the screen configuration :( > - meanwhile the brightness notification is propaga

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

2012-07-27 Thread Alex Deucher
On Fri, Jul 27, 2012 at 11:32 AM, joeyli wrote: > 於 五,2012-07-27 於 09:21 -0400,Alex Deucher 提到: >> On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: >> > >> > + * flags >> > + * bits 1:0: >> > + * 0 - Notify(VGA, 0x81) is not used for notification >> > + * 1 - Notify(VGA, 0x81) is used for notifica

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

2012-07-27 Thread Alex Deucher
On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > ? ??2012-07-26 ? 23:31 -0400?Alex Deucher ??? >> On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: >> > ? ??2012-07-26 ? 14:58 +0200?Luca Tettamanti ??? >> >> - again ACPI video module gets the nodification (in this case >> >> ACPI_VIDEO_NOTIFY_PR

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

2012-07-27 Thread Alex Deucher
On Fri, Jul 27, 2012 at 12:46 AM, joeyli wrote: > 於 四,2012-07-26 於 23:31 -0400,Alex Deucher 提到: >> On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: >> > 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: >> >> - again ACPI video module gets the nodification (in this case >> >> ACPI_VIDEO_NOTIFY_PR

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

2012-07-27 Thread Luca Tettamanti
On Fri, Jul 27, 2012 at 12:46:50PM +0800, joeyli wrote: > 於 四,2012-07-26 於 23:31 -0400,Alex Deucher 提到: > > On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > > > 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: > > >> - again ACPI video module gets the nodification (in this case > > >> ACPI_VIDE

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > ? ??2012-07-26 ? 14:58 +0200?Luca Tettamanti ??? >> - again ACPI video module gets the nodification (in this case >> ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE >> - KDE seems this and muck with the screen configuration :

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

2012-07-26 Thread Luca Tettamanti
On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti > wrote: > > The other missing bit is how to actually change the brightness... Alex, > > do you know what registers to poke? > > You need to check if the GPU controls the backlight or

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 10:50 PM, joeyli wrote: > 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: >> - again ACPI video module gets the nodification (in this case >> ACPI_VIDEO_NOTIFY_PROBE), re-enumerated and send KEY_SWITCHVIDEOMODE >> - KDE seems this and muck with the screen configuration :

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 3:42 PM, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti > wrote: >> On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >>> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti >>> wrote: >>> > The other missing bit is how to actually chang

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti wrote: > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti >> wrote: >> > The other missing bit is how to actually change the brightness... Alex, >> > do you know what registers to po

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

2012-07-26 Thread Luca Tettamanti
On Wed, Jul 25, 2012 at 01:38:09PM -0400, alexdeucher at gmail.com wrote: > From: Alex Deucher > > Add a new header that defines the AMD ACPI interface used > for laptops, PowerXpress, and chipset specific functionality > and update the current code to use it. Great! Now my DSDT makes sense ;)

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 3:42 PM, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti wrote: >> On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >>> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti >>> wrote: >>> > The other missing bit is how to actually change t

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti wrote: > On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: >> On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti wrote: >> > The other missing bit is how to actually change the brightness... Alex, >> > do you know what registers to poke?

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

2012-07-26 Thread Luca Tettamanti
On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: > On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti wrote: > > The other missing bit is how to actually change the brightness... Alex, > > do you know what registers to poke? > > You need to check if the GPU controls the backlight or th

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti wrote: > On Wed, Jul 25, 2012 at 01:38:09PM -0400, alexdeucher at gmail.com wrote: >> From: Alex Deucher >> >> Add a new header that defines the AMD ACPI interface used >> for laptops, PowerXpress, and chipset specific functionality >> and update t

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

2012-07-26 Thread Alex Deucher
On Thu, Jul 26, 2012 at 8:58 AM, Luca Tettamanti wrote: > On Wed, Jul 25, 2012 at 01:38:09PM -0400, alexdeuc...@gmail.com wrote: >> From: Alex Deucher >> >> Add a new header that defines the AMD ACPI interface used >> for laptops, PowerXpress, and chipset specific functionality >> and update the

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

2012-07-26 Thread Luca Tettamanti
On Wed, Jul 25, 2012 at 01:38:09PM -0400, alexdeuc...@gmail.com wrote: > From: Alex Deucher > > Add a new header that defines the AMD ACPI interface used > for laptops, PowerXpress, and chipset specific functionality > and update the current code to use it. Great! Now my DSDT makes sense ;) > T

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

2012-07-25 Thread alexdeuc...@gmail.com
From: Alex Deucher Add a new header that defines the AMD ACPI interface used for laptops, PowerXpress, and chipset specific functionality and update the current code to use it. Todo: - properly verify the ACPI interfaces - hook up and handle ACPI notifications - make PX code more robust - implem

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

2012-07-25 Thread alexdeucher
From: Alex Deucher Add a new header that defines the AMD ACPI interface used for laptops, PowerXpress, and chipset specific functionality and update the current code to use it. Todo: - properly verify the ACPI interfaces - hook up and handle ACPI notifications - make PX code more robust - implem