[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

[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

[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

[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,

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,

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 alexdeuc...@gmail.com 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

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 kronos...@gmail.com wrote: On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher alexdeuc...@gmail.com 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

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 alexdeuc...@gmail.com wrote: On Thu, Aug 2, 2012 at 12:31 PM, Luca Tettamanti kronos...@gmail.com wrote: On Thu, Aug 2, 2012 at 5:03 PM, Alex Deucher alexdeuc...@gmail.com wrote: I admit I'm not really an ACPI expert, but thinking about this more,

[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,

[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: >>

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

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 kronos...@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:

[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

[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

[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. > >

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 alexdeuc...@gmail.com 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

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 kronos...@gmail.com wrote: On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher alexdeuc...@gmail.com 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

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 kronos...@gmail.com wrote: On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher alexdeuc...@gmail.com wrote: Regarding patches 3 and 4, it might be easier to just store a pointer to

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: On Mon, Jul 30, 2012 at 10:45 PM, Alex Deucher alexdeuc...@gmail.com wrote:

[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

[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

[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

[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

[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

[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

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 BIOS bug):

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: I just found the first problem (probably a BIOS bug):

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 kronos...@gmail.com 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

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: I just

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: On Sat, Jul 28,

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 alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 4:24 PM, Luca Tettamanti kronos...@gmail.com wrote: On Mon, Jul 30, 2012 at 10:20:15AM -0400, Alex Deucher wrote: Supported DWORD Bit vector providing supported functions information. Each

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: On Sat, Jul 28, 2012 at 05:29:25PM -0400, Alex Deucher wrote: On Sat, Jul 28,

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 the DSDT,

[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

[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

[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

[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,

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 kronos...@gmail.com 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

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

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

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

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 j...@suse.com 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-27 於 23:32 +0800,joeyli 提到: 於 五,2012-07-27 於 09:21 -0400,Alex Deucher 提到: On Fri, Jul 27, 2012 at 12:46 AM, joeyli j...@suse.com wrote: + * flags + * bits 1:0: + * 0 - Notify(VGA, 0x81) is not used for notification + * 1 - Notify(VGA, 0x81) is used for notification

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 kronos...@gmail.com wrote: On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: On Thu, Jul 26, 2012 at

[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

[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

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

2012-07-28 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

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

2012-07-28 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-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 kronos...@gmail.com 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 kronos...@gmail.com wrote: The other

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: On Thu, Jul 26, 2012 at 11:35:25AM -0400, Alex Deucher wrote: On Thu, Jul 26,

[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 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 > > >>

[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

[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 >> >>

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

2012-07-27 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

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 j...@suse.com wrote: 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: - again ACPI video module gets the nodification (in this case

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 j...@suse.com wrote: 於 四,2012-07-26 於 23:31 -0400,Alex Deucher 提到: On Thu, Jul 26, 2012 at 10:50 PM, joeyli j...@suse.com wrote: 於 四,2012-07-26 於 14:58 +0200,Luca Tettamanti 提到: - again ACPI video module gets the nodification (in this case

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 j...@suse.com wrote: 於 五,2012-07-27 於 09:21 -0400,Alex Deucher 提到: On Fri, Jul 27, 2012 at 12:46 AM, joeyli j...@suse.com wrote: + * flags + * bits 1:0: + * 0 - Notify(VGA, 0x81) is not used for notification + * 1 - Notify(VGA, 0x81) is used for

[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

[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 ;)

[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

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 alexander.deuc...@amd.com 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

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 kronos...@gmail.com wrote: On Wed, Jul 25, 2012 at 01:38:09PM -0400, alexdeuc...@gmail.com wrote: From: Alex Deucher alexander.deuc...@amd.com Add a new header that defines the AMD ACPI interface used for laptops, PowerXpress, and chipset

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 kronos...@gmail.com 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

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 kronos...@gmail.com 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 kronos...@gmail.com wrote: The other missing bit is how to actually change the brightness... Alex, do you

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 alexdeuc...@gmail.com wrote: On Thu, Jul 26, 2012 at 3:33 PM, Luca Tettamanti kronos...@gmail.com 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 kronos...@gmail.com wrote:

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 j...@suse.com 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

[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

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

2012-07-25 Thread alexdeucher
From: Alex Deucher alexander.deuc...@amd.com 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