On Tue, 17 Aug 2021 17:59:40 -0400
Tom Rini tr...@konsulko.com wrote:

> The instances of CONFIG_SYS_I2C_EEPROM_ADDR on these platforms doesn't
> match up with the rest of the EERPOM related defines in U-Boot.  Move
> these out of the CONFIG namespace rather than make complex Kconfig
> logic.
> 
> Cc: Samuel Egli <samuel.e...@siemens.com>
> Cc: Anatolij Gustschin <ag...@denx.de>
> Cc: Heiko Schocher <h...@denx.de>
> Signed-off-by: Tom Rini <tr...@konsulko.com>

Reviewed-by: Anatolij Gustschin <ag...@denx.de>

But probably some more cleanup in other board config headers needed,
see comment below.

...
> diff --git a/include/configs/draco.h b/include/configs/draco.h
> index 396eb7ded500..7c1f071d99ec 100644
> --- a/include/configs/draco.h
> +++ b/include/configs/draco.h
> @@ -32,10 +32,6 @@
>  /* I2C Configuration */
>  #define CONFIG_SYS_I2C_SPEED         100000
>  
> -#define CONFIG_SYS_I2C_EEPROM_ADDR              0x50
> -#define EEPROM_ADDR_DDR3 0x90
> -#define EEPROM_ADDR_CHIP 0x120

these defines are also in

 include/configs/etamin.h
 include/configs/rastaban.h
 include/configs/thuban.h

--
Anatolij

Reply via email to