[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v13 09/12] qdev: pass the check callback to qdev_ini
From: |
Eric Auger |
Subject: |
[Qemu-devel] [PATCH v13 09/12] qdev: pass the check callback to qdev_init_gpio_out_named |
Date: |
Tue, 28 Apr 2015 17:51:10 +0100 |
qdev_init_gpio_out_named takes a new argument corresponding to the
check callback passed to object_property_add_link. In qdev_init_gpio_out
and sysbus_init_irq, this callback is currently set to the dummy
object_property_allow_set_link.
This will allow qdev_init_gpio_out_named callers to specialize this
callback. A subsequent patch will implement that for sysbus.
Signed-off-by: Eric Auger <address@hidden>
---
v1 -> v2:
- fix qdev_init_gpio_out_named call in sysbus_init_irq
- rewording of commit message
---
hw/core/qdev.c | 8 +++++---
hw/core/sysbus.c | 3 ++-
include/hw/qdev-core.h | 3 ++-
include/qom/object.h | 6 ++++--
4 files changed, 13 insertions(+), 7 deletions(-)
diff --git a/hw/core/qdev.c b/hw/core/qdev.c
index 6e6a65d..857217d 100644
--- a/hw/core/qdev.c
+++ b/hw/core/qdev.c
@@ -452,7 +452,8 @@ void qdev_init_gpio_in(DeviceState *dev, qemu_irq_handler
handler, int n)
}
void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq *pins,
- const char *name, int n)
+ const char *name, int n,
+ LinkPropertySetter check_cb)
{
int i;
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
@@ -465,7 +466,7 @@ void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq
*pins,
memset(&pins[i], 0, sizeof(*pins));
object_property_add_link(OBJECT(dev), propname, TYPE_IRQ,
(Object **)&pins[i],
- object_property_allow_set_link,
+ check_cb,
OBJ_PROP_LINK_UNREF_ON_RELEASE,
&error_abort);
}
@@ -474,7 +475,8 @@ void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq
*pins,
void qdev_init_gpio_out(DeviceState *dev, qemu_irq *pins, int n)
{
- qdev_init_gpio_out_named(dev, pins, NULL, n);
+ qdev_init_gpio_out_named(dev, pins, NULL, n,
+ object_property_allow_set_link);
}
qemu_irq qdev_get_gpio_in_named(DeviceState *dev, const char *name, int n)
diff --git a/hw/core/sysbus.c b/hw/core/sysbus.c
index b53c351..1bcb64d 100644
--- a/hw/core/sysbus.c
+++ b/hw/core/sysbus.c
@@ -159,7 +159,8 @@ void sysbus_mmio_map_overlap(SysBusDevice *dev, int n,
hwaddr addr,
/* Request an IRQ source. The actual IRQ object may be populated later. */
void sysbus_init_irq(SysBusDevice *dev, qemu_irq *p)
{
- qdev_init_gpio_out_named(DEVICE(dev), p, SYSBUS_DEVICE_GPIO_IRQ, 1);
+ qdev_init_gpio_out_named(DEVICE(dev), p, SYSBUS_DEVICE_GPIO_IRQ, 1,
+ object_property_allow_set_link);
}
/* Pass IRQs from a target device. */
diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h
index 4e673f9..0885f39 100644
--- a/include/hw/qdev-core.h
+++ b/include/hw/qdev-core.h
@@ -293,7 +293,8 @@ void qdev_init_gpio_out(DeviceState *dev, qemu_irq *pins,
int n);
void qdev_init_gpio_in_named(DeviceState *dev, qemu_irq_handler handler,
const char *name, int n);
void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq *pins,
- const char *name, int n);
+ const char *name, int n,
+ LinkPropertySetter fn);
void qdev_pass_gpios(DeviceState *dev, DeviceState *container,
const char *name);
diff --git a/include/qom/object.h b/include/qom/object.h
index d2d7748..95d1a1d 100644
--- a/include/qom/object.h
+++ b/include/qom/object.h
@@ -33,6 +33,9 @@ typedef struct TypeInfo TypeInfo;
typedef struct InterfaceClass InterfaceClass;
typedef struct InterfaceInfo InterfaceInfo;
+typedef void (*LinkPropertySetter)(Object *, const char *,
+ Object *, Error **);
+
#define TYPE_OBJECT "object"
/**
@@ -1165,8 +1168,7 @@ void object_property_allow_set_link(Object *, const char
*,
*/
void object_property_add_link(Object *obj, const char *name,
const char *type, Object **child,
- void (*check)(Object *obj, const char *name,
- Object *val, Error **errp),
+ LinkPropertySetter check,
ObjectPropertyLinkFlags flags,
Error **errp);
--
1.8.3.2
- [Qemu-devel] [PATCH v13 01/12] linux-headers: update headers according to 4.1-rc0, (continued)
- [Qemu-devel] [PATCH v13 01/12] linux-headers: update headers according to 4.1-rc0, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 02/12] hw/vfio/platform: vfio-platform skeleton, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 04/12] hw/vfio/platform: calxeda xgmac device, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 03/12] hw/vfio/platform: add irq assignment, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 06/12] kvm: rename kvm_irqchip_[add, remove]_irqfd_notifier with gsi suffix, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 05/12] hw/arm/sysbus-fdt: enable vfio-calxeda-xgmac dynamic instantiation, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 07/12] kvm-all.c: add qemu_irq/gsi hash table and utility routines, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 08/12] intc: arm_gic_kvm: set the qemu_irq/gsi mapping, Eric Auger, 2015/04/28
- [Qemu-devel] [PATCH v13 09/12] qdev: pass the check callback to qdev_init_gpio_out_named,
Eric Auger <=
- [Qemu-devel] [PATCH v13 10/12] qdev: check callback takes the property child as third argument, Eric Auger, 2015/04/28
[Qemu-devel] [PATCH v13 11/12] sysbus: add irq_set_hook, Eric Auger, 2015/04/28
[Qemu-devel] [PATCH v13 12/12] hw/vfio/platform: add irqfd support, Eric Auger, 2015/04/28