[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters t
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters type |
Date: |
Thu, 08 Jun 2017 19:23:39 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> Based on the underlying type of the data accessed, use the appropriate
> getters/setters.
Enumerating data here would be nice.
* AcpiPmInfo members s3_disabled, s4_disabled are bool, member s4_val is
an uint8_t.
* Property ACPI_PCIHP_IO_PROP is defined with
object_property_add_uint32_ptr().
* Property PCIE_HOST_MCFG_SIZE is implemented with visit_type_uint64()
* PCIDevice property "addr" is backed by PCIDevice member devfn, which
is an int32_t.
> Signed-off-by: Marc-André Lureau <address@hidden>
Preferably with a more detailed commit message:
Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PATCH v3 15/43] qdev: rename DEFINE_PROP_DEFAULT() to DEFINE_PROP_SIGNED(), (continued)
- [Qemu-devel] [PATCH v3 16/43] qdev: wrap default property value in an union, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 17/43] qdev: avoid type casts between signed and unsigned, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint32, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters type, Marc-André Lureau, 2017/06/07
- Re: [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters type,
Markus Armbruster <=
- [Qemu-devel] [PATCH v3 20/43] isa: use get_uint() for "io-base", Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 21/43] pc-dimm: use get_uint() for dimm properties, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 22/43] pc-dimm: make "size" property uint64, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 23/43] pcihp: use get_uint() for "bsel" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 24/43] aspeed: use {set, get}_uint() for "ram-size" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 25/43] bcm2835_fb: use {get, set}_uint() for "vcram-size" and "vcram-base", Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 26/43] platform-bus: use get_uint() for "addr" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 27/43] acpi: use get_uint() for "acpi-pcihp-io*" properties, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 28/43] acpi: use get_uint() for various acpi properties, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 29/43] acpi: use get_uint() for "pci-hole*" properties, Marc-André Lureau, 2017/06/07