- Status Closed
- Percent Complete
- Task Type Patches
- Category Drivers
-
Assigned To
kugel. - Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Version 3.2
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10120 - fuze/e200v2: poll the wheel more often
This patch uses the tick_task timer to poll the scrollwheel every 5ms (the tick tasks stay at every 10ms of course).
This makes the scrollwheel on my fuze pretty much perfectly usable.
I’d like to hear opinions (from e200v2 owners too). It’s a bit of a hack, but I’m fairly sure it cannot be done better (without an interrupt, that is), so I hope it’s acceptable.
Closed by kugel.
2009-04-13 10:29
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-04-13 10:29
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Committed in r20699.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Works very good on my e250v2. A little cleanup for button-e200v2.c: scrollwheel(); is not necessary in button_read_device() because its called already through button_dbop() (which is called by button_read_device() ).
Cool. And yes, you're correct :)
Tried this out and it works nicely on my Fuze.
No real functional change, just a bit of cleanup. And also add the comments for the dbp read function to the fuze driver.
Sadly, I cannot test anymore due to the lack of hardware. But if it works well, feel free to commit.
I can't compile it, get some errors: http://pastebin.com/d3bb99831
There was a stupid typo :/
ok, now it's working again
Works fine on my e280v2. I dropped in the fuze scrollcode also and the scrolling action was perfectly acceptable. It wraps the menus always, which I personally tend to dislike, but the scrolling action was just fine. I did not need to change any parameters in the fuze scrollcode. It worked as is with only minor modifications to variable declarations so as not to redeclare.
You also needed to change "++count >= 4" to "++count >= 2", right?