[U-Boot] [PATCH] Marvell Sheevaplug Board support

[Stefan Roese <sr@denx.de>]

Hi Prafulla,

On Thursday 21 May 2009 22:24:30 Prafulla Wadaskar wrote:
> References: http://plugcomputer.org/
> Serial console Setup
> http://openplug.org/plugwiki/index.php/Serial_terminal_program#Linux
> OpenOCD Setup
> http://openplug.org/plugwiki/index.php/Setting_Up_OpenOCD_Under_Linux
>
> This patch is tested for-
> 1. Boot from DRAM/NAND flash
> 2. File transfer using tftp
> 3. NAND flash read/write/erase
>
> Signed-off-by: Prafulla Wadaskar <prafulla@marvell.com>
> ---
>  MAKEALL                               |    1 +
>  Makefile                              |    8 +-
>  board/Marvell/sheevaplug/Makefile     |   52 +++++++++
>  board/Marvell/sheevaplug/config.mk    |   25 +++++
>  board/Marvell/sheevaplug/nand.c       |   80 ++++++++++++++
>  board/Marvell/sheevaplug/sheevaplug.c |  135 +++++++++++++++++++++++
>  board/Marvell/sheevaplug/sheevaplug.h |   37 +++++++
>  include/configs/sheevaplug.h          |  191
> +++++++++++++++++++++++++++++++++ 8 files changed, 526 insertions(+), 3
> deletions(-)
>  create mode 100644 board/Marvell/sheevaplug/Makefile
>  create mode 100644 board/Marvell/sheevaplug/config.mk
>  create mode 100644 board/Marvell/sheevaplug/nand.c
>  create mode 100644 board/Marvell/sheevaplug/sheevaplug.c
>  create mode 100644 board/Marvell/sheevaplug/sheevaplug.h
>  create mode 100644 include/configs/sheevaplug.h

You didn't add a maintainer for this new board to MAINTAINERS. Please add this 
in your next patch version. And for the MV88F6281GTW_GE board as well.

> diff --git a/MAKEALL b/MAKEALL
> index 6719d7b..e3d33cb 100755
> --- a/MAKEALL
> +++ b/MAKEALL
> @@ -144,6 +144,7 @@ LIST_8xx="		\
>  	RPXlite		\
>  	RPXlite_DW	\
>  	RRvision	\
> +	sheevaplug	\
>  	SM850		\
>  	spc1920		\
>  	SPD823TS	\
> diff --git a/Makefile b/Makefile
> index 8144ecd..34a863b 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1122,6 +1122,9 @@ RRvision_LCD_config:	unconfig
>  	@echo "#define CONFIG_SHARP_LQ104V7DS01" >>$(obj)include/config.h
>  	@$(MKCONFIG) -a RRvision ppc mpc8xx RRvision
>
> +sheevaplug_config: unconfig
> +	@$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) Marvell kirkwood
> +
>  SM850_config	:	unconfig
>  	@$(MKCONFIG) $(@:_config=) ppc mpc8xx tqm8xx tqc
>
> @@ -2808,9 +2811,6 @@ lpd7a400_config \
>  lpd7a404_config:	unconfig
>  	@$(MKCONFIG) $(@:_config=) arm lh7a40x lpd7a40x
>
> -mv88f6281gtw_ge_config: unconfig
> -	@$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) Marvell kirkwood
> -

Why do you remove this board here?

>  mx1ads_config	:	unconfig
>  	@$(MKCONFIG) $(@:_config=) arm arm920t mx1ads NULL imx
>
> @@ -3110,6 +3110,8 @@ omap2420h4_config	: unconfig
>  qong_config		: unconfig
>  	@$(MKCONFIG) $(@:_config=) arm arm1136 qong davedenx mx31
>
> +sheevapluge_config: unconfig
> +	@$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) Marvell kirkwood


"sheevapluge" here and "sheevaplug" a few lines above? This probably just 
slipped in.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================