[U-Boot] [PATCH 00/10] Support for LogicPD i.MX27-LITEKIT development board

Wolfgang Denk wd at denx.de
Wed May 6 23:26:18 CEST 2009


Dear Ilya,

In message <1241634633-13917-1-git-send-email-yanok at emcraft.com> you wrote:
> This patch set adds support for LogicPD i.MX27-LITEKIT development board.
> It contains support for i.MX27 CPU, support for serial console, FEC
> ethernet controller, NFC NAND controller and SDHC controller. It also
> contains some fixes to generic MMC subsystem.
> 
> Signed-off-by: Ilya Yanok <yanok at emcraft.com>

General comments in addition to what the other posters already
remarked:

The whole patch set needs clean up regarding coding style: line
length, indentation and vertical alignment by TABs, brace style, no
C++ comments, incorrect multiline comment style, etc.

Also, large parts of the code are based on accessing device registers
through pointers, usualy hiddewn in some macros  using  base  address
and  offset. We don't accept such code any more. Please use C structs
and I/O accessor calls.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"...and the fully armed nuclear warheads, are, of  course,  merely  a
courtesy detail."


More information about the U-Boot mailing list