- Status Closed
- Percent Complete
- Task Type Patches
- Category Operating System/Drivers
- Assigned To No-one
- Operating System iPod 5G
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#11580 - Unify 32mb and 64mb ipod video builds
This quick and dirty patch makes use of the fact that on a 32MB ipod video the RAM is mirrored in the address space, so the only thing actually needed to make a 64MB build not do nasty things on a 32MB board is adjusting audiobufend.
A quick test on my 32MB 5G with a 64MB build seems to work fine. A bit more testing is needed though, especially to see if a 64MB board uses all available RAM.
Also, firmware/target/arm/thread-pp.c and firmware/drivers/ata.c seem to do something different on 64MB builds. I don’t know what should be done with those.
This *isn’t* ready for commit yet. it probably doesn’t even compile for most targets
Closed by fg
2010-08-31 19:07
Reason for closing: Accepted
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
2010-08-31 19:07
Reason for closing: Accepted
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
Committed as r27960
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Slightly improved version. Functionally exactly the same, but this one doesn’t break other builds. It also sets the default memory amount in configure to 64.
This version also handles the different current measurements in debug_menu.c and unifies the BATTERY_CAPACITY settings.
We might want to set BATTERY_CAPACITY_DEFAULT differently depending on detected RAM. I’m not sure of how to do that cleanly, and I’m not convinced we actually want that (due to boards being routinely swapped by repair centres and refurbishers). Apart from that, I think this patch is complete.
Final patch for the day: this one resolves the last remaining issue using the best solution for all thorny issues: document it in the manual.
I think this is ready for commit, assuming people don’t find unexpected problems.