Rockbox

Tasklist

FS#10253 - mkamsboot version 1.0

Attached to Project: Rockbox
Opened by Rafaël Carré (funman) - Thursday, 28 May 2009, 00:25 GMT
Last edited by Rafaël Carré (funman) - Thursday, 28 May 2009, 18:27 GMT
Task Type Patches
Category Bootloader
Status Closed
Assigned To Dave Chapman (linuxstb)
Operating System Another
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Next release
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

cosmetics:
indent dualboot.S properly
remove trailing spaces

add some comments in dualboot.S
correct some comments in mkamsboot.c

move back extract_fw.c to utils/AMS/hacking

modify bin2c.c to produce only one pair of .c/.h files with several files embedded in it

move files needing to be built by an ARM cross compiler into dualboot/

commit produced .c/.h files (containing nrv2e_d8.S and dualboot.S built for Clip, Fuze, e200v2, c200v2, m200v4)

Build tested on linux i386, OSX intel, mingw cross compiler hosted on linux.

Produces the same bootloaders (bitexact) than current SVN.

Bump version to 1.0

Real README


I am waiting for approval from the mkamsboot author linuxstb ;)
This task depends upon

Closed by  Rafaël Carré (funman)
Thursday, 28 May 2009, 18:27 GMT
Reason for closing:  Accepted
Comment by Rafaël Carré (funman) - Thursday, 28 May 2009, 11:24 GMT
I'll also merge the mkamsboot part of  FS#10185 
Comment by Rafaël Carré (funman) - Thursday, 28 May 2009, 13:25 GMT
Make mkamsboot a library (work by domonoky)

Limit lines to 80 characters.
Use a macro with variadic arguments for error cases in functions which might error.
Add detailed descriptions to functions exported by the library (in the header file)

EDIT : the whole patch didn't get in sorry
Comment by Rafaël Carré (funman) - Thursday, 28 May 2009, 17:02 GMT
Add Clipv2 , update README

Some more cosmetics in dualboot.S

Tested on my Clipv1, Clipv2, Fuzev1

linuxstb, on my side it's ok for commit; i would just like to know if you have comments (coding style for example)

Loading...