Just a remainder. The email address should be updated from motorola.com to freescale.com.

Best Regards, Leo

-----Original Message----- From: u-boot-users-bounces@lists.sourceforge.net [mailto:u-boot-users-bounces@lists.sourceforge.net] On Behalf Of Jeff

Mann

Sent: Wednesday, July 12, 2006 3:39 AM To: u-boot-users@lists.sourceforge.net; u-boot-patches@denx.de Subject: [U-Boot-Users] [patch] Added support for Embedded planet

ep85xxm

board

CHANGELOG

• Adds support for ep85xx boards from Embedded Planet. Board port originally by Xianghua Xiao X.Xiao@motorola.com. Merged and submitted by J. Mann, Embedded Planet, July, 11 2006.

Dear Jeff,

in message 1628E43D99629C46988BE46087A3FBB96077DF@ep-01.EmbeddedPlanet.local you wrote:

• Adds support for ep85xx boards from Embedded Planet. Board port originally by Xianghua Xiao X.Xiao@motorola.com. Merged and submitted by J. Mann, Embedded Planet, July, 11 2006.

I'm sorry, but I have to reject this patch, mostly because it comes in an unacceptable format. Please see the README for patch submission rules. Sending separate patches for each affected source file is simply not acceptable.

Also, please fix the following Coding Style issues before re-submitting:

* Please keep lists sorted (like board names in MAKEALL) * Remove trailing white spaces (for example: include/configs/ep85xxm.h) * Use TAB for indentation instead of spaces (for example: board/ep85xxm/init.S) * Indent by 8 character positions (TABs) (for example: include/configs/ep85xxm.h)

Also, I consider it a very bad ide to hard-wire MAC and IP addreses in the board config file. And pre-defining constants like "unknown" for hostname, "/nfsroot" for rootpath or "your.uImage" for bootfile is, well, let's say not very helpful. If there are no useful common setttings it is better NOT to set such variables at all.

Finally, I recommend to use "${...}" variable syntax in all your definitions, as this makes it possible to switch command line parsers without need to change the environment definitions.

Best regards,

Wolfgang Denk