Hello,
Just tested an ugly quick patch just for testing. By assigning it a hw_version of 4 it works fine.
I'll find out what fw_version it gives and see how it translate for the ver variable
You can see what I did below. I just played with the "default" statement.
Thank you.
Patrick.
static int elantech_set_properties(struct elantech_data *etd)
{
/* This represents the version of IC body. */
int ver = (etd->fw_version & 0x0f0000) >> 16;
/* Early version of Elan touchpads doesn't obey the rule. */
if (etd->fw_version < 0x020030 || etd->fw_version == 0x020600)
etd->hw_version = 1;
else {
switch (ver) {
case 2:
case 4:
etd->hw_version = 2;
break;
case 5:
etd->hw_version = 3;
break;
case 6:
case 7:
case 8:
case 9:
case 10:
case 13:
case 14:
etd->hw_version = 4;
break;
default:
etd->hw_version = 4;
break;
/*return -1;*/
The information in this message, including in all attachments, is
confidential or privileged. In the event you have received this message in error
and are not the intended recipient, you are hereby advised that any use, copying
or reproduction of this document is strictly forbidden. Please notify
immediately the sender of this error and destroy this message, including its
attachments, as the case may be.
</P>
L'information apparaissant dans ce message electronique et dans les documents
qui y sont joints est de nature confidentielle ou privilegiee. Si ce message
vous est parvenu par erreur et que vous n'en etes pas le destinataire vise, vous
etes par les presentes avise que toute utilisation, copie ou distribution de ce
message est strictement interdite. Vous etes donc prie d’en informer
immediatement l’expediteur et de detruire ce message, ainsi que les documents
qui y sont joints, le
request a hard-copy version.
Ce courriel provient de Patrick.Lessard@xxxxxxxxxx . Pour assurer la livraison de futurs envois, veuillez inclure la presente adresse courriel a votre carnet
d’adresses ou votre liste d’expediteurs autorises.
Si vous ne souhaitez plus recevoir de messages promotionnels de la part de Cogeco, veuillez transférer ce courriel a desabonnement@xxxxxxxxxx. Merci!
Politique en matiere de protection des renseignements personnels de Cogeco et Engagement en matiere d’anti-spam – Contactez-nous
Cogeco Cable Canada, 5 Place Ville-Marie, Bureau 1700, Montreal, Quebec, H3B 0B3
--
This email is from Patrick.Lessard@xxxxxxxxxx . To ensure the delivery of future emails, please add the current email address to your address book or safe senders list.
If you no longer wish to receive promotional emails from Cogeco, please forward this message to unsubscribe@xxxxxxxxxx. Thank you!
Privacy Policy and Anti-spam Commitment - Contact us
Cogeco Cable Canada, 5 Place Ville-Marie, Suite 1700, Montreal, Quebec, H3B 0B3
________________________________________
De : Dmitry Torokhov [dmitry.torokhov@xxxxxxxxx]
Envoyé : 20 juin 2016 14:21
À : Patrick Lessard; kt.liao@xxxxxxxxxx
Cc : linux-input@xxxxxxxxxxxxxxx
Objet : Re: New Elantech touchpad version?
Hi Patrick,
On Mon, Jun 20, 2016 at 06:10:41PM +0000, Patrick Lessard wrote:
> Hello!
>
> First, thank you very much for creating a driver for the Elantech touchpads. Very appeciated.
>
> I’ve contacted one of the original author, Arjan Opmeer and he directed me to you. (those 2 email addresses actually)
>
> I just bought a HP Pavilion 14-ab057ca laptop and I think it uses a new version of the Elantech touchpad.
>
> On linux, when I boot, I get this message :
>
> « psmouse serio1: elantech: unknown hardware version, aborting... »
>
> By looking at the kernel driver code in elantech.c below, I might try a quick patch and add a case higher than 14, compile and try it.
Looking at HP site it seems that at least some batches of this laptop
are shipping with Elan touchpads.
KT, do we need to expand the list in elantech_set_properties() to
include more IC types?
>
> If succesfull, could you include a fix in the near future?
>
> Are you still maintaining this driver?
>
> How should I proceed?
>
> Thank you very much for your attention.
>
> Patrick.
>
>
> Code snippet :
>
> ….
>
> static int elantech_set_properties(struct elantech_data *etd)
> {
> /* This represents the version of IC body. */
> int ver = (etd->fw_version & 0x0f0000) >> 16;
>
> /* Early version of Elan touchpads doesn't obey the rule. */
> if (etd->fw_version < 0x020030 || etd->fw_version == 0x020600)
> etd->hw_version = 1;
> else {
> switch (ver) {
> case 2:
> case 4:
> etd->hw_version = 2;
> break;
> case 5:
> etd->hw_version = 3;
> break;
> case 6:
> case 7:
> case 8:
> case 9:
> case 10:
> case 13:
> case 14:
> etd->hw_version = 4;
> break;
> default:
> return -1;
> }
> }
>
Thanks.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
