Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Infrastructure → Build environment
  • 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 linuxstb - 2006-04-24
Last edited by petur - 2007-08-13

FS#5226 - Remove icode and long-calls from PP502x iPod builds

The attached patch removes all code from IRAM on the PP502x ipods (but leaves it there for the 3g) and removes the -mlong-calls GCC option. This reduces the size of rockbox.ipod from 376692 bytes to 323704 bytes.

Some brief testing with an MP3 file on my 4G Color ipod gave about 74% boost ratio with CVS Rockbox, 75% boost with this patch, and about 90% boost if I disabled the use of IRAM completely, but more thorough testing would be useful before deciding if this is to be committed to CVS.

   ipod_icode.diff (3.9 KiB)
Closed by  petur
2007-08-13 21:24
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

<linuxstb> I think amiconn's fixes (which IIUC have removed most long calls) probably make it obsolete.

ep0ch commented on 2006-04-28 20:30

FLAC with EQ on, gained just 1% boost with this patch.
MOD (without EQ) also gained 1% with patched applied.

running on 5g

mickgoanna commented on 2006-05-11 06:09

works for my nano

decayed.cell commented on 2007-01-14 10:16

Needs updating if this still works… or is it in SVN already?

barrywardell commented on 2007-07-31 12:16

This patch is out of date, but I just tested the attached updated version on my H10 with a test FLAC file. With the code in IRAM, i get 393.20% realtime. Moving the code out of IRAM using the attached patch, I get 392.25% realtime.

   iram.patch (0.8 KiB)
petur commented on 2007-08-05 00:18

Dave, are you going to bring up to date and commit?

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing