Hi Pete,
On Saturday 24 March 2007 03:16, Pete Zaitcev wrote:
> - } else if (!mousedev->touch)
> + } else if (!mousedev->touch) {
> mousedev->touch = jiffies;
> + /*
> + * Seed the ring with new position where finger has just
> landed.
> + *
> + * Without this, a touchpad may report an unchanged position,
> + * then a sync. The input_event() eats the position report,
> but
> + * lets the sync through. We increment pkt_count and leave
> + * a stale position in the ring. If a future reference to
> fx(2)
> + * hits the stale position, a large dx is reported, and the
> + * pointer warps across the screen.
> + */
> + dev = mousedev->handle.dev;
> + fx(0) = dev->abs[ABS_X];
> + fy(0) = dev->abs[ABS_Y];
>
I do not like input hanlders poking into input devices... Can't we just
reset pkt_count at the beginning of the touch to get rid of stale data?
--
Dmitry
