Hi Tom, On 11 May 2017 at 18:33, Tom Rini <tr...@konsulko.com> wrote: > On Thu, May 11, 2017 at 06:23:57PM -0600, Simon Glass wrote: > >> Drop use of this long-deprecated option. > [snip] >> diff --git a/include/configs/PATI.h b/include/configs/PATI.h >> index e53db2485b..046aa0d5f1 100644 >> --- a/include/configs/PATI.h >> +++ b/include/configs/PATI.h >> @@ -38,7 +38,6 @@ >> */ >> #define CONFIG_CMD_REGINFO >> #define CONFIG_CMD_REGINFO >> -#define CONFIG_CMD_EEPROM >> #define CONFIG_CMD_IRQ >> >> #define CONFIG_BOOTCOMMAND "" /* autoboot command >> */ > [snip] >> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h >> index fc8a08f5b7..5c1a6d64da 100644 >> --- a/include/configs/am335x_evm.h >> +++ b/include/configs/am335x_evm.h >> @@ -176,11 +176,6 @@ >> #define CONFIG_SYS_NS16550_COM5 0x481a8000 /* UART4 */ >> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */ >> >> -#define CONFIG_CMD_EEPROM >> -#define CONFIG_ENV_EEPROM_IS_ON_I2C >> -#define CONFIG_SYS_I2C_EEPROM_ADDR 0x50 /* Main EEPROM */ >> -#define CONFIG_SYS_I2C_EEPROM_ADDR_LEN 2 >> - >> /* PMIC support */ >> #define CONFIG_POWER_TPS65217 >> #define CONFIG_POWER_TPS65910 >> diff --git a/include/configs/am43xx_evm.h b/include/configs/am43xx_evm.h >> index 1feb946834..ba185ebf5a 100644 >> --- a/include/configs/am43xx_evm.h >> +++ b/include/configs/am43xx_evm.h >> @@ -22,12 +22,6 @@ >> #define CONFIG_SYS_NS16550_SERIAL >> #endif >> >> -/* I2C Configuration */ >> -#define CONFIG_CMD_EEPROM >> -#define CONFIG_ENV_EEPROM_IS_ON_I2C >> -#define CONFIG_SYS_I2C_EEPROM_ADDR 0x50 /* Main EEPROM */ >> -#define CONFIG_SYS_I2C_EEPROM_ADDR_LEN 2 >> - >> /* Power */ >> #define CONFIG_POWER >> #define CONFIG_POWER_I2C >> diff --git a/include/configs/ti_armv7_keystone2.h >> b/include/configs/ti_armv7_keystone2.h >> index 868464cd32..3161c50abb 100644 >> --- a/include/configs/ti_armv7_keystone2.h >> +++ b/include/configs/ti_armv7_keystone2.h >> @@ -208,7 +208,6 @@ >> /* U-Boot command configuration */ >> #define CONFIG_CMD_SAVES >> #define CONFIG_CMD_UBIFS >> -#define CONFIG_CMD_EEPROM >> >> /* U-Boot general configuration */ >> #define CONFIG_MISC_INIT_R > > Er, what's all of this about EEPROM stuff you're dropping?
It uses I2C (as does the environment and RTC on some boards) so I need to drop this too. Let me know if you can think of a better way. I am not sure this will be enough though. I looked through the CONFIG whitelist a few days ago and there are many boards that have board-specific settings like I2C addresses, speeds, etc. I am not sure how we can keep these board around since we really don't want to add these sorts of board-specific settings to Kconfig. They should be in the device tree. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot