Re: Fix sudden warps in mousedev

[Dmitry Torokhov <dtor@insightbb.com>]

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