- Status Closed
- Percent Complete
- 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 Dave Chapman - 2006-04-24
Last edited by Peter D'Hoye - 2007-08-13
Opened by Dave Chapman - 2006-04-24
Last edited by Peter D'Hoye - 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 Peter D'Hoye
2007-08-13 21:24
Reason for closing: Out of Date
Additional comments about closing:
2007-08-13 21:24
Reason for closing: Out of Date
Additional comments about closing:
<linuxstb> I think amiconn's fixes
(which IIUC have removed most long
calls) probably make it obsolete.
FLAC with EQ on, gained just 1% boost with this patch.
MOD (without EQ) also gained 1% with patched applied.
running on 5g
works for my nano
Needs updating if this still works… or is it in SVN already?
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.
Dave, are you going to bring up to date and commit?