Dear Jonathan,
in message 20060613165927.GE14634@earth.li you wrote:
The attached patch adds support for the Amstrad Delta (E3) Videophone. Currently the Amstrad primary bootloader provides the main hardware initialisation and then loads u-boot to handle user interaction and kernel loading.
I'd appreciate any comments you may have. Patch is against current git.
I'm afraid I have to reject his patch.
The major reason is that I don't want to see any board specific code in files like cpu/arm925t/start.S ; maybe you have a good reason for your modifiations of such a file, but then you must explain it, and try to implement it in a board-independent way.
Also, there are a few formal resons for the reject: you missed to include a CHANGELOG entry, and your files violate the Coding Style requirements (trailing white space, C++ comments, etc.).
Please clean up and resubmit (and while doing this, please also adapt the makefiles to the new style to allow for building in another directory).
Best regards,
Wolfgang Denk