[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/6] hw/arm/aspeed: Remove local copy of at24c_eeprom_init
From: |
Peter Delevoryas |
Subject: |
Re: [PATCH 2/6] hw/arm/aspeed: Remove local copy of at24c_eeprom_init |
Date: |
Mon, 16 Jan 2023 09:21:52 -0800 |
On Mon, Jan 16, 2023 at 01:24:36PM +0100, Philippe Mathieu-Daudé wrote:
> On 14/1/23 18:01, Peter Delevoryas wrote:
> > Signed-off-by: Peter Delevoryas <peter@pjd.dev>
> > ---
> > hw/arm/aspeed.c | 10 +---------
> > 1 file changed, 1 insertion(+), 9 deletions(-)
>
> > -static void at24c_eeprom_init(I2CBus *bus, uint8_t addr, uint32_t rsize)
> > -{
> > - I2CSlave *i2c_dev = i2c_slave_new("at24c-eeprom", addr);
> > - DeviceState *dev = DEVICE(i2c_dev);
> > -
> > - qdev_prop_set_uint32(dev, "rom-size", rsize);
> > - i2c_slave_realize_and_unref(i2c_dev, bus, &error_abort);
> > -}
>
> Why not squash in previous commit as 'extract helper' change?
+1, I'll squash this.
>
> Anyhow,
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>
>
- [PATCH 0/6] hw/nvram/eeprom_at24c: Cleanup + FRUID EEPROM init example, Peter Delevoryas, 2023/01/14
- [PATCH 2/6] hw/arm/aspeed: Remove local copy of at24c_eeprom_init, Peter Delevoryas, 2023/01/14
- [PATCH 1/6] hw/nvram/eeprom_at24c: Add header w/ init helper, Peter Delevoryas, 2023/01/14
- [PATCH 3/6] hw/arm/aspeed: Replace aspeed_eeprom_init with at24c_eeprom_init, Peter Delevoryas, 2023/01/14
- [PATCH 5/6] hw/nvram/eeprom_at24c: Add I2C write helper, Peter Delevoryas, 2023/01/14
- [PATCH 6/6] hw/arm/aspeed: Init fby35 BMC FRUID EEPROM, Peter Delevoryas, 2023/01/14