[U-Boot] [PATCH v6 2/2] APM821xx: Add bluestone board support
Wolfgang Denk
wd at denx.de
Tue Sep 28 21:02:30 CEST 2010
Dear tmarri at apm.com,
In message <1285698755-4972-1-git-send-email-tmarri at apm.com> you wrote:
> From: Tirumala Marri <tmarri at apm.com>
>
> Add support code for bluestone board wth APM821XX processor based.
> This patch includes early board init, misc init, configure EBC,
> initializes UIC, MAKEALL, board.cfg and MAINTAINERS file.
>
> Signed-off-by: Tirumala R Marri <tmarri at apm.com
...
> --- /dev/null
> +++ b/include/configs/bluestone.h
> @@ -0,0 +1,178 @@
> +/*
> + * bluestone.h - configuration for Blouestone (APM821XX)
s/Blouestone/Bluestone/ ?
...
> +#define SPD_EEPROM_ADDRESS {0x53, 0x51} /* SPD i2c spd addresses */
...
> +#define CONFIG_SYS_I2C_SPEED 400000 /* I2C speed */
> +#define CONFIG_SYS_I2C_MULTI_EEPROMS
> +#define CONFIG_SYS_I2C_EEPROM_ADDR (0xa8 >> 1)
...
> +#define CONFIG_4xx_CONFIG_I2C_EEPROM_ADDR 0x52
Please replace the "(0xa8 >> 1)" by a plain 0x54 like youdo for the
other I2C addresses as well. No need to make reading more difficult
than necessary.
Thanks.
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
Real computer scientists don't comment their code. The identifiers
are so long they can't afford the disk space.
More information about the U-Boot
mailing list