[U-Boot] [PATCH 1/1] Add support for Eukrea CPUAT91 SBC

[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]

On 15:24 Thu 16 Jul     , Eric Benard wrote:
> CPUAT91 is built around Atmel's AT91RM9200 and has up to 16MB of NOR
> flash, up to 128MB of SDRAM, and includes a Micrel KS8721 PHY in RMII
> mode.
> 
> Signed-off-by: Eric Benard <eric@eukrea.com>
> ---
>  MAINTAINERS                     |    6 +-
>  MAKEALL                         |    3 +-
>  Makefile                        |   16 +++
>  board/eukrea/cpuat91/Makefile   |   50 ++++++++
>  board/eukrea/cpuat91/config.mk  |    1 +
>  board/eukrea/cpuat91/cpuat91.c  |   85 ++++++++++++++
>  cpu/arm920t/at91rm9200/Makefile |    5 +-
>  cpu/arm920t/at91rm9200/ks8721.c |  244 +++++++++++++++++++++++++++++++++++++++
>  include/configs/cpuat91.h       |  231 ++++++++++++++++++++++++++++++++++++
>  include/ks8721.h                |   78 +++++++++++++
>  10 files changed, 715 insertions(+), 4 deletions(-)
>  create mode 100644 board/eukrea/cpuat91/Makefile
>  create mode 100644 board/eukrea/cpuat91/config.mk
>  create mode 100644 board/eukrea/cpuat91/cpuat91.c
>  create mode 100644 cpu/arm920t/at91rm9200/ks8721.c
>  create mode 100644 include/configs/cpuat91.h
>  create mode 100644 include/ks8721.h
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 575a7ec..da64571 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1,4 +1,4 @@
> -#########################################################################
> +9#########################################################################
>  #									#
>  # Regular Maintainers for U-Boot board support:				#
>  #									#
> @@ -515,6 +515,10 @@ Dirk Behme <dirk.behme@gmail.com>
>  
>  	omap3_beagle	ARM CORTEX-A8 (OMAP3530 SoC)
>  
> +Eric Benard <eric@eukrea.com>
> +
> +	cpcuat91	ARM920T
> +
>  Rishi Bhattacharya <rishi@ti.com>
>  
>  	omap5912osk	ARM926EJS
> diff --git a/MAKEALL b/MAKEALL
> index 020ff73..0e0b657 100755
> --- a/MAKEALL
> +++ b/MAKEALL
> @@ -587,11 +587,12 @@ LIST_at91="			\
>  	at91sam9260ek		\
>  	at91sam9261ek		\
>  	at91sam9263ek		\
> -	at91sam9g10ek	\
> +	at91sam9g10ek		\
>  	at91sam9g20ek		\
>  	at91sam9m10g45ek	\
>  	at91sam9rlek		\
>  	cmc_pu2			\
> +	cpuat91			\
>  	csb637			\
>  	kb9202			\
>  	meesc			\
> diff --git a/Makefile b/Makefile
> index 090e645..02efe1b 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2683,6 +2683,22 @@ at91rm9200ek_config	:	unconfig
>  cmc_pu2_config	:	unconfig
>  	@$(MKCONFIG) $(@:_config=) arm arm920t cmc_pu2 NULL at91rm9200
>  
> +xtract_cpuat91 = $(subst _ram,, $(subst _config,,$1))
no need please remove
> +
> +cpuat91_ram_config \
> +cpuat91_config	:	unconfig
> +	@if [ "$(findstring _ram_,$@)" ] ; then \
> +		echo "#define CONFIG_CPUAT91_RAM 1" >> ./include/config.h ; \
what is this ram config?
what is its goal?
> +	fi;
> +	@$(MKCONFIG) -a $(call xtract_cpuat91,$@) arm arm920t cpuat91 eukrea at91rm9200
cpuat91 will be enough as you do not use other option
> +	@if [ "$(findstring _ram_,$@)" ] ; then \
> +		echo "#define CONFIG_CPUAT91_RAM 1" >> ./include/config.h ; \
> +		echo "... CPUAT91 configured for RAM !" ; \
> +	else \
> +		echo "#define CONFIG_BOOTDELAY 1" >> ./include/config.h ;\
> +		echo "... CPUAT91 configure for Flash !" ;\
> +	fi;
> +
>  csb637_config	:	unconfig
>  	@$(MKCONFIG) $(@:_config=) arm arm920t csb637 NULL at91rm9200
>  
<snip>
> diff --git a/cpu/arm920t/at91rm9200/Makefile b/cpu/arm920t/at91rm9200/Makefile
> index 73aeeac..114d8ad 100644
> --- a/cpu/arm920t/at91rm9200/Makefile
> +++ b/cpu/arm920t/at91rm9200/Makefile
> @@ -31,14 +31,15 @@ COBJS	+= bcm5221.o
>  COBJS	+= dm9161.o
>  COBJS	+= ether.o
>  COBJS	+= i2c.o
> +COBJS-$(CONFIG_KS8721_PHY)	+= ks8721.o
>  COBJS	+= lxt972.o
>  COBJS	+= reset.o
>  COBJS	+= spi.o
>  COBJS	+= timer.o
>  COBJS	+= usb.o
>  
> -SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)
> -OBJS	:= $(addprefix $(obj),$(SOBJS) $(COBJS))
> +SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c) $(COBJS-y:.o=.c)
> +OBJS	:= $(addprefix $(obj),$(SOBJS) $(COBJS) $(COBJS-y))
>  
>  all:	$(obj).depend $(LIB)
>  
> diff --git a/cpu/arm920t/at91rm9200/ks8721.c b/cpu/arm920t/at91rm9200/ks8721.c
> new file mode 100644
> index 0000000..703e58c
> --- /dev/null
> +++ b/cpu/arm920t/at91rm9200/ks8721.c
> @@ -0,0 +1,244 @@
> +/*
> + * (C) Copyright 2006
> + * Author : Eric Benard (Eukrea Electromatique)
> + * based on dm9161.c which is :
> + * (C) Copyright 2003
> + * Author : Hamid Ikdoumi (Atmel)
I'm not a fan of this implementation at all

but as I've not yet finish the rm9200 cleanup
I'll ack this ben what do you think?
ben please note that the rm9200 cleanup will introduce a phylib support
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */

Best Regards,
J.