[U-Boot] [PATCH 3/3] Add support for EDB93xx boards

Wolfgang Denk wd at denx.de
Tue Jan 19 22:58:51 CET 2010


Dear Matthias Kaehlcke,

In message <20100119191627.GM16182 at darwin> you wrote:
> Added support for the following EDB93xx boards:
> 
> EDB9301
> EDB9302
> EDB9302A
> EDB9307
> EDB9307A
> EDB93012
> EDB9315
> EDB9315A
...
> +int board_init(void)
> +{
> +	DECLARE_GLOBAL_DATA_PTR;

This is bound to fail. DECLARE_GLOBAL_DATA_PTR must be on file level,
never at function level! Fix globally!

> +	/* Machine number, as defined in linux/arch/arm/tools/mach-types */
> +	gd->bd->bi_arch_number = MACH_TYPE;

Please use CONFIG_MACH_TYPE instead (fix globally).

...
> +	/* copy exception vectors */
> +	src = (unsigned int *)_armboot_start;
> +
> +	for (i = 0; i < 16; i++)
> +		*dst++ = *src++;

Any reason for not using memcpy() ?

> diff --git a/include/configs/edb93xx.h b/include/configs/edb93xx.h
> new file mode 100644
> index 0000000..457dbb2
> --- /dev/null
> +++ b/include/configs/edb93xx.h
> @@ -0,0 +1,273 @@
> +/*
> + */

Please add valid file header.

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
Doubt is a pain too lonely to know that faith is his twin brother.
- Kahlil Gibran


More information about the U-Boot mailing list