From: Alex Deucher <alexdeucher@gmail.com>
To: joeyli <jlee@suse.com>
Cc: dri-devel@lists.freedesktop.org, linux-acpi@vger.kernel.org,
Alex Deucher <alexander.deucher@amd.com>,
Zhang Rui <rui.zhang@intel.com>
Subject: Re: [PATCH] drm/radeon: add new AMD ACPI header and update relevant code
Date: Thu, 26 Jul 2012 23:31:05 -0400 [thread overview]
Message-ID: <CADnq5_Mz5cjyYZ=1gjq+iaGtizhckKBL23gLaNJQFAPXHGF4TQ@mail.gmail.com> (raw)
In-Reply-To: <1343357441.8142.18.camel@linux-s257.site>
On Thu, Jul 26, 2012 at 10:50 PM, joeyli <jlee@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 configuration :(
>> - meanwhile the brightness notification is propagated, the
>> hypothetical
>> radeon driver does its magic to adjust the screen.
>>
>> My first idea would be to make ACPI_VIDEO_NOTIFY_PROBE also call to
>> the
>> acpi notifier chain, and allow the handlers to veto the key press
>> (like
>> it's done for ACPI_VIDEO_NOTIFY_SWITCH).
>
> I welcome this approach!
>
> On some ATI machine's DSDT also issue ACPI_VIDEO_NOTIFY_PROBE when
> AC-power unplug, because BIOS want to nodify video driver do some power
> saving stuff.
> It causes KEY_SWITCHVIDEOMODE issued by acpi/video driver when AC-power
> unplug.
>
> At least acpi/video driver need to know this 0x81 event is filed by BIOS
> to radeon-acpi for notify but not do video mode switch. That means the
> radeon drm need take the video switch responsibility.
Probably we'd just want the radeon acpi handler to just forward the
events to userspace so that the user can choose what to do with it
(xrandr command, etc.), otherwise we'll need to define policy in the
driver.
Alex
>
>
> Thanks a lot!
> Joey Lee
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2012-07-27 3:31 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-25 17:38 [PATCH] drm/radeon: add new AMD ACPI header and update relevant code alexdeucher
2012-07-26 12:58 ` Luca Tettamanti
2012-07-26 15:35 ` Alex Deucher
2012-07-26 19:33 ` Luca Tettamanti
2012-07-26 19:42 ` Alex Deucher
2012-07-26 19:58 ` Alex Deucher
2012-07-28 14:56 ` Luca Tettamanti
2012-07-28 21:29 ` Alex Deucher
2012-07-29 13:06 ` Luca Tettamanti
2012-07-30 14:20 ` Alex Deucher
2012-07-30 20:24 ` Luca Tettamanti
2012-07-30 20:30 ` Alex Deucher
2012-07-30 20:36 ` Luca Tettamanti
2012-07-30 20:45 ` Alex Deucher
2012-07-31 9:16 ` Luca Tettamanti
2012-07-31 13:58 ` Alex Deucher
2012-07-31 20:05 ` Luca Tettamanti
2012-07-31 21:33 ` Alex Deucher
2012-08-01 8:57 ` Luca Tettamanti
2012-08-01 13:56 ` Alex Deucher
2012-08-02 15:03 ` Alex Deucher
2012-08-02 16:31 ` Luca Tettamanti
2012-08-02 16:33 ` Alex Deucher
2012-08-02 20:54 ` Alex Deucher
2012-08-01 13:49 ` [PATCH/RFC] drm/radeon: ACPI: veto the keypress on ATIF events Luca Tettamanti
2012-08-01 14:02 ` Alex Deucher
2012-08-01 14:50 ` joeyli
2012-08-02 0:45 ` Zhang Rui
2012-08-02 13:46 ` Luca Tettamanti
2012-08-03 1:40 ` Zhang Rui
2012-08-03 1:45 ` Alex Deucher
2012-08-03 2:06 ` Zhang Rui
2012-07-29 19:33 ` [PATCH] drm/radeon: add new AMD ACPI header and update relevant code Luca Tettamanti
2012-07-30 14:29 ` Alex Deucher
2012-07-29 3:51 ` joeyli
2012-07-29 13:10 ` Luca Tettamanti
2012-07-30 8:32 ` joeyli
2012-07-30 14:16 ` Luca Tettamanti
2012-07-28 14:39 ` Pali Rohár
2012-07-27 2:50 ` joeyli
2012-07-27 3:31 ` Alex Deucher [this message]
2012-07-27 4:46 ` joeyli
2012-07-27 9:02 ` Luca Tettamanti
2012-07-27 13:21 ` Alex Deucher
2012-07-27 15:32 ` joeyli
2012-07-27 15:36 ` joeyli
2012-07-27 16:31 ` Alex Deucher
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CADnq5_Mz5cjyYZ=1gjq+iaGtizhckKBL23gLaNJQFAPXHGF4TQ@mail.gmail.com' \
--to=alexdeucher@gmail.com \
--cc=alexander.deucher@amd.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jlee@suse.com \
--cc=linux-acpi@vger.kernel.org \
--cc=rui.zhang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.