[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 6/7] hw/nvram: Update at24c EEPROM init function in NPCM7x
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v4 6/7] hw/nvram: Update at24c EEPROM init function in NPCM7xx boards |
Date: |
Sat, 06 Nov 2021 07:53:27 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hao Wu <wuhaotsh@google.com> writes:
> We made 3 changes to the at24c_eeprom_init function in
> npcm7xx_boards.c:
>
> 1. We allow the function to take a I2CBus* as parameter. This allows
> us to attach an EEPROM device behind an I2C mux which is not
> possible with the old method.
>
> 2. We make at24c EEPROMs are backed by drives so that we can
> specify the content of the EEPROMs.
>
> 3. Instead of using i2c address as unit number, This patch assigns
> unique unit numbers for each eeproms in each board. This avoids
> conflict in providing multiple eeprom contents with the same address.
> In the old method if we specify two drives with the same unit number,
> the following error will occur: `Device with id 'none85' exists`.
When a commit does three things, chances are splitting it into three
commits would be an improvement. This is *not* a demand.
>
> Signed-off-by: Hao Wu <wuhaotsh@google.com>
> ---
> hw/arm/npcm7xx_boards.c | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/hw/arm/npcm7xx_boards.c b/hw/arm/npcm7xx_boards.c
> index dec7d16ae5..9121e081fa 100644
> --- a/hw/arm/npcm7xx_boards.c
> +++ b/hw/arm/npcm7xx_boards.c
> @@ -126,13 +126,17 @@ static I2CBus *npcm7xx_i2c_get_bus(NPCM7xxState *soc,
> uint32_t num)
> return I2C_BUS(qdev_get_child_bus(DEVICE(&soc->smbus[num]), "i2c-bus"));
> }
>
> -static void at24c_eeprom_init(NPCM7xxState *soc, int bus, uint8_t addr,
> - uint32_t rsize)
> +static void at24c_eeprom_init(I2CBus *i2c_bus, int bus, uint8_t addr,
> + uint32_t rsize, int unit_number)
I'd keep bus and unit number together.
I'd name the new parameter @unit, to match drive_get().
> {
> - I2CBus *i2c_bus = npcm7xx_i2c_get_bus(soc, bus);
> I2CSlave *i2c_dev = i2c_slave_new("at24c-eeprom", addr);
> DeviceState *dev = DEVICE(i2c_dev);
> + DriveInfo *dinfo;
>
> + dinfo = drive_get(IF_OTHER, bus, unit_number);
> + if (dinfo) {
> + qdev_prop_set_drive(dev, "drive", blk_by_legacy_dinfo(dinfo));
> + }
> qdev_prop_set_uint32(dev, "rom-size", rsize);
> i2c_slave_realize_and_unref(i2c_dev, i2c_bus, &error_abort);
> }
> @@ -239,8 +243,8 @@ static void quanta_gsj_i2c_init(NPCM7xxState *soc)
> i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 3), "tmp105", 0x5c);
> i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 4), "tmp105", 0x5c);
>
> - at24c_eeprom_init(soc, 9, 0x55, 8192);
> - at24c_eeprom_init(soc, 10, 0x55, 8192);
> + at24c_eeprom_init(npcm7xx_i2c_get_bus(soc, 9), 9, 0x55, 8192, 0);
> + at24c_eeprom_init(npcm7xx_i2c_get_bus(soc, 10), 10, 0x55, 8192, 1);
>
> /*
> * i2c-11: