From: Rafaël Carré <rafael.carre_at_gmail.com>
Date: Wed, 3 Jun 2009 16:28:43 +0200

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Le Wed, 3 Jun 2009 13:35:56 +0200,
mailer_at_svn.rockbox.org a écrit :

> Date: 2009-06-03 13:35:56 +0200 (Wed, 03 Jun 2009)
> New Revision: 21177
>
> Log Message:
> Change the timer interrupt setup so that TIMER_FREQ is changed for
> HAVE_SCROLLWHEEL (which read the scrollwheel between tick tasks too)
> instead of the generated variable. No functional change.

There is a problem in this code (not with the change):

INT_TIMER2() will be called twice per tick, and the described behaviour
is

interrupt 2*n : call_tick_tasks() ; button_read_dbop();
interrupt 2*n+1 : button_read_dbop(); /* 2 calls per tick */

But the actual behaviour is:

interrupt 2*n : call_tick_tasks() ;
interrupt 2*n+1 : button_read_dbop(); /* 1 call per tick */

So if the wheel is smooth enough, the code should be simplified and run
button_read_dbop() at the same time than call_tick_tasks(), by using
kernel.c tick_add_task()

- --
Rafaël Carré
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkomiJsACgkQYWCeGMCv8Q/U0ACfd6Y98WQBxGGYLx8823z/5/or
GPoAoN0kV3MqzhMdBQaxfJ2WHk/hSx4N
=cKgG
-----END PGP SIGNATURE-----
Received on 2009-06-03