[U-Boot-Users] [PATCH] Add support for Amstrad Delta (E3).

Wolfgang Denk
2006-10-08T20:34:39Z

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

--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A star captain's most solemn oath is that he will give his life, even
his entire crew, rather than violate the Prime Directive.
-- Kirk, "The Omega Glory", stardate unknown