[U-Boot] 83xx: add support for ve8313 board

Kim Phillips kim.phillips at freescale.com
Wed Jul 7 01:24:03 CEST 2010


On Mon, 5 Jul 2010 12:23:10 +0200
Heiko Schocher <hs at denx.de> wrote:

> ---
>  board/ve8313/Makefile    |   50 +++++
>  board/ve8313/config.mk   |   10 +
>  board/ve8313/ve8313.c    |  212 ++++++++++++++++++
>  boards.cfg               |    1 +
>  include/configs/ve8313.h |  534 ++++++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 807 insertions(+), 0 deletions(-)

missing MAKEALL, MAINTAINERS entries, and, if there's anything unique
about this board, a doc/README.ve8313.

> diff --git a/board/ve8313/config.mk b/board/ve8313/config.mk

> +ifndef TEXT_BASE
> +#TEXT_BASE = 0x100000
> +TEXT_BASE = 0xfe000000
> +endif
> +
> +#PLATFORM_CPPFLAGS += -DDEBUG

please clean this up - remove commented out code.

> diff --git a/board/ve8313/ve8313.c b/board/ve8313/ve8313.c
> new file mode 100644
> index 0000000..b13d1f3
> --- /dev/null
> +++ b/board/ve8313/ve8313.c
> @@ -0,0 +1,212 @@
> +/*
> + * (C) Copyright 2010
> + * Heiko Schocher, DENX Software Engineering, hs at denx.de.

this file contains code that looks like it was copied from files with
Freescale copyrights; why are you deleting Freescale's copyrights?

> +#if defined(CONFIG_OF_LIBFDT)
> +#include <libfdt.h>
> +#endif

I realise this was copied in, but we don't need the ifdef anymore.

> +/* Fixed sdram init -- doesn't use serial presence detect.
> + *
> + * This is useful for faster booting in configs where the RAM is unlikely
> + * to be changed, or for things like NAND booting where space is tight.
> + */

Is the memory soldered on to the board?  If not, does the serial
presence detect code not work for you?

> +#if defined(CONFIG_HW_WATCHDOG)
> +	clrbits_be32(&gpio->dat, (VE8313_WDT_EN | VE8313_WDT_TRIG));
> +	/* set WDT pins as output */
> +	setbits_be32(&gpio->dir, (VE8313_WDT_EN | VE8313_WDT_TRIG));
> +#else
> +	/* disable WDT */
> +	setbits_be32(&gpio->dat, (VE8313_WDT_EN | VE8313_WDT_TRIG));

inner parens not necessary, also the last WDT_EN bit setting doesn't
match the comment above it (ENable vs. disable) - perhaps nothing
need be done if HW_WATCHDOG is not set?

> +	setbits_be32(&gpio->dat, (VE8313_WDT_TRIG));
> +	clrbits_be32(&gpio->dat, (VE8313_WDT_TRIG));

inner parens not necessary.

> +#undef CONFIG_HW_WATCHDOG

no #undefs please.

> +#define CONFIG_SYS_LBC_LBCR	(0x00040000)

parens not necessary.

> +#define CONFIG_SYS_PCI_SUBSYS_VENDORID 0x1057	/* Motorola */

It's Freescale: 0x1957.

Thanks,

Kim


More information about the U-Boot mailing list