linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] input: alps: Add support for the touchpad on Dell Latitude E6410
@ 2010-05-15 13:52 Thomas Bächler
  2010-05-15 14:05 ` Thomas Bächler
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Bächler @ 2010-05-15 13:52 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: linux-input, alessandro, Thomas Bächler

---
 drivers/input/mouse/alps.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c
index 99d5876..0c83070 100644
--- a/drivers/input/mouse/alps.c
+++ b/drivers/input/mouse/alps.c
@@ -66,6 +66,8 @@ static const struct alps_model_info alps_model_data[] = {
 	{ { 0x73, 0x02, 0x50 }, 0xcf, 0xcf, ALPS_FOUR_BUTTONS },	  /* Dell Vostro 1400 */
 	{ { 0x52, 0x01, 0x14 }, 0xff, 0xff,
 		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED },	  /* Toshiba Tecra A11-11L */
+	{ { 0x73, 0x02, 0x64 }, 0xff, 0xff,
+		ALPS_PASS | ALPS_DUALPOINT },				  /* Dell Latitude E6410 */
 };
 
 /*
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] input: alps: Add support for the touchpad on Dell Latitude E6410
  2010-05-15 13:52 [PATCH] input: alps: Add support for the touchpad on Dell Latitude E6410 Thomas Bächler
@ 2010-05-15 14:05 ` Thomas Bächler
  2010-05-20 23:44   ` Chase Douglas
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Bächler @ 2010-05-15 14:05 UTC (permalink / raw)
  To: Thomas Bächler; +Cc: Dmitry Torokhov, linux-input, alessandro

[-- Attachment #1: Type: text/plain, Size: 1121 bytes --]

Am 15.05.2010 15:52, schrieb Thomas Bächler:
> ---
>  drivers/input/mouse/alps.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c
> index 99d5876..0c83070 100644
> --- a/drivers/input/mouse/alps.c
> +++ b/drivers/input/mouse/alps.c
> @@ -66,6 +66,8 @@ static const struct alps_model_info alps_model_data[] = {
>  	{ { 0x73, 0x02, 0x50 }, 0xcf, 0xcf, ALPS_FOUR_BUTTONS },	  /* Dell Vostro 1400 */
>  	{ { 0x52, 0x01, 0x14 }, 0xff, 0xff,
>  		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED },	  /* Toshiba Tecra A11-11L */
> +	{ { 0x73, 0x02, 0x64 }, 0xff, 0xff,
> +		ALPS_PASS | ALPS_DUALPOINT },				  /* Dell Latitude E6410 */
>  };
>  
>  /*

Okay, never mind, we were too quick in thinking everything was correct,
the parameters seem to be wrong. The touchpad is detected and seems to
work, but in 'synclient -m', you see that actually nothing happens.

Dmitry, do you know which parameters we need to tweak to fix this? Maybe
the 0xff, 0xff mask is wrong? Or the flags? Sorry, my knowledge ends here.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] input: alps: Add support for the touchpad on Dell Latitude E6410
  2010-05-15 14:05 ` Thomas Bächler
@ 2010-05-20 23:44   ` Chase Douglas
  0 siblings, 0 replies; 3+ messages in thread
From: Chase Douglas @ 2010-05-20 23:44 UTC (permalink / raw)
  To: Thomas Bächler; +Cc: Dmitry Torokhov, linux-input, alessandro

On Sat, May 15, 2010 at 10:05 AM, Thomas Bächler <thomas@archlinux.org> wrote:
> Am 15.05.2010 15:52, schrieb Thomas Bächler:
>> ---
>>  drivers/input/mouse/alps.c |    2 ++
>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c
>> index 99d5876..0c83070 100644
>> --- a/drivers/input/mouse/alps.c
>> +++ b/drivers/input/mouse/alps.c
>> @@ -66,6 +66,8 @@ static const struct alps_model_info alps_model_data[] = {
>>       { { 0x73, 0x02, 0x50 }, 0xcf, 0xcf, ALPS_FOUR_BUTTONS },          /* Dell Vostro 1400 */
>>       { { 0x52, 0x01, 0x14 }, 0xff, 0xff,
>>               ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED },      /* Toshiba Tecra A11-11L */
>> +     { { 0x73, 0x02, 0x64 }, 0xff, 0xff,
>> +             ALPS_PASS | ALPS_DUALPOINT },                             /* Dell Latitude E6410 */
>>  };
>>
>>  /*
>
> Okay, never mind, we were too quick in thinking everything was correct,
> the parameters seem to be wrong. The touchpad is detected and seems to
> work, but in 'synclient -m', you see that actually nothing happens.
>
> Dmitry, do you know which parameters we need to tweak to fix this? Maybe
> the 0xff, 0xff mask is wrong? Or the flags? Sorry, my knowledge ends here.

Been there, done that. I originally pushed a patch with this signature
about a month ago. It was applied, but then reverted because it didn't
work right. It seems there's a new protocol for these alps touchpads
that will need to be figured out first.

-- Chase
--
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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-05-20 23:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-15 13:52 [PATCH] input: alps: Add support for the touchpad on Dell Latitude E6410 Thomas Bächler
2010-05-15 14:05 ` Thomas Bächler
2010-05-20 23:44   ` Chase Douglas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).