[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 31/37] smbus-eeprom: remove PROP_PTR
From: |
Corey Minyard |
Subject: |
Re: [PATCH v4 31/37] smbus-eeprom: remove PROP_PTR |
Date: |
Fri, 29 Nov 2019 08:39:34 -0600 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Fri, Nov 29, 2019 at 01:06:41PM +0400, Marc-André Lureau wrote:
> Hi Corey
>
> On Fri, Nov 22, 2019 at 2:44 AM Corey Minyard <address@hidden> wrote:
> >
> > On Wed, Nov 20, 2019 at 07:24:36PM +0400, Marc-André Lureau wrote:
> > > Instead, set the initial data field directly.
> > >
> > > (the initial data is an array of 256 bytes. As I don't know if it may
> > > change over time, I keep the pointer to original buffer as is, but it
> > > might be worth to consider to copy it instead)
> >
> > This looks ok to me, I can add it to my tree. I think we are in hard
> > freeze now, so this will have to wait until that's done.
>
> Thanks, I think I would rather keep it in this series, as the goal is
> to remove PROP_PTR, and tracking various subsystems will be tedious.
> Instead, we can hopefully do it in one go.
>
> Does it get your reviewed-by then?
Yes, I thought I had done that.
Reviewed-by: Corey Minyard <address@hidden>
>
> >
> > -corey
> >
> > >
> > > Signed-off-by: Marc-André Lureau <address@hidden>
> > > ---
> > > hw/i2c/smbus_eeprom.c | 17 ++++++++---------
> > > 1 file changed, 8 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
> > > index 54c86a0112..533c728b3b 100644
> > > --- a/hw/i2c/smbus_eeprom.c
> > > +++ b/hw/i2c/smbus_eeprom.c
> > > @@ -44,7 +44,7 @@
> > > typedef struct SMBusEEPROMDevice {
> > > SMBusDevice smbusdev;
> > > uint8_t data[SMBUS_EEPROM_SIZE];
> > > - void *init_data;
> > > + uint8_t *init_data;
> > > uint8_t offset;
> > > bool accessed;
> > > } SMBusEEPROMDevice;
> > > @@ -129,14 +129,14 @@ static void smbus_eeprom_reset(DeviceState *dev)
> > >
> > > static void smbus_eeprom_realize(DeviceState *dev, Error **errp)
> > > {
> > > + SMBusEEPROMDevice *eeprom = SMBUS_EEPROM(dev);
> > > +
> > > smbus_eeprom_reset(dev);
> > > + if (eeprom->init_data == NULL) {
> > > + error_setg(errp, "init_data cannot be NULL");
> > > + }
> > > }
> > >
> > > -static Property smbus_eeprom_properties[] = {
> > > - DEFINE_PROP_PTR("data", SMBusEEPROMDevice, init_data),
> > > - DEFINE_PROP_END_OF_LIST(),
> > > -};
> > > -
> > > static void smbus_eeprom_class_initfn(ObjectClass *klass, void *data)
> > > {
> > > DeviceClass *dc = DEVICE_CLASS(klass);
> > > @@ -146,9 +146,8 @@ static void smbus_eeprom_class_initfn(ObjectClass
> > > *klass, void *data)
> > > dc->reset = smbus_eeprom_reset;
> > > sc->receive_byte = eeprom_receive_byte;
> > > sc->write_data = eeprom_write_data;
> > > - dc->props = smbus_eeprom_properties;
> > > dc->vmsd = &vmstate_smbus_eeprom;
> > > - /* Reason: pointer property "data" */
> > > + /* Reason: init_data */
> > > dc->user_creatable = false;
> > > }
> > >
> > > @@ -172,7 +171,7 @@ void smbus_eeprom_init_one(I2CBus *smbus, uint8_t
> > > address, uint8_t *eeprom_buf)
> > >
> > > dev = qdev_create((BusState *) smbus, TYPE_SMBUS_EEPROM);
> > > qdev_prop_set_uint8(dev, "address", address);
> > > - qdev_prop_set_ptr(dev, "data", eeprom_buf);
> > > + SMBUS_EEPROM(dev)->init_data = eeprom_buf;
> > > qdev_init_nofail(dev);
> > > }
> > >
> > > --
> > > 2.24.0
> > >
> >
>
>
> --
> Marc-André Lureau
- Re: [PATCH v4 27/37] leon3: use qdev gpio facilities for the PIL, (continued)
- [PATCH v4 28/37] qdev: use g_strcmp0() instead of open-coding it, Marc-André Lureau, 2019/11/20
- [PATCH v4 29/37] RFC: mips/cps: fix setting saar property, Marc-André Lureau, 2019/11/20
- [PATCH v4 30/37] cris: improve passing PIC interrupt vector to the CPU, Marc-André Lureau, 2019/11/20
- [PATCH v4 31/37] smbus-eeprom: remove PROP_PTR, Marc-André Lureau, 2019/11/20
- Re: [PATCH v4 31/37] smbus-eeprom: remove PROP_PTR, Philippe Mathieu-Daudé, 2019/11/22
[PATCH v4 32/37] omap-intc: remove PROP_PTR, Marc-André Lureau, 2019/11/20
[PATCH v4 33/37] omap-i2c: remove PROP_PTR, Marc-André Lureau, 2019/11/20
[PATCH v4 34/37] omap-gpio: remove PROP_PTR, Marc-André Lureau, 2019/11/20
[PATCH v4 35/37] qdev: remove PROP_MEMORY_REGION, Marc-André Lureau, 2019/11/20
[PATCH v4 36/37] qdev: remove QDEV_PROP_PTR, Marc-André Lureau, 2019/11/20
[PATCH v4 37/37] qdev/qom: remove some TODO limitations now that PROP_PTR is gone, Marc-André Lureau, 2019/11/20