From 2eefb5acb847eeb2d10bac860d37c4cef00be67b Mon Sep 17 00:00:00 2001 From: Miika Pekkarinen Date: Sun, 11 Mar 2007 10:52:36 +0000 Subject: Optimized the gui list code performance, including automatic frame dropping and cpu boosting when button events are getting queued. Improved scrollwheel acceleration code is needed to notice a real change. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@12721 a1c6a512-1295-4272-9138-f99709370657 --- firmware/drivers/button.c | 36 ++++++++++++++++++++++++++++++++++-- 1 file changed, 34 insertions(+), 2 deletions(-) (limited to 'firmware/drivers/button.c') diff --git a/firmware/drivers/button.c b/firmware/drivers/button.c index a1fd58c..7c5858a 100644 --- a/firmware/drivers/button.c +++ b/firmware/drivers/button.c @@ -26,6 +26,7 @@ #include "system.h" #include "button.h" #include "kernel.h" +#include "thread.h" #include "backlight.h" #include "serial.h" #include "power.h" @@ -255,21 +256,52 @@ static void button_tick(void) lastbtn = btn & ~(BUTTON_REL | BUTTON_REPEAT); } +void button_boost(bool state) +{ + static bool boosted = false; + + if (state && !boosted) + { + cpu_boost(true); + boosted = true; + } + else if (!state && boosted) + { + cpu_boost(false); + boosted = false; + } +} + long button_get(bool block) { struct event ev; - - if ( block || !queue_empty(&button_queue) ) + int pending_count = queue_count(&button_queue); + + /* Control the CPU boost trying to keep queue empty. */ + if (pending_count == 0) + button_boost(false); + else if (pending_count > 2) + button_boost(true); + + if ( block || pending_count ) { queue_wait(&button_queue, &ev); return ev.id; } + return BUTTON_NONE; } long button_get_w_tmo(int ticks) { struct event ev; + + /* Be sure to keep boosted state. */ + if (!queue_empty(&button_queue)) + return button_get(true); + + button_boost(false); + queue_wait_w_tmo(&button_queue, &ev, ticks); return (ev.id != SYS_TIMEOUT)? ev.id: BUTTON_NONE; } -- cgit v1.1