FS#7379 - Rockboy sound patch

Attached to Project: Rockbox
Opened by Karl Kurbjun (kkurbjun) - Saturday, 30 June 2007, 20:29 GMT
Last edited by Karl Kurbjun (kkurbjun) - Saturday, 21 July 2007, 04:04 GMT
Task Type Patches
Category Games
Status Closed
Assigned To Karl Kurbjun (kkurbjun)
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No


This is a patch for rockboy that attempts to clean up the sound by adding in sound code from VisualBoy. If you test this please let me know the results including the player used and the game played.
This task depends upon

Closed by  Karl Kurbjun (kkurbjun)
Saturday, 21 July 2007, 04:04 GMT
Reason for closing:  Accepted
Additional comments about closing:  Patch committed to SVN - sound will be different on players that cannot emulate full speed (sound buffers are not improperly repeated any more)
Comment by Karl Kurbjun (kkurbjun) - Saturday, 30 June 2007, 20:31 GMT
Forgot the file
Comment by Alexander Spyridakis (xaviergr) - Saturday, 30 June 2007, 23:17 GMT
Plays excellent on my F40. Tremendous difference from the current one in SVN.
Choppy on H300 but that's because it can't get more than 50+ fps.
Comment by Benjamin Lebsanft (Benjamin_L) - Sunday, 01 July 2007, 11:25 GMT
Still choppy on the sansa but better than before.
Comment by Peter D'Hoye (petur) - Friday, 13 July 2007, 17:18 GMT
/me wonders why this isn't making it into svn...
Comment by nick (deathdr0ne) - Friday, 13 July 2007, 21:33 GMT
will test on ipod 64mb "60GB" model, and get back to you.
Comment by Travis Tooke (tdtooke) - Friday, 20 July 2007, 23:16 GMT
I've tested this on my iPod 30g 5.5g and I'm not sure if it has improved the sound, but it's still definitely worse than before they commited that rockboy sound change recently. I'm toying with the idea of taking this patch and merging it with another that basically throws alotta ifdefs in so some of the changes recently committed are undone if the target is an iPod. I'm quite some ways from actually being able to do that, since that would involve tracking down exactly what change killed the sound on the iPod, but it's a thought.
Comment by Karl Kurbjun (kkurbjun) - Saturday, 21 July 2007, 02:27 GMT
Here is a newer version of the patch, this is an almost complete rewrite of the sound core based on visualboy advance.