Rockbox

  • Status Unconfirmed
  • Percent Complete
    0%
  • Task Type Patches
  • Category Codecs
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by gama - 2011-03-01

FS#11978 - New Atari SC68/SNDH codec based on the SC68 library by Benjamin Gerard

Hi, this is a new codec based on the SC68 library
by Benjamin Gerard (sc68.atari.org).

Revision: 29484
Tested on: Sansa Fuze v2

Features:

  1. 44 Khz Stereo sound.
  2. Supports sc68 and sndh files.
  3. Support for multi track and single track files.
  4. Metadata parsing.
  5. Fade out support.

** This codec needs the sc68 Replay folder to work, just put it in the root of
your player. you can find one here: http://aminet.net/package.php?package=mus/play/sc68.lha

Notes:

1.  This is a beta codec, some (big) songs may cause rockbox to crash.
2.  When compiling the sc68 library i had a lot of dependencies errors with
     missing includes.  It is compiling fine now on the vmware image, but not in cygwin.
     I still have to find the problem with that.
3.  Some songs doesn't work, specially some big sndh files.  I'm still working on it.
4.  Seek support is broken, i'm working on it.

Please feel free to make any comment or suggestion.

complies fine here, cygwin with gcc 4.4.4 eabi, cowon d2,
ps: codecs.h/c still need to be modified for compiling in non-recording targets

Just some thoughts about not playing big files: as i understand this program emulates minimal amiga\atari ST hardware then it loads player code and music to emulated RAM (i guess it's 512kb), then maybe it's not enough ram to load big tunes? or this is only rockbox problem? (couldn't get this plyer to work)

Finally got this one to compile :)
Gamma, please note that sc68 is multiformat not only for atari st(e) but for amiga too
attached edited codecs.h\c which works with r29533

gama commented on 2011-03-07 21:45
Gamma, please note that sc68 is multiformat ..

I'm a little counfused, what do you mean by multiformat?

And yeah this codec also needs Joshua's patch to compile in all targets,
(the same changes you posted), i just don't have the time to update it right now.

Umm I mean it plays tunes made for Atari ST (emulating YM2149F chip) and for Amiga (emulating Paula chip).
Attached tune made for amiga.

Also i noticed that i cant select last subtrack, it jumps to next file

Noiced now that I can't compile rockbox as application for windows (SDL target, not simulator)
error: make: *** No rule to make target `/home/tails_/rockbox_clean/sdlbuild/emu68/type68.h', needed by `/home/tails_/rockbox_clean/sdlbuild/apps/codecs/libsc68/emu68/emu68.o'. Stop.
Is that only my problem? :S

gama commented on 2011-03-09 13:48
Noiced now that I can't compile rockbox as application for windows (SDL target, not simulator)

I pointed that out in note2, it seems it is related to the use of subfolders in libsc68, and some files include
headers from other folders,

I don't want to put all files in the one folder but if there is no other option i guess that would be an ugly way to fix it,
and we would still have to change many include directives manually.

fix a compilation error in mips targets, put the file to libsc68 directory:)

gama commented on 2011-03-21 04:13

Thanks Joshua. That must be affecting the gme codecs too (?). By the way i will spend some time on this codec once i finish
the VGM format from game_music_emu, please let me know if there are any other problems with it.

this file just affect the sc68 codec, other gme codecs are compiled without any problem.

Hmm how could i change default playtime value? editing two lines in sc68.c changes displayed time but song still stops after 3 minutes

Tip: If you change max amount of memory avaible for codecs big sndh files plays fine

Looking through this patch, its actually licensed GPLv3 rather then the usual GPLv2. Rockbox is licensed under GPLv2 or later, meaning you can use GPLv3 code with it.

That said, we've never committed GPLv3 code before, and it would be nice to retain GPLv2 compatibility. IMO if this is going to be committed I think it should be entirely enclosed inside #ifdef GPLV3 blocks, so that if someone needs GPLv2 binaries, they can easily disable the v3 bits.

gama commented on 2011-06-09 23:35

Here is the updated codec that works with the new playback engine.

Changelog:

- Remove all metadata handling from the codec.
- Removed unnecesary files from patch
- Fixed some track length issue.

gama commented on 2011-06-10 13:44

Please update the files apps/codecs/libsc68/config.h and apps/codecs/libsc68/api68.c with these ones:

- Fixed some strncpy bug

gama commented on 2011-06-10 13:46

the files ;)

   api68.c (47.7 KiB)
   config.h (1.1 KiB)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing