[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 10/10] i2c: Add vmstate handling to the smbus
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v2 10/10] i2c: Add vmstate handling to the smbus eeprom |
Date: |
Wed, 29 Aug 2018 13:01:44 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
* Dr. David Alan Gilbert (address@hidden) wrote:
> * address@hidden (address@hidden) wrote:
> > From: Corey Minyard <address@hidden>
> >
> > Signed-off-by: Corey Minyard <address@hidden>
> > Cc: Paolo Bonzini <address@hidden>
> > Cc: "Michael S . Tsirkin" <address@hidden>
> > Cc: Dr. David Alan Gilbert <address@hidden>
> > ---
> > hw/i2c/smbus_eeprom.c | 16 +++++++++++++++-
> > 1 file changed, 15 insertions(+), 1 deletion(-)
> >
> > diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
> > index f18aa3d..d4430b0 100644
> > --- a/hw/i2c/smbus_eeprom.c
> > +++ b/hw/i2c/smbus_eeprom.c
> > @@ -29,6 +29,8 @@
> >
> > //#define DEBUG
> >
> > +#define TYPE_SMBUS_EEPROM_DEVICE "smbus-eeprom"
> > +
> > typedef struct SMBusEEPROMDevice {
> > SMBusDevice smbusdev;
> > void *data;
> > @@ -97,6 +99,17 @@ static uint8_t eeprom_read_data(SMBusDevice *dev,
> > uint8_t cmd, int n)
> > return eeprom_receive_byte(dev);
> > }
> >
> > +static const VMStateDescription vmstate_smbus_eeprom = {
> > + .name = TYPE_SMBUS_EEPROM_DEVICE,
> > + .version_id = 1,
> > + .minimum_version_id = 1,
>
> Can you add a .needed and wire it up to a property that we can then wire
> to machine types; that way we don't break migration compatibility.
>
> > + .fields = (VMStateField[]) {
> > + VMSTATE_SMBUS_DEVICE(smbusdev, SMBusEEPROMDevice),
> > + VMSTATE_UINT8(offset, SMBusEEPROMDevice),
>
> What about the data?
Also; now I'm looking - I'm confused by 'offset' being an 8bit int
when I can see hw/arm/aspeed.c calling smbus_eeprom_init_one passing 8KB
and 32KB buffers in.
Dave
> Dave
>
> > + VMSTATE_END_OF_LIST()
> > + }
> > +};
> > +
> > static void smbus_eeprom_realize(DeviceState *dev, Error **errp)
> > {
> > SMBusEEPROMDevice *eeprom = (SMBusEEPROMDevice *)dev;
> > @@ -121,12 +134,13 @@ static void smbus_eeprom_class_initfn(ObjectClass
> > *klass, void *data)
> > sc->write_data = eeprom_write_data;
> > sc->read_data = eeprom_read_data;
> > dc->props = smbus_eeprom_properties;
> > + dc->vmsd = &vmstate_smbus_eeprom;
> > /* Reason: pointer property "data" */
> > dc->user_creatable = false;
> > }
> >
> > static const TypeInfo smbus_eeprom_info = {
> > - .name = "smbus-eeprom",
> > + .name = TYPE_SMBUS_EEPROM_DEVICE,
> > .parent = TYPE_SMBUS_DEVICE,
> > .instance_size = sizeof(SMBusEEPROMDevice),
> > .class_init = smbus_eeprom_class_initfn,
> > --
> > 2.7.4
> >
> --
> Dr. David Alan Gilbert / address@hidden / Manchester, UK
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH v2 01/10] i2c:pm_smbus: Clean up some style issues, (continued)
- [Qemu-devel] [PATCH v2 08/10] i2c:pm_smbus: Add the ability to force block transfer enable, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 09/10] i2c: Add an SMBus vmstate structure, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 05/10] i2c:pm_smbus: Fix state transfer, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 10/10] i2c: Add vmstate handling to the smbus eeprom, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 03/10] i2c:pm_smbus: Make the I2C block read command read-only, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 06/10] i2c:pm_smbus: Add interrupt handling, minyard, 2018/08/20
- [Qemu-devel] [PATCH v2 07/10] Don't delay host status register busy bit when interrupts are enabled, minyard, 2018/08/20
- Re: [Qemu-devel] [PATCH v2 0/8] Make the pm_smbus code more correct, Paolo Bonzini, 2018/08/21