[U-Boot] [PATCH] ppc4xx: Fix EEPROM configuration on Kilauea

[Wolfgang Denk]

Dear Stefan Roese,

In message <1248182100-15318-1-git-send-email-sr at denx.de> you wrote:
> Kilauea has an AT24C02 EEPROM which has an 8 byte page. Without defining
> CONFIG_SYS_EEPROM_PAGE_WRITE_BITS to 3 the "eeprom" command doesn't
> work correctly.
> 
> Signed-off-by: Stefan Roese <sr at denx.de>
> ---
>  include/configs/kilauea.h |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/include/configs/kilauea.h b/include/configs/kilauea.h
> index df1b061..1e7e731 100644
> --- a/include/configs/kilauea.h
> +++ b/include/configs/kilauea.h
> @@ -374,9 +374,10 @@
>   *----------------------------------------------------------------------*/
>  #define CONFIG_SYS_I2C_SPEED		400000	/* I2C speed and slave address	*/
>  
> -#define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS	6	/* 24C02 requires 5ms delay */
>  #define CONFIG_SYS_I2C_EEPROM_ADDR	0x52	/* I2C boot EEPROM (24C02BN)	*/
>  #define CONFIG_SYS_I2C_EEPROM_ADDR_LEN	1	/* Bytes of address		*/
> +#define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS	3
> +#define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS	10

The commit message does not mention the increased write delay. Is htis
really needed? The (removed) comment says 5ms should be sufficient?

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
If A equals success, then the formula is A = X + Y + Z. X is work.  Y
is play. Z is keep your mouth shut.                 - Albert Einstein