Rockbox

Tasklist

FS#9679 - c200v2: prepare Rockbox for it

Attached to Project: Rockbox
Opened by Thomas Martitz (kugel.) - Friday, 19 December 2008, 17:08 GMT
Last edited by MichaelGiacomelli (saratoga) - Wednesday, 24 December 2008, 04:24 GMT
Task Type Patches
Category Build environment
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.0
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This task aims to add the c200v2 as a build target. It seems suitable as we have a c200v2 owner and tester (and I think he also codes).

The first patch makes mkamsboot create patched firmwares. Well, at least if you have a valid bootloader file, which isn't possible yet. Please review that one as it's critical (linuxstb?).
This task depends upon

Closed by  MichaelGiacomelli (saratoga)
Wednesday, 24 December 2008, 04:24 GMT
Reason for closing:  Accepted
Additional comments about closing:  Accepted in r19574+r19575.
Comment by Thomas Martitz (kugel.) - Friday, 19 December 2008, 17:53 GMT
This patch the c200v2 to the target tree. It'll build a bootloader which you can pass to mkamsboot.

Note: Most is guess work, except for the button driver. Especially the memory size would be valuable to know for sure, I guessed 8MB for now.

Please review this also. Thanks.
Comment by Thomas Martitz (kugel.) - Friday, 19 December 2008, 18:00 GMT
Just a little disclaimer: Both patches are completely untested. They may or may not brick your player. Please do not apply if you can't live with a brick.
Comment by Thomas Martitz (kugel.) - Tuesday, 23 December 2008, 15:41 GMT
So, this one is tested and declared safe. I only changed the right button detection to trigger on C6 == low, not high. The other is readability (#ifdef SANSA_C200V2 instead of #ifndef ..).

I guess it can be committed
Comment by Thomas Martitz (kugel.) - Tuesday, 23 December 2008, 15:57 GMT
Ok, changed the target tree patch to reflect the new findings (more butons, 2MB RAM).

Given that RockRabbit successfully built a bootloader with both patches they can go it (if there's no objections regarding the more-dummy-than-working lcd-driver).

Loading...