FS#6678 - Improved h10 scrollpad

Attached to Project: Rockbox
Opened by David Search (Damium) - Tuesday, 20 February 2007, 03:19 GMT
Last edited by Daniel Stenberg (bagder) - Monday, 19 November 2007, 22:55 GMT
Task Type Patches
Category Drivers
Status Closed
Assigned To No-one
Operating System iriver H10
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No


This causes the following logic to be used on the (quite chaotic) h10 scrollpad.

Check to see if a scroll button was being pressed last time we looked.
Scroll the same direction if it is within extended bounds (0x150-0x3FF is UP, 0x0-0x250 is DOWN).
If a button was not being pressed start scrolling if we are within the limited bounds (0x250-0x3FF is UP, 0x0-0x150 is DOWN).

ADC Scrollpad hardware reading:
0x0 0x150 0x250 0x3FF

I have tested this on my h10 20g. The "scrollback" effect if you were not pressing on the very top or bottom of the scrollpad is much less frequent with this logic.
This task depends upon

Closed by  Daniel Stenberg (bagder)
Monday, 19 November 2007, 22:55 GMT
Reason for closing:  Rejected
Additional comments about closing:  "Latest SVN code is much better at scrolling. Patch is not needed."
Comment by Hepdog (007quick) - Tuesday, 20 February 2007, 16:13 GMT
Seems good except that when I tried it on my h10 5g the paramiters seemed awsome for the down scroll but unfortunetly the up was not working at all! the screen would even go dark when touching the up scroll. Sorry for the bad news! unless this is not designed for the 5gb!
Comment by Barry Wardell (barrywardell) - Tuesday, 20 February 2007, 16:18 GMT
There was a small bug in that code. I've attached a working version. I'm not totally convinced that it's better than what we already have though.
Comment by David Search (Damium) - Tuesday, 20 February 2007, 16:40 GMT
That's what I get for making last moment changes before posting a patch...
Comment by Hepdog (007quick) - Wednesday, 09 May 2007, 20:40 GMT
Here is a different version which I think works better. Some of the code is an attempt to mimic the original firmware, so just bear with me. But I think that it all helps. Give it a try!
Comment by Nikkhil (AceNik) - Thursday, 10 May 2007, 09:47 GMT
here is the version that works, you forgot to include the path to file to patch just use "p0" in this 1 & it will work
Comment by Barry Wardell (barrywardell) - Wednesday, 22 August 2007, 11:42 GMT
There has been an improvement to the scrollpad in SVN recently. I don't think we still need this solution, do we?
Comment by Nikkhil (AceNik) - Wednesday, 22 August 2007, 19:05 GMT
well but even though there is an improvement in the list scan acceleration , we still need this to supplement scrolling to smoothen it, my upper end n lower end of the scrollpad still jumpy
Comment by Nikkhil (AceNik) - Wednesday, 22 August 2007, 19:26 GMT
kindly ignore my last comment thanks