[U-Boot-Users] [PATCH v3] Add MIMC200 board

Wolfgang Denk wd at denx.de
Wed Jul 30 12:24:14 CEST 2008


In message <489030FA.2030908 at mimc.co.uk> you wrote:
> The MIMC200 board is based on Atmel's NGW100 dev kit, but with an extra 8MByte FLASH and 128KByte FRAM.
> ---
> 
>  CREDITS                       |    4 +
>  MAINTAINERS                   |    4 +
>  MAKEALL                       |    1 +
>  Makefile                      |    3 +
>  board/mimc/mimc200/Makefile   |   40 ++++++++
>  board/mimc/mimc200/config.mk  |    3 +
>  board/mimc/mimc200/mimc200.c  |  209 +++++++++++++++++++++++++++++++++++++++++
>  board/mimc/mimc200/u-boot.lds |   73 ++++++++++++++
>  common/console.c              |    5 +
>  include/configs/mimc200.h     |  177 ++++++++++++++++++++++++++++++++++
>  10 files changed, 519 insertions(+), 0 deletions(-)

Please splitt the changes to common/console.c into a separate patch.

And make sure to explain exactly why this is needed, and why you think
your implementation works (for example, why do you add the test only
to tstc() but not to getc() ?)

Without better explanations I reject the common/console.c part of the
patch.

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
CONSUMER NOTICE:  Because  of  the  "Uncertainty  Principle,"  It  Is
Impossible  for  the  Consumer  to  Find  Out  at  the Same Time Both
Precisely Where This Product Is and How Fast It Is Moving.




More information about the U-Boot mailing list