[U-Boot-Users] [PATCH] add support for Toradex Colibri module

Wolfgang Denk wd at denx.de
Mon Feb 4 00:07:48 CET 2008


Dear Daniel,

in message <20071130231335.GA10869 at buzzloop.caiaq.de> you wrote:
> 
> Patch against 1.3.0 is attached.
> 
> Best regards,
> Daniel
> 
> 
>  MAKEALL                       |    1 
>  Makefile                      |    3 
>  board/colibri/Makefile        |   51 ++++
>  board/colibri/colibri.c       |   61 +++++
>  board/colibri/config.mk       |    1 
>  board/colibri/lowlevel_init.S |  465 ++++++++++++++++++++++++++++++++++++++++++
>  board/colibri/pxavoltage.S    |  178 ++++++++++++++++
>  board/colibri/u-boot.lds      |   56 +++++
>  include/asm-arm/mach-types.h  |   13 +
>  include/configs/colibri.h     |  261 +++++++++++++++++++++++
>  10 files changed, 1090 insertions(+)

There are coding style issues with your patch: trailing white space,
C++ comments, indentation not by multiples of 8 characters.

Please clean up and resubmit.

And while you are at it - please keep lists sorted (MAKEALL,
Makefile).



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
He had been eight years upon a project for extracting sunbeams out of
cucumbers, which were to be put in vials hermetically sealed, and let
out to warm the air in raw inclement summers.        - Jonathan Swift
              _Gulliver's Travels_ ``A Voyage to Laputa, etc.'' ch. 5




More information about the U-Boot mailing list