[PATCH] Add model data for Dell Precision 3510 trackpad.

[PATCH] Add model data for Dell Precision 3510 trackpad.

[John L. Preston]

---
 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 328edc8c8786..c0a4edefe54d 100644
--- a/drivers/input/mouse/alps.c
+++ b/drivers/input/mouse/alps.c
@@ -139,6 +139,7 @@ static const struct alps_model_info alps_model_data[] = {
 	{ { 0x52, 0x01, 0x14 }, 0x00, { ALPS_PROTO_V2, 0xff, 0xff,
 		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED } },				/* Toshiba Tecra A11-11L */
 	{ { 0x73, 0x02, 0x64 }, 0x8a, { ALPS_PROTO_V4, 0x8f, 0x8f, 0 } },
+	{ { 0x73, 0x03, 0x28 }, 0x13, { ALPS_PROTO_V8, 0x18, 0x18, 0 } },			/* Dell Precision 3510 */
 };
 
 static const struct alps_protocol_info alps_v3_protocol_data = {
-- 
2.11.1

Re: [PATCH] Add model data for Dell Precision 3510 trackpad.

[Pali Rohár]

On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
> ---
>  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 328edc8c8786..c0a4edefe54d 100644
> --- a/drivers/input/mouse/alps.c
> +++ b/drivers/input/mouse/alps.c
> @@ -139,6 +139,7 @@ static const struct alps_model_info alps_model_data[] = {
>  	{ { 0x52, 0x01, 0x14 }, 0x00, { ALPS_PROTO_V2, 0xff, 0xff,
>  		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED } },				/* Toshiba Tecra A11-11L */
>  	{ { 0x73, 0x02, 0x64 }, 0x8a, { ALPS_PROTO_V4, 0x8f, 0x8f, 0 } },
> +	{ { 0x73, 0x03, 0x28 }, 0x13, { ALPS_PROTO_V8, 0x18, 0x18, 0 } },			/* Dell Precision 3510 */
>  };
>  
>  static const struct alps_protocol_info alps_v3_protocol_data = {

Hi John! This is not a correct place do add new proto data. I already
proposed reducing this table to just for PROTO_V2, but Dmitry was not
happy about it.

Anyway, look at function alps_identify() where is detection for ALPS_V8:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/input/mouse/alps.c?h=v4.10-rc8#n2829

V8 is already detected when e7 is (0x73, 0x03, 0x28) and ec[1] is 0x01.
What are your ec data? Probably ec[1] is not 0x01 as otherwise it should
be already detected.

Btw, from where you got your proto data in this patch?

-- 
Pali Rohár
pali.rohar@gmail.com

Re: [PATCH] Add model data for Dell Precision 3510 trackpad.

[John Preston]

On 2017-02-19 13:35, Pali Rohár wrote:
> On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
>> ---
>>  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 328edc8c8786..c0a4edefe54d 100644
>> --- a/drivers/input/mouse/alps.c
>> +++ b/drivers/input/mouse/alps.c
>> @@ -139,6 +139,7 @@ static const struct alps_model_info 
>> alps_model_data[] = {
>>  	{ { 0x52, 0x01, 0x14 }, 0x00, { ALPS_PROTO_V2, 0xff, 0xff,
>>  		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED } },				/* 
>> Toshiba Tecra A11-11L */
>>  	{ { 0x73, 0x02, 0x64 }, 0x8a, { ALPS_PROTO_V4, 0x8f, 0x8f, 0 } },
>> +	{ { 0x73, 0x03, 0x28 }, 0x13, { ALPS_PROTO_V8, 0x18, 0x18, 0 } 
>> },			/* Dell Precision 3510 */
>>  };
>> 
>>  static const struct alps_protocol_info alps_v3_protocol_data = {
> 
> Hi John! This is not a correct place do add new proto data. I already
> proposed reducing this table to just for PROTO_V2, but Dmitry was not
> happy about it.

Hi Pali. :)

> 
> Anyway, look at function alps_identify() where is detection for 
> ALPS_V8:
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/input/mouse/alps.c?h=v4.10-rc8#n2829
> 
> V8 is already detected when e7 is (0x73, 0x03, 0x28) and ec[1] is 0x01.
> What are your ec data? Probably ec[1] is not 0x01 as otherwise it 
> should
> be already detected.

I have just found that code and am confused because that is not the same 
code I have running on my system. My ec[1] is 0x01 so you are correct, 
it should already be detected and my patch is unnecessary. Sorry for the 
inconvenience.

> 
> Btw, from where you got your proto data in this patch?

I pulled it from dmesg:

[john@slate ~]$ dmesg|grep "Unknown ALPS touchpad"
[    3.071891] psmouse serio1: alps: Unknown ALPS touchpad: E7=73 03 28, 
EC=73 01 13

Re: [PATCH] Add model data for Dell Precision 3510 trackpad.

[Marcos Paulo de Souza]

Hi John and Pali,

On Sun, Feb 19, 2017 at 03:24:58PM +0000, John Preston wrote:
> On 2017-02-19 13:35, Pali Rohár wrote:
> > On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
> > > ---
> > >  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 328edc8c8786..c0a4edefe54d 100644
> > > --- a/drivers/input/mouse/alps.c
> > > +++ b/drivers/input/mouse/alps.c
> > > @@ -139,6 +139,7 @@ static const struct alps_model_info
> > > alps_model_data[] = {
> > >  	{ { 0x52, 0x01, 0x14 }, 0x00, { ALPS_PROTO_V2, 0xff, 0xff,
> > >  		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED } },				/*
> > > Toshiba Tecra A11-11L */
> > >  	{ { 0x73, 0x02, 0x64 }, 0x8a, { ALPS_PROTO_V4, 0x8f, 0x8f, 0 } },
> > > +	{ { 0x73, 0x03, 0x28 }, 0x13, { ALPS_PROTO_V8, 0x18, 0x18, 0 }
> > > },			/* Dell Precision 3510 */
> > >  };
> > > 
> > >  static const struct alps_protocol_info alps_v3_protocol_data = {
> > 
> > Hi John! This is not a correct place do add new proto data. I already
> > proposed reducing this table to just for PROTO_V2, but Dmitry was not
> > happy about it.
> 
> Hi Pali. :)
> 
> > 
> > Anyway, look at function alps_identify() where is detection for ALPS_V8:
> > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/input/mouse/alps.c?h=v4.10-rc8#n2829
> > 
> > V8 is already detected when e7 is (0x73, 0x03, 0x28) and ec[1] is 0x01.
> > What are your ec data? Probably ec[1] is not 0x01 as otherwise it should
> > be already detected.
> 
> I have just found that code and am confused because that is not the same
> code I have running on my system. My ec[1] is 0x01 so you are correct, it
> should already be detected and my patch is unnecessary. Sorry for the
> inconvenience.
> 
> > 
> > Btw, from where you got your proto data in this patch?
> 
> I pulled it from dmesg:
> 
> [john@slate ~]$ dmesg|grep "Unknown ALPS touchpad"
> [    3.071891] psmouse serio1: alps: Unknown ALPS touchpad: E7=73 03 28,
> EC=73 01 13

There are other open bugs mentioning the same problem, with similar ALPS devices:
https://bugzilla.kernel.org/show_bug.cgi?id=57841
https://bugzilla.kernel.org/show_bug.cgi?id=43931
https://bugzilla.kernel.org/show_bug.cgi?id=80181
https://bugzilla.kernel.org/show_bug.cgi?id=67941

This one without even a dmesg output:
https://bugzilla.kernel.org/show_bug.cgi?id=84081

Maybe they are related, or  also need some touch to let them being discovered as proper devices?

Thanks,
Marcos

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

Unknown ALPS touchpad

[Pali Rohár]

Adding Masaki into discussion.

On Sunday 19 February 2017 20:01:07 Marcos Paulo de Souza wrote:
> There are other open bugs mentioning the same problem, with similar ALPS devices:
> https://bugzilla.kernel.org/show_bug.cgi?id=57841

Unknown ALPS touchpad: E7=73 03 50, EC=73 02 02

> https://bugzilla.kernel.org/show_bug.cgi?id=43931

Unknown ALPS touchpad: E7=73 00 14, EC=10 00 64

Should be supported since commit 95f75e91 which was introduced in Linux
version v3.13-rc4.

> https://bugzilla.kernel.org/show_bug.cgi?id=80181

Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 22

> https://bugzilla.kernel.org/show_bug.cgi?id=67941

Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 18

> This one without even a dmesg output:
> https://bugzilla.kernel.org/show_bug.cgi?id=84081

Sorry, this one does not contain any useful information.

> Maybe they are related, or  also need some touch to let them being discovered as proper devices?

Masaki, can you look at those unknown ALPS touchpad identifiers and
provide some information about protocol which they are using?

Basically touchpads with these identifiers are unsupported by last
kernel version:

E7=73 03 50, EC=73 02 02
E7=73 03 0a, EC=88 b3 22
E7=73 03 0a, EC=88 b3 18

-- 
Pali Rohár
pali.rohar@gmail.com

RE: Unknown ALPS touchpad

[Masaki Ota]

Hi, Pali,

I know these devices, and they are supported in new kernel.
I'm not sure when they are supported from.
In alps.c has below code.

static int alps_identify(struct psmouse *psmouse, struct alps_data *priv)
{...
...
		if (e7[0] == 0x73 && e7[1] == 0x03 && e7[2] == 0x50 &&
			   ec[0] == 0x73 && (ec[1] == 0x01 || ec[1] == 0x02)) {
			protocol = &alps_v5_protocol_data;
		} else if (ec[0] == 0x88 &&
			   ((ec[1] & 0xf0) == 0xb0 || (ec[1] & 0xf0) == 0xc0)) {
			protocol = &alps_v7_protocol_data;}
}

E7=73 03 50, EC=73 02 02 <- alps_v5_protocol_data
E7=73 03 0a, EC=88 b3 22 <- alps_v7_protocol_data
E7=73 03 0a, EC=88 b3 18 <- alps_v7_protocol_data

Best Regards,
Masaki Ota
-----Original Message-----
From: Pali Rohár [mailto:pali.rohar@gmail.com] 
Sent: Tuesday, February 21, 2017 12:51 AM
To: Marcos Paulo de Souza <marcos.souza.org@gmail.com>; 太田 真喜 Masaki Ota <masaki.ota@jp.alps.com>
Cc: John Preston <wcerfgba@riseup.net>; Dmitry Torokhov <dmitry.torokhov@gmail.com>; linux-input@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: Unknown ALPS touchpad

Adding Masaki into discussion.

On Sunday 19 February 2017 20:01:07 Marcos Paulo de Souza wrote:
> There are other open bugs mentioning the same problem, with similar ALPS devices:
> https://bugzilla.kernel.org/show_bug.cgi?id=57841

Unknown ALPS touchpad: E7=73 03 50, EC=73 02 02

> https://bugzilla.kernel.org/show_bug.cgi?id=43931

Unknown ALPS touchpad: E7=73 00 14, EC=10 00 64

Should be supported since commit 95f75e91 which was introduced in Linux version v3.13-rc4.

> https://bugzilla.kernel.org/show_bug.cgi?id=80181

Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 22

> https://bugzilla.kernel.org/show_bug.cgi?id=67941

Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 18

> This one without even a dmesg output:
> https://bugzilla.kernel.org/show_bug.cgi?id=84081

Sorry, this one does not contain any useful information.

> Maybe they are related, or  also need some touch to let them being discovered as proper devices?

Masaki, can you look at those unknown ALPS touchpad identifiers and provide some information about protocol which they are using?

Basically touchpads with these identifiers are unsupported by last kernel version:

E7=73 03 50, EC=73 02 02
E7=73 03 0a, EC=88 b3 22
E7=73 03 0a, EC=88 b3 18

--
Pali Rohár
pali.rohar@gmail.com

Re: Unknown ALPS touchpad

[Pali Rohár]

On Tuesday 21 February 2017 03:10:06 Masaki Ota wrote:
> Hi, Pali,
> 
> I know these devices, and they are supported in new kernel.
> I'm not sure when they are supported from.
> In alps.c has below code.
> 
> static int alps_identify(struct psmouse *psmouse, struct alps_data *priv)
> {...
> ...
> 		if (e7[0] == 0x73 && e7[1] == 0x03 && e7[2] == 0x50 &&
> 			   ec[0] == 0x73 && (ec[1] == 0x01 || ec[1] == 0x02)) {
> 			protocol = &alps_v5_protocol_data;
> 		} else if (ec[0] == 0x88 &&
> 			   ((ec[1] & 0xf0) == 0xb0 || (ec[1] & 0xf0) == 0xc0)) {
> 			protocol = &alps_v7_protocol_data;}
> }

Ou, sorry I have missed this checks for ec[0] == 0x88.

Marcos, so conclusion is that all those ALPS devices are already
supported by last mainline kernel. So you can close those bug reports.
If you need with git blame you can identify commit in which was support
added and then you can find in which kernel version was particular
commit first time.

> 
> E7=73 03 50, EC=73 02 02 <- alps_v5_protocol_data
> E7=73 03 0a, EC=88 b3 22 <- alps_v7_protocol_data
> E7=73 03 0a, EC=88 b3 18 <- alps_v7_protocol_data
> 
> Best Regards,
> Masaki Ota
> -----Original Message-----
> From: Pali Rohár [mailto:pali.rohar@gmail.com] 
> Sent: Tuesday, February 21, 2017 12:51 AM
> To: Marcos Paulo de Souza <marcos.souza.org@gmail.com>; 太田 真喜 Masaki Ota <masaki.ota@jp.alps.com>
> Cc: John Preston <wcerfgba@riseup.net>; Dmitry Torokhov <dmitry.torokhov@gmail.com>; linux-input@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Unknown ALPS touchpad
> 
> Adding Masaki into discussion.
> 
> On Sunday 19 February 2017 20:01:07 Marcos Paulo de Souza wrote:
> > There are other open bugs mentioning the same problem, with similar ALPS devices:
> > https://bugzilla.kernel.org/show_bug.cgi?id=57841
> 
> Unknown ALPS touchpad: E7=73 03 50, EC=73 02 02
> 
> > https://bugzilla.kernel.org/show_bug.cgi?id=43931
> 
> Unknown ALPS touchpad: E7=73 00 14, EC=10 00 64
> 
> Should be supported since commit 95f75e91 which was introduced in Linux version v3.13-rc4.
> 
> > https://bugzilla.kernel.org/show_bug.cgi?id=80181
> 
> Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 22
> 
> > https://bugzilla.kernel.org/show_bug.cgi?id=67941
> 
> Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 18
> 
> > This one without even a dmesg output:
> > https://bugzilla.kernel.org/show_bug.cgi?id=84081
> 
> Sorry, this one does not contain any useful information.
> 
> > Maybe they are related, or  also need some touch to let them being discovered as proper devices?
> 
> Masaki, can you look at those unknown ALPS touchpad identifiers and provide some information about protocol which they are using?
> 
> Basically touchpads with these identifiers are unsupported by last kernel version:
> 
> E7=73 03 50, EC=73 02 02
> E7=73 03 0a, EC=88 b3 22
> E7=73 03 0a, EC=88 b3 18
> 
> --
> Pali Rohár
> pali.rohar@gmail.com

-- 
Pali Rohár
pali.rohar@gmail.com

Re: Unknown ALPS touchpad

[Marcos Paulo de Souza]

On Tue, Feb 21, 2017 at 09:34:39AM +0100, Pali Rohár wrote:
> On Tuesday 21 February 2017 03:10:06 Masaki Ota wrote:
> > Hi, Pali,
> > 
> > I know these devices, and they are supported in new kernel.
> > I'm not sure when they are supported from.
> > In alps.c has below code.
> > 
> > static int alps_identify(struct psmouse *psmouse, struct alps_data *priv)
> > {...
> > ...
> > 		if (e7[0] == 0x73 && e7[1] == 0x03 && e7[2] == 0x50 &&
> > 			   ec[0] == 0x73 && (ec[1] == 0x01 || ec[1] == 0x02)) {
> > 			protocol = &alps_v5_protocol_data;
> > 		} else if (ec[0] == 0x88 &&
> > 			   ((ec[1] & 0xf0) == 0xb0 || (ec[1] & 0xf0) == 0xc0)) {
> > 			protocol = &alps_v7_protocol_data;}
> > }
> 
> Ou, sorry I have missed this checks for ec[0] == 0x88.
> 
> Marcos, so conclusion is that all those ALPS devices are already
> supported by last mainline kernel. So you can close those bug reports.
> If you need with git blame you can identify commit in which was support
> added and then you can find in which kernel version was particular
> commit first time.

Thanks Pali and Masaki, I closed all mentioned commits, pointing the
right kernel version that supports such devices.

> 
> > 
> > E7=73 03 50, EC=73 02 02 <- alps_v5_protocol_data
> > E7=73 03 0a, EC=88 b3 22 <- alps_v7_protocol_data
> > E7=73 03 0a, EC=88 b3 18 <- alps_v7_protocol_data
> > 
> > Best Regards,
> > Masaki Ota
> > -----Original Message-----
> > From: Pali Rohár [mailto:pali.rohar@gmail.com] 
> > Sent: Tuesday, February 21, 2017 12:51 AM
> > To: Marcos Paulo de Souza <marcos.souza.org@gmail.com>; 太田 真喜 Masaki Ota <masaki.ota@jp.alps.com>
> > Cc: John Preston <wcerfgba@riseup.net>; Dmitry Torokhov <dmitry.torokhov@gmail.com>; linux-input@vger.kernel.org; linux-kernel@vger.kernel.org
> > Subject: Unknown ALPS touchpad
> > 
> > Adding Masaki into discussion.
> > 
> > On Sunday 19 February 2017 20:01:07 Marcos Paulo de Souza wrote:
> > > There are other open bugs mentioning the same problem, with similar ALPS devices:
> > > https://bugzilla.kernel.org/show_bug.cgi?id=57841
> > 
> > Unknown ALPS touchpad: E7=73 03 50, EC=73 02 02
> > 
> > > https://bugzilla.kernel.org/show_bug.cgi?id=43931
> > 
> > Unknown ALPS touchpad: E7=73 00 14, EC=10 00 64
> > 
> > Should be supported since commit 95f75e91 which was introduced in Linux version v3.13-rc4.
> > 
> > > https://bugzilla.kernel.org/show_bug.cgi?id=80181
> > 
> > Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 22
> > 
> > > https://bugzilla.kernel.org/show_bug.cgi?id=67941
> > 
> > Unknown ALPS touchpad: E7=73 03 0a, EC=88 b3 18
> > 
> > > This one without even a dmesg output:
> > > https://bugzilla.kernel.org/show_bug.cgi?id=84081
> > 
> > Sorry, this one does not contain any useful information.
> > 
> > > Maybe they are related, or  also need some touch to let them being discovered as proper devices?
> > 
> > Masaki, can you look at those unknown ALPS touchpad identifiers and provide some information about protocol which they are using?
> > 
> > Basically touchpads with these identifiers are unsupported by last kernel version:
> > 
> > E7=73 03 50, EC=73 02 02
> > E7=73 03 0a, EC=88 b3 22
> > E7=73 03 0a, EC=88 b3 18
> > 
> > --
> > Pali Rohár
> > pali.rohar@gmail.com
> 
> -- 
> Pali Rohár
> pali.rohar@gmail.com

Re: [PATCH] Add model data for Dell Precision 3510 trackpad.

[Pali Rohár]

On Sunday 19 February 2017 15:24:58 John Preston wrote:
> On 2017-02-19 13:35, Pali Rohár wrote:
> >On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
> >>---
> >> 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 328edc8c8786..c0a4edefe54d 100644
> >>--- a/drivers/input/mouse/alps.c
> >>+++ b/drivers/input/mouse/alps.c
> >>@@ -139,6 +139,7 @@ static const struct alps_model_info
> >>alps_model_data[] = {
> >> 	{ { 0x52, 0x01, 0x14 }, 0x00, { ALPS_PROTO_V2, 0xff, 0xff,
> >> 		ALPS_PASS | ALPS_DUALPOINT | ALPS_PS2_INTERLEAVED } },				/* Toshiba
> >>Tecra A11-11L */
> >> 	{ { 0x73, 0x02, 0x64 }, 0x8a, { ALPS_PROTO_V4, 0x8f, 0x8f, 0 } },
> >>+	{ { 0x73, 0x03, 0x28 }, 0x13, { ALPS_PROTO_V8, 0x18, 0x18, 0 } },			/*
> >>Dell Precision 3510 */
> >> };
> >>
> >> static const struct alps_protocol_info alps_v3_protocol_data = {
> >
> >Hi John! This is not a correct place do add new proto data. I already
> >proposed reducing this table to just for PROTO_V2, but Dmitry was not
> >happy about it.
> 
> Hi Pali. :)
> 
> >
> >Anyway, look at function alps_identify() where is detection for ALPS_V8:
> >https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/input/mouse/alps.c?h=v4.10-rc8#n2829
> >
> >V8 is already detected when e7 is (0x73, 0x03, 0x28) and ec[1] is 0x01.
> >What are your ec data? Probably ec[1] is not 0x01 as otherwise it should
> >be already detected.
> 
> I have just found that code and am confused because that is not the same
> code I have running on my system. My ec[1] is 0x01 so you are correct, it
> should already be detected and my patch is unnecessary. Sorry for the
> inconvenience.
> 
> >
> >Btw, from where you got your proto data in this patch?
> 
> I pulled it from dmesg:
> 
> [john@slate ~]$ dmesg|grep "Unknown ALPS touchpad"
> [    3.071891] psmouse serio1: alps: Unknown ALPS touchpad: E7=73 03 28,
> EC=73 01 13
> 

So you have old kernel without support for your touchpad...

-- 
Pali Rohár
pali.rohar@gmail.com