Comment 185 for bug 606238

Revision history for this message
Vreixo Formoso (metalpain2002) wrote : Re: synaptic touchpad not recognized on dell latitude e6510 and others

Hi!

Some comments about mt and my previous comment:

- The claim about mt being received one-byte-at-a-time is just a problem with how the driver loads the packets in alps_process_byte. It is easy to change that function to let the full packet be processed by apls_process_tocuhpad_packet_v6. However, the if clause with comment /* Bytes 2 - pktsize should have 0 in the highest bit */ should be removed, as this condition does not hold for mt packets. If you remove that if condition, the hack to ignore mt packets can be moved to v6 function. I think the right thing is to add a "AND packet version != v6" to that if condition, and move the hack to our function. Given that, mt packets are received by v6 processing function as 6 bytes packets. All fine.

- Packets whose first byte is c8 (st) or ca (mt) follow the same layour. x,y, and z make sense for both c8 and ca packets. At this point I think the extra bit enabled in ca could mean multitouch, but I'am not sure.

- I have also found packets whose first byte is ec, ed, ea, and ee. They seem to follow a different layout, as values reported for x,y,z make no sense. I was wrong before.

If I found more details I will report them here.

Cheers
Vreixi