ACK: [PATCH natty,oneiric] UBUNTU: SAUCE: Unregister input device only if it is registered
Seth Forshee
seth.forshee at canonical.com
Mon Sep 19 13:37:43 UTC 2011
On Mon, Sep 19, 2011 at 06:56:15AM -0600, Tim Gardner wrote:
> On 09/18/2011 07:25 PM, Jesse Sung wrote:
> >https://bugs.launchpad.net/bugs/839238
> >
> >SRU Justification:
> >
> >Using an ALPS touchpad in the quirk list makes unloading psmouse.ko hang
> >the system.
> >Also, due to the protocol problem of ALPS touchpad, the disconnect
> >function would be called when system is running, and cause a random hang.
> >
> >Fix:
> >
> > From e14874429daac619a2bea5030494ffd68230bfa1 Mon Sep 17 00:00:00 2001
> >From: Wen-chien Jesse Sung <jesse.sung at canonical.com>
> >Date: Fri, 16 Sep 2011 15:20:22 +0800
> >Subject: [PATCH] UBUNTU: SAUCE: Unregister input device only if it is
> >registered
> >
> >BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/839238
> >
> >dev2 is not registered in alps_model_quirk_enabled mode, do not
> >unregister while disconnecting.
> >
> >Signed-off-by: Wen-chien Jesse Sung <jesse.sung at canonical.com>
> >---
> >drivers/input/mouse/alps.c | 3 ++-
> >1 files changed, 2 insertions(+), 1 deletions(-)
> >
> >diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c
> >index da20b62..9adff60 100644
> >--- a/drivers/input/mouse/alps.c
> >+++ b/drivers/input/mouse/alps.c
> >@@ -745,7 +745,8 @@ static void alps_disconnect(struct psmouse *psmouse)
> >
> >psmouse_reset(psmouse);
> >del_timer_sync(&priv->timer);
> >- input_unregister_device(priv->dev2);
> >+ if (!alps_model_quirk_enabled)
> >+ input_unregister_device(priv->dev2);
> >kfree(priv);
> >}
> >
>
> Have you run this patch by the upstream maintainer ? It seems like a
> good stable patch as well.
I think this problem is caused by our sauce patches for IMPS/2 support
on certain models, therefore it isn't applicable to upstream.
Acked-by: Seth Forshee <seth.forshee at canonical.com>
>
> Acked-by: Tim Gardner <tim.gardner at canonical.com>
>
> rtg
> --
> Tim Gardner tim.gardner at canonical.com
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
More information about the kernel-team
mailing list