| Message ID | 1382715581-2584-1-git-send-email-yunkang.tang@cn.alps.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Oct 25, 2013 at 11:39:41PM +0800, Yunkang Tang wrote: > Hi all, > > This patch adding the support for touchpad on Dell XT2 model. > It's a dual device with device ID: 73, 00, 14, that comply with "ALPS_PROTO_V2". > > > Signed-off-by: Yunkang Tang <yunkang.tang@cn.alps.com> Tested-by: Gaspard Jankowiak <gaspard@oknaj.eu> Reviewed-by: Niels de Vos <ndevos@redhat.com> (well, for as far a one-line review goes) Thanks! > --- > drivers/input/mouse/alps.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > index ca7a26f..24b3626 100644 > --- a/drivers/input/mouse/alps.c > +++ b/drivers/input/mouse/alps.c > @@ -103,6 +103,7 @@ static const struct alps_model_info alps_model_data[] = { > /* Dell Latitude E5500, E6400, E6500, Precision M4400 */ > { { 0x62, 0x02, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, > ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, > + { { 0x73, 0x00, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_DUALPOINT }, /* Dell XT2 */ > { { 0x73, 0x02, 0x50 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_FOUR_BUTTONS }, /* Dell Vostro 1400 */ > { { 0x52, 0x01, 0x14 }, 0x00, ALPS_PROTO_V2, 0xff, 0xff, > ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, /* Toshiba Tecra A11-11L */ > -- > 1.8.1.2 > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Oct 25, 2013 at 11:39:41PM +0800, Yunkang Tang wrote: > Hi all, > > This patch adding the support for touchpad on Dell XT2 model. > It's a dual device with device ID: 73, 00, 14, that comply with "ALPS_PROTO_V2". > > > Signed-off-by: Yunkang Tang <yunkang.tang@cn.alps.com> Applied, thank you. > --- > drivers/input/mouse/alps.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > index ca7a26f..24b3626 100644 > --- a/drivers/input/mouse/alps.c > +++ b/drivers/input/mouse/alps.c > @@ -103,6 +103,7 @@ static const struct alps_model_info alps_model_data[] = { > /* Dell Latitude E5500, E6400, E6500, Precision M4400 */ > { { 0x62, 0x02, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, > ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, > + { { 0x73, 0x00, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_DUALPOINT }, /* Dell XT2 */ > { { 0x73, 0x02, 0x50 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_FOUR_BUTTONS }, /* Dell Vostro 1400 */ > { { 0x52, 0x01, 0x14 }, 0x00, ALPS_PROTO_V2, 0xff, 0xff, > ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, /* Toshiba Tecra A11-11L */ > -- > 1.8.1.2 >
On 31/10/13 08:56, Dmitry Torokhov wrote: > > Applied, thank you. > Hi, I don't know how to go about this, but it seems the patch is affecting other input devices. The trackpoint goes haywire when used, making the cursor jump all over the place and triggering random "right clicks" on the Latitude XT2. I don't know if it affects other models too. Sorry for not spotting this earlier. Name : kernel Version : 3.11.6 Release : 201.fc19 Architecture: x86_64 Install Date: Tue 05 Nov 2013 09:22:37 CET Group : System Environment/Kernel Size : 134433560 License : GPLv2 and Redistributable, no modification permitted Signature : RSA/SHA256, Sat 02 Nov 2013 22:00:51 CET, Key ID 07477e65fb4b18e6 Source RPM : kernel-3.11.6-201.fc19.src.rpm Build Date : Sat 02 Nov 2013 15:35:07 CET Best, Gaspard -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c index ca7a26f..24b3626 100644 --- a/drivers/input/mouse/alps.c +++ b/drivers/input/mouse/alps.c @@ -103,6 +103,7 @@ static const struct alps_model_info alps_model_data[] = { /* Dell Latitude E5500, E6400, E6500, Precision M4400 */ { { 0x62, 0x02, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, + { { 0x73, 0x00, 0x14 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_DUALPOINT }, /* Dell XT2 */ { { 0x73, 0x02, 0x50 }, 0x00, ALPS_PROTO_V2, 0xcf, 0xcf, ALPS_FOUR_BUTTONS }, /* Dell Vostro 1400 */ { { 0x52, 0x01, 0x14 }, 0x00, ALPS_PROTO_V2, 0xff, 0xff, ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED }, /* Toshiba Tecra A11-11L */
Hi all, This patch adding the support for touchpad on Dell XT2 model. It's a dual device with device ID: 73, 00, 14, that comply with "ALPS_PROTO_V2". Signed-off-by: Yunkang Tang <yunkang.tang@cn.alps.com> --- drivers/input/mouse/alps.c | 1 + 1 file changed, 1 insertion(+)