Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#10495 - More stable MP3 playing for Sansa Clip v1

Attached to Project: Rockbox
Opened by Matthias Schneider (matsch) - Saturday, 08 August 2009, 00:24 GMT+2
Last edited by Rafaël Carré (funman) - Sunday, 11 October 2009, 15:52 GMT+2
Task Type Patches
Category Music playback
Status Closed
Assigned To No-one
Player Type Another
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

This patch reserves the full IRAM for the Codec.
Playing 196kbit/s VBS MP3s is more stable with this patch.
I do not know the reason for this. Either there is interference between CODEC and other loaded memory segments in IRAM or the Codec just needs more space.
Influence on plugins and Fuze is not tested.
   stable_MP3_clip.patch (1.1 KiB) 
 firmware/export/config-clip.h      |    2 +-
 firmware/target/arm/as3525/app.lds |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

This task depends upon

Closed by  Rafaël Carré (funman)
Sunday, 11 October 2009, 15:52 GMT+2
Reason for closing:  Invalid
Additional comments about closing:  a better fix is in fs#10605
Comment by MichaelGiacomelli (saratoga) - Saturday, 08 August 2009, 01:13 GMT+2
Moving around things in memory does seem to make playback better or worse on the clip, presumably because some area of memory is overwritten and shuffling like this increases the odds that nothing important is corrupted. However I do not think this is an actual fix.

Regarding memory, MP3 needs about 110KB of RAM:

http://www.rockbox.org/twiki/bin/view/Main/CodecMemoryUsage

Since MP3 does not use malloc, lack of memory will cause a compiler error, but cannot cause runtime problems.

Loading...