[U-Boot] [PATCH] at91: add support for CDU9G25 board

Wolfgang Denk wd at denx.de
Fri Sep 6 14:54:16 CEST 2013


Dear Jiri Prchal,

In message <1378465417-18082-1-git-send-email-jiri.prchal at aksignal.cz> you wrote:
> This patch adds support for our companies board CDU9G25 with Atmel AT91SAM9G25, 128MB DDR2, 256MB NAND.
> 
> Signed-off-by: Jiri Prchal <jiri.prchal at aksignal.cz>
> ---
>  board/cdu9g25/Makefile    |   48 ++++++++++
>  board/cdu9g25/cdu9g25.c   |  129 +++++++++++++++++++++++++++
>  board/cdu9g25/config.mk   |    1 +
>  boards.cfg                |    1 +
>  include/configs/cdu9g25.h |  217 +++++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 396 insertions(+)
>  create mode 100644 board/cdu9g25/Makefile
>  create mode 100644 board/cdu9g25/cdu9g25.c
>  create mode 100644 board/cdu9g25/config.mk
>  create mode 100644 include/configs/cdu9g25.h

Entry to MAINTAINERS file missing.

Also, there is a large number of checkpatch warnings that need to be
fixed:

	WARNING: please, no spaces at the start of a line
	WARNING: space prohibited between function name and open parenthesis '('
	ERROR: do not use C99 // comments
	WARNING: line over 80 characters


> diff --git a/include/configs/cdu9g25.h b/include/configs/cdu9g25.h
> new file mode 100644
> index 0000000..f8f8008
> --- /dev/null
> +++ b/include/configs/cdu9g25.h
> @@ -0,0 +1,217 @@
> +/*
> + * (C) Copyright 2007-2008
> + * Stelian Pop <stelian at popies.net>
> + * Lead Tech Design <www.leadtechdesign.com>
> + *
> + * Configuation settings for the CDUNG boards.
> + *
> + * 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

Please use SPDX ID string here.

> +#define CONFIG_HW_WATCHDOG		1
> +#define CONFIG_AT91SAM9_WATCHDOG	1

Please remove values from all logical variables.

> +#define CONFIG_BOOTARGS			"console=ttyS0,115200 ubi.mtd=root root=ubi0:root rootfstype=ubifs rw g_ether.dev_addr=02:04:25:aa:55:5e g_ether.host_addr=02:04:25:aa:55:5f"
> +#define CONFIG_ETHADDR			02:04:de:ad:be:ef
> +#define CONFIG_IPADDR			10.0.1.2
> +#define CONFIG_SERVERIP			10.0.1.1

NAK. We don't allow any such kind of static network initialization
which would be common for all boards. Please remove all of these.

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
"There was no difference between  the  behavior  of  a  god  and  the
operations of pure chance..."   - Thomas Pynchon, _Gravity's Rainbow_


More information about the U-Boot mailing list