[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 12/16] i2c:smbus_eeprom: Add normal type name
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 12/16] i2c:smbus_eeprom: Add normal type name and cast to smbus_eeprom.c |
Date: |
Mon, 26 Nov 2018 21:35:35 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 |
On 26/11/18 21:04, address@hidden wrote:
> From: Corey Minyard <address@hidden>
>
> Create a type name and a cast macro and use those through the
> code.
>
> Signed-off-by: Corey Minyard <address@hidden>
> Reviewed-by: Peter Maydell <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/i2c/smbus_eeprom.c | 15 ++++++++++-----
> 1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/hw/i2c/smbus_eeprom.c b/hw/i2c/smbus_eeprom.c
> index 4d25222e23..8d4eed129f 100644
> --- a/hw/i2c/smbus_eeprom.c
> +++ b/hw/i2c/smbus_eeprom.c
> @@ -30,6 +30,11 @@
>
> //#define DEBUG
>
> +#define TYPE_SMBUS_EEPROM "smbus-eeprom"
> +
> +#define SMBUS_EEPROM(obj) \
> + OBJECT_CHECK(SMBusEEPROMDevice, (obj), TYPE_SMBUS_EEPROM)
> +
> typedef struct SMBusEEPROMDevice {
> SMBusDevice smbusdev;
> void *data;
> @@ -38,7 +43,7 @@ typedef struct SMBusEEPROMDevice {
>
> static uint8_t eeprom_receive_byte(SMBusDevice *dev)
> {
> - SMBusEEPROMDevice *eeprom = (SMBusEEPROMDevice *) dev;
> + SMBusEEPROMDevice *eeprom = SMBUS_EEPROM(dev);
> uint8_t *data = eeprom->data;
> uint8_t val = data[eeprom->offset++];
>
> @@ -51,7 +56,7 @@ static uint8_t eeprom_receive_byte(SMBusDevice *dev)
>
> static int eeprom_write_data(SMBusDevice *dev, uint8_t *buf, uint8_t len)
> {
> - SMBusEEPROMDevice *eeprom = (SMBusEEPROMDevice *) dev;
> + SMBusEEPROMDevice *eeprom = SMBUS_EEPROM(dev);
> uint8_t *data = eeprom->data;
>
> #ifdef DEBUG
> @@ -73,7 +78,7 @@ static int eeprom_write_data(SMBusDevice *dev, uint8_t
> *buf, uint8_t len)
>
> static void smbus_eeprom_realize(DeviceState *dev, Error **errp)
> {
> - SMBusEEPROMDevice *eeprom = (SMBusEEPROMDevice *)dev;
> + SMBusEEPROMDevice *eeprom = SMBUS_EEPROM(dev);
>
> eeprom->offset = 0;
> }
> @@ -97,7 +102,7 @@ static void smbus_eeprom_class_initfn(ObjectClass *klass,
> void *data)
> }
>
> static const TypeInfo smbus_eeprom_info = {
> - .name = "smbus-eeprom",
> + .name = TYPE_SMBUS_EEPROM,
> .parent = TYPE_SMBUS_DEVICE,
> .instance_size = sizeof(SMBusEEPROMDevice),
> .class_init = smbus_eeprom_class_initfn,
> @@ -114,7 +119,7 @@ void smbus_eeprom_init_one(I2CBus *smbus, uint8_t
> address, uint8_t *eeprom_buf)
> {
> DeviceState *dev;
>
> - dev = qdev_create((BusState *) smbus, "smbus-eeprom");
> + dev = qdev_create((BusState *) smbus, TYPE_SMBUS_EEPROM);
> qdev_prop_set_uint8(dev, "address", address);
> qdev_prop_set_ptr(dev, "data", eeprom_buf);
> qdev_init_nofail(dev);
>
- Re: [Qemu-devel] [PATCH v3 03/16] arm:i2c: Don't mask return from i2c_recv(), (continued)
- [Qemu-devel] [PATCH v3 09/16] migration: Add a VMSTATE_BOOL_TEST() macro, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 10/16] i2c:pm_smbus: Fix state transfer, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 13/16] i2c:smbus_eeprom: Add a size constant for the smbus_eeprom size, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 04/16] i2c: Don't check return value from i2c_recv(), minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 12/16] i2c:smbus_eeprom: Add normal type name and cast to smbus_eeprom.c, minyard, 2018/11/26
- Re: [Qemu-devel] [PATCH v3 12/16] i2c:smbus_eeprom: Add normal type name and cast to smbus_eeprom.c,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v3 05/16] i2c: Simplify and correct the SMBus state machine, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 01/16] i2c: Split smbus into parts, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 02/16] i2c: have I2C receive operation return uint8_t, minyard, 2018/11/26
- [Qemu-devel] [PATCH v3 11/16] i2c:smbus_slave: Add an SMBus vmstate structure, minyard, 2018/11/26