FS#9679 : c200v2: prepare Rockbox for it

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
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?).

mkamsboot.vc.diff (6.2 KiB)

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.

Comments (4)
Related Tasks (0/0)

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.

c200v2-target-tree.diff (31.1 KiB)

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

mkamsboot.vc.diff (6.5 KiB)

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).

c200v2-target-tree.diff (31.4 KiB)

	Tasks related to this task (0)

Duplicate tasks of this task (0)

Rockbox

FS#9679 - c200v2: prepare Rockbox for it

Details

Loading...