• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Codecs
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Rainer Sinsch - 2006-04-17

FS#5157 - SID codec based on tinysid

This is the latest version of the rockbox SID codec.

Whats new since 15. April 2006
* Filter implementation(!) using integer calculations
* Audio quality improvements
* Speed improvements

Please note that the current version only suports single speed PAL playback. I will correct this as soon as I find time.

Closed by  Dave Chapman
2006-07-18 19:36
Reason for closing:  Accepted
Eli Sherer commented on 2006-04-17 12:55

Well Done…I listen to SID files every now and then…10x!

Brandon Low commented on 2006-04-17 19:29

Here’s an updated patch for latest CVS

(application/octet-stream)    sid.patch (41.9 KiB)
Rainer Sinsch commented on 2006-04-17 19:48

Here you go with version 1.2:

Added sample playback
Added cia timing routines
Added fast forwarding capabilities
Corrected bug in sid loading routine

David Brown commented on 2006-04-19 20:24

Great stuff, I have a couple of feature requests though :)

1. Is it possible to make a plugin version of this codec? My only reason for asking this is to change subsongs via a menu.
2. I’ve recently been playing with SIDPlayer and it enhances the sound loads with the stereo + spatial effect, any chance something similar can be added to this?


Dave Chapman commented on 2006-04-21 12:19

I’ve edited the patch very slightly so that it applies cleanly against current CVS (21 April).

Paul Louden commented on 2006-04-23 12:13

CODEC_DSP_ENABLE is no longer valid. Fixed to compile against CVS.

Marco Alanen commented on 2006-04-30 17:01

Subsong switching can be done by setting the song length to <num subsongs> * 1000 and then switch subsongs on seeking.
I’m currently playing around with another codec, and that’s how I’ve solved it there. Seems to work.

Rainer Sinsch commented on 2006-05-04 19:48

This is a nice idea! Here is a patch for the subsong selection, each subsong now represents a second. As a result listened seconds are not displayed anymore - but who cares ;)

Rainer Sinsch commented on 2006-05-04 19:52

Just to make sure everyone understands:

If you wish to seek to subsong #0: Rewind to playing time: 0:00
If you wish to seek to subsong #1: Seek to playing time: 0:01
If you wish to seek to subsong #2: Seek to playing time: 0:02

and so on…

The real playing time is not counted anymore so the display directly reflects the subsong number.

Rhapsody commented on 2006-05-10 06:30

Any chance of there being one patch for the SID and MOD players? I cannot patch the source for one player after the other. I did try entering the code into the appropriate source files manually but I ended up with SIDs being playable and MOD files not being recognized. Thanks.

Mathieu Stempell commented on 2006-06-17 16:27

Could this be implemented in main builds?

Falk commented on 2006-07-07 13:10

Great work, Rainer. Just love it. Any chance to implement a timing feature to play a SID file for a certain length and than switch to the next … or better to use the songlenth.txt from the HVCS- (High Voltag Sid Collection)? This would be great.


Available keyboard shortcuts


Task Details

Task Editing