siemens: Move CONFIG_SYS_I2C_EEPROM_ADDR usage out of CONFIG namespace
authorTom Rini <trini@konsulko.com>
Tue, 17 Aug 2021 21:59:40 +0000 (17:59 -0400)
committerTom Rini <trini@konsulko.com>
Mon, 30 Aug 2021 18:10:05 +0000 (14:10 -0400)
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.egli@siemens.com>
Cc: Anatolij Gustschin <agust@denx.de>
Cc: Heiko Schocher <hs@denx.de>
Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Reviewed-by: Anatolij Gustschin <agust@denx.de>
board/siemens/common/board.c
board/siemens/draco/board.c
include/configs/draco.h

index 1bdf404..5628366 100644 (file)
@@ -70,6 +70,7 @@ void sdram_init(void)
 #endif /* #ifdef CONFIG_SPL_BUILD */
 
 #ifndef CONFIG_SPL_BUILD
+#define FACTORYSET_EEPROM_ADDR         0x50
 /*
  * Basic board specific setup.  Pinmux has been handled already.
  */
@@ -87,7 +88,7 @@ int board_init(void)
        gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100;
 
 #ifdef CONFIG_FACTORYSET
-       factoryset_read_eeprom(CONFIG_SYS_I2C_EEPROM_ADDR);
+       factoryset_read_eeprom(FACTORYSET_EEPROM_ADDR);
 #endif
 
        gpmc_init();
index af35bc1..f898bba 100644 (file)
@@ -132,12 +132,16 @@ struct am335x_nand_geometry {
        u8 nand_bus;
 };
 
+#define EEPROM_ADDR            0x50
+#define EEPROM_ADDR_DDR3       0x90
+#define EEPROM_ADDR_CHIP       0x120
+
 static int draco_read_nand_geometry(void)
 {
        struct am335x_nand_geometry geo;
 
        /* Read NAND geometry */
-       if (i2c_read(CONFIG_SYS_I2C_EEPROM_ADDR, 0x80, 2,
+       if (i2c_read(EEPROM_ADDR, 0x80, 2,
                     (uchar *)&geo, sizeof(struct am335x_nand_geometry))) {
                printf("Could not read the NAND geomtery; something fundamentally wrong on the I2C bus.\n");
                return -EIO;
@@ -160,20 +164,20 @@ static int draco_read_nand_geometry(void)
 static int read_eeprom(void)
 {
        /* Check if baseboard eeprom is available */
-       if (i2c_probe(CONFIG_SYS_I2C_EEPROM_ADDR)) {
+       if (i2c_probe(EEPROM_ADDR)) {
                printf("Could not probe the EEPROM; something fundamentally wrong on the I2C bus.\n");
                return 1;
        }
 
 #ifdef CONFIG_SPL_BUILD
        /* Read Siemens eeprom data (DDR3) */
-       if (i2c_read(CONFIG_SYS_I2C_EEPROM_ADDR, EEPROM_ADDR_DDR3, 2,
+       if (i2c_read(EEPROM_ADDR, EEPROM_ADDR_DDR3, 2,
                     (uchar *)&settings.ddr3, sizeof(struct ddr3_data))) {
                printf("Could not read the EEPROM; something fundamentally wrong on the I2C bus.\nUse default DDR3 timings\n");
                set_default_ddr3_timings();
        }
        /* Read Siemens eeprom data (CHIP) */
-       if (i2c_read(CONFIG_SYS_I2C_EEPROM_ADDR, EEPROM_ADDR_CHIP, 2,
+       if (i2c_read(EEPROM_ADDR, EEPROM_ADDR_CHIP, 2,
                     (uchar *)&settings.chip, sizeof(settings.chip)))
                printf("Could not read chip settings\n");
 
index 396eb7d..7c1f071 100644 (file)
 /* 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
-
 #define CONFIG_FACTORYSET
 
 /* Define own nand partitions */