[stable] [PATCH] Input: ALPS - add interleaved protocol support (Dell E6x00 series)

Greg KH greg at kroah.com
Thu Jan 21 04:13:03 UTC 2010


On Fri, Jan 15, 2010 at 10:11:22AM -0800, Andy Isaacson wrote:
> From: Sebastian Kapfer <sebastian_kapfer at gmx.net>
> Date: Tue, 15 Dec 2009 08:39:50 -0800
> Subject: [PATCH] Input: ALPS - add interleaved protocol support (Dell E6x00 series)
> 
> commit 1d9f26262aef6d63ff65eba0fd5f1583f342b69b upstream
> 
> Properly handle version of the protocol where standard PS/2 packets
> from trackpoint are stuffed into middle (byte 3-6) of the standard
> ALPS packets when both the touchpad and trackpoint are used together.
> 
> The patch is based on work done by Matthew Chapman and additional
> research done by David Kubicek and Erik Osterholm:
> 
> 	https://bugs.launchpad.net/ubuntu/+source/linux/+bug/296610
> 
> Many thanks to David Kubicek for his efforts in researching fine points
> of this new version of the protocol, especially interaction between pad
> and stick in these models.
> 
> Signed-off-by: Sebastian Kapfer <sebastian_kapfer at gmx.net>
> Signed-off-by: Dmitry Torokhov <dtor at mail.ru>
> ---
> 
> Cherrypick to 2.6.31 by Andy Isaacson <adi at hexapodia.org>.  Tested on
> Dell E4300.

What -stable tree is this for?  .31 is no longer being maintained.  Is
this the proper patch for .32?  .27?

> 
>  drivers/input/mouse/alps.c |  252 +++++++++++++++++++++++++++++++++++++++-----
>  drivers/input/mouse/alps.h |    1 +
>  2 files changed, 228 insertions(+), 25 deletions(-)

Ick, that's a big patch, and kind of violates the -stable rules, right?
It's a "add new functionality" type thing that you are doing here from
what I can tell.  This doesn't fix a regression, does it?

confused,

greg k-h




More information about the kernel-team mailing list