[PATCH/RFC] drm/radeon: ACPI: veto the keypress on ATIF events
Zhang Rui
rui.zhang at intel.com
Thu Aug 2 18:40:35 PDT 2012
On 四, 2012-08-02 at 15:46 +0200, Luca Tettamanti wrote:
> On Thu, Aug 02, 2012 at 08:45:30AM +0800, Zhang Rui wrote:
> > On 三, 2012-08-01 at 15:49 +0200, Luca Tettamanti wrote:
> > > AMD ACPI interface may overload the standard event
> > > ACPI_VIDEO_NOTIFY_PROBE (0x81) to signal AMD-specific events. In such
> > > cases we don't want to send the keypress (KEY_SWITCHVIDEOMODE) to the
> > > userspace because the user did not press the mode switch key (the
> > > spurious keypress confuses the DE which usually changes the
> > > display configuration and messes up a dual-screen setup).
> > > This patch gives the radeon driver the chance to examine the event and
> > > block the keypress if the event is an "AMD event".
> > >
> > > Signed-off-by: Luca Tettamanti <kronos.it at gmail.com>
> > > ---
> > > Any comment from ACPI front?
> > >
> > it looks good to me.
> > But I'm wondering if we can use the following code for ACPI part, which
> > looks cleaner.
> > I know this may change the behavior of other events, but in theory, we
> > should not send any input event if we know something wrong in kernel.
> >
> > what do you think?
>
> I like it, it's cleaner.
> I've split the patch in two pieces (one for video, the other for
> radeon) and adopted your suggestion.
>
Great.
Acked-by: Zhang Rui <rui.zhang at intel.com>
hmm, who should take these two patches?
and which tree the second patch is based on?
thanks,
rui
More information about the dri-devel
mailing list