Hi Sebastian, On Tue, Dec 15, 2009 at 10:01:33PM +0100, Sebastian Kapfer wrote: > > Hi Dmitry, > > I really hate to say, but I can still desync it. > > [1240043.147596] alps.c: handle cf > [1240043.148778] alps.c: handle 14 > [1240043.150043] alps.c: handle 1a > [1240043.157696] alps.c: handle 5f > [1240043.158851] alps.c: handle c > [1240043.160044] alps.c: handle 29 > [1240043.163885] alps.c: handle 2f > [1240043.165051] alps.c: handle 0 > [1240043.166213] alps.c: handle ff > [1240043.166218] alps.c: refusing packet[5] = ff > [1240043.166221] > [1240043.166227] psmouse.c: DualPoint TouchPad at isa0060/serio1/input0 lost sync at byte 6 > > In short, it's a 6-byte followed by a 3-byte; we mistakenly process it > as a interleaved PS/2 (oh well), and, while the final bytes come in, > happen upon high bits set in the PS/2 packet. In a way, this is correct > behaviour, because we _are_ processing the data incorrectly... In my > last patch, this went unnoticed because I didn't check for the high bit > in bytes 7,8,9. > *sigh* Yes, unfortunately in both cases we report incorrect data to userspace... Unfortunately I don't see any way to fix that; not unless somebody (ALPS?) tells us a bit more about protocol... But it is likely that this as good as it gets even for them. The good thing is that it shoudl not trigger during nrmal usage (I doubt many people work the device with both hands). > For all practical purposes (i.e. not pressing three buttons and fumbling > with both pointers at the same time), the patch works very well. > > In particular, no double clicks. :-) > OK, then I will apply it and send it on. Thank you for all your efforts. -- 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
