On 9/26/07, Soeren Sonnenburg [off-list ref] wrote:

On Mon, 2007-09-24 at 15:22 +0200, Thomas Rohwer wrote:

quoted

Hello,

quoted

could you please re-send the patch? I for some reason have yet to see
it ...

here it is again, adressing also the comments from Dmitry.

Thomas, Matthew and Dmitry,

I think there is another bug in this. I mean whenever a mouse button is
pressed or the mouse is moved the counter should be reset - no?

Currently the idle counter is just increased...

I mean shouldn't it be

if (x || y || key)
       dev->idlecount=0;

if (!x && !y && !key)
{
       dev->idlecount++;
       if (dev->idlecount == 10) {
               dev->valid = 0;
               schedule_work(&dev->work);
       }
}

Yes, I think you are right. I guess one could trigger an extra reset
by pressing the button repeatedly witout touching the pad. But because
we won't do reset while the button is pressed we'll never lose release
event.

I guess we want to fix it but it can wait for 2.6.24. A patch woudl be
appreciated.

Thanks!

-- 
Dmitry