[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv3 06/13] sparc32_dma: use object link instead of
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCHv3 06/13] sparc32_dma: use object link instead of qdev property to pass IOMMU reference |
Date: |
Thu, 19 Oct 2017 01:38:43 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 10/14/2017 03:38 PM, Mark Cave-Ayland wrote:
> This enables us to remove the last remaining (opaque) qdev property. Whilst we
> are here, also update iommu_init() to use TYPE_SUN4M_IOMMU instead of a
> hardcoded string.
>
> Signed-off-by: Mark Cave-Ayland <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/dma/sparc32_dma.c | 13 +++++--------
> hw/sparc/sun4m.c | 4 ++--
> 2 files changed, 7 insertions(+), 10 deletions(-)
>
> diff --git a/hw/dma/sparc32_dma.c b/hw/dma/sparc32_dma.c
> index ae8fa06..c56a2ba 100644
> --- a/hw/dma/sparc32_dma.c
> +++ b/hw/dma/sparc32_dma.c
> @@ -263,24 +263,21 @@ static void sparc32_dma_device_init(Object *obj)
>
> sysbus_init_mmio(sbd, &s->iomem);
>
> + object_property_add_link(OBJECT(dev), "iommu", TYPE_SUN4M_IOMMU,
> + (Object **) &s->iommu,
> + qdev_prop_allow_set_link_before_realize,
> + 0, NULL);
> +
> qdev_init_gpio_in(dev, dma_set_irq, 1);
> qdev_init_gpio_out(dev, s->gpio, 2);
> }
>
> -static Property sparc32_dma_device_properties[] = {
> - DEFINE_PROP_PTR("iommu_opaque", DMADeviceState, iommu),
> - DEFINE_PROP_END_OF_LIST(),
> -};
> -
> static void sparc32_dma_device_class_init(ObjectClass *klass, void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->reset = sparc32_dma_device_reset;
> dc->vmsd = &vmstate_sparc32_dma_device;
> - dc->props = sparc32_dma_device_properties;
> - /* Reason: pointer property "iommu_opaque" */
> - dc->user_creatable = false;
> }
>
> static const TypeInfo sparc32_dma_device_info = {
> diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c
> index 4f2ed4b..12d36b5 100644
> --- a/hw/sparc/sun4m.c
> +++ b/hw/sparc/sun4m.c
> @@ -297,7 +297,7 @@ static void *iommu_init(hwaddr addr, uint32_t version,
> qemu_irq irq)
> DeviceState *dev;
> SysBusDevice *s;
>
> - dev = qdev_create(NULL, "iommu");
> + dev = qdev_create(NULL, TYPE_SUN4M_IOMMU);
> qdev_prop_set_uint32(dev, "version", version);
> qdev_init_nofail(dev);
> s = SYS_BUS_DEVICE(dev);
> @@ -313,7 +313,7 @@ static void *sparc32_dma_init(hwaddr daddr, void *iommu,
> int is_ledma)
> SysBusDevice *s;
>
> dev = qdev_create(NULL, is_ledma ? "sparc32-ledma" : "sparc32-espdma");
> - qdev_prop_set_ptr(dev, "iommu_opaque", iommu);
> + object_property_set_link(OBJECT(dev), OBJECT(iommu), "iommu",
> &error_abort);
> qdev_init_nofail(dev);
> s = SYS_BUS_DEVICE(dev);
> sysbus_mmio_map(s, 0, daddr);
>
- [Qemu-devel] [PATCHv3 00/13] sun4m: sparc32_dma tidy-ups, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 04/13] sun4m: move DMA device wiring from sparc32_dma_init() to sun4m_hw_init(), Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 03/13] sparc32_dma: move type declarations from sparc32_dma.c to sparc32_dma.h, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 07/13] esp: move TYPE_ESP and SysBusESPState from esp.c to esp.h, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 06/13] sparc32_dma: use object link instead of qdev property to pass IOMMU reference, Mark Cave-Ayland, 2017/10/14
- Re: [Qemu-devel] [PATCHv3 06/13] sparc32_dma: use object link instead of qdev property to pass IOMMU reference,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCHv3 05/13] sun4m_iommu: move TYPE_SUN4M_IOMMU declaration to sun4m.h, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 01/13] sparc32_dma: rename SPARC32_DMA type to SPARC32_DMA_DEVICE, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to sun4m.h, Mark Cave-Ayland, 2017/10/14
- [Qemu-devel] [PATCHv3 08/13] sparc32_dma: make esp device child of espdma device, Mark Cave-Ayland, 2017/10/14