From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:51696) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rhscc-0001mA-7c for qemu-devel@nongnu.org; Mon, 02 Jan 2012 19:52:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RhscU-0008Jz-F3 for qemu-devel@nongnu.org; Mon, 02 Jan 2012 19:52:57 -0500 Received: from e3.ny.us.ibm.com ([32.97.182.143]:49020) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RhscT-0008J3-KG for qemu-devel@nongnu.org; Mon, 02 Jan 2012 19:52:49 -0500 Received: from /spool/local by e3.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 2 Jan 2012 19:52:48 -0500 Received: from d01av02.pok.ibm.com (d01av02.pok.ibm.com [9.56.224.216]) by d01relay06.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id q030qbtC3530806 for ; Mon, 2 Jan 2012 19:52:37 -0500 Received: from d01av02.pok.ibm.com (loopback [127.0.0.1]) by d01av02.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id q030qaQm001572 for ; Mon, 2 Jan 2012 22:52:36 -0200 From: Anthony Liguori Date: Mon, 2 Jan 2012 18:52:00 -0600 Message-Id: <1325551939-24749-12-git-send-email-aliguori@us.ibm.com> In-Reply-To: <1325551939-24749-1-git-send-email-aliguori@us.ibm.com> References: <1325551939-24749-1-git-send-email-aliguori@us.ibm.com> Subject: [Qemu-devel] [PATCH 11/30] qdev: use a wrapper to access reset and promote reset to a class method List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Paolo Bonzini , Anthony Liguori , Markus Armbruster , =?UTF-8?q?Andreas=20F=C3=A4rber?= Signed-off-by: Anthony Liguori --- hw/intel-hda.c | 4 +--- hw/lsi53c895a.c | 2 +- hw/qdev.c | 22 +++++++++++++++++----- hw/qdev.h | 8 ++++++++ 4 files changed, 27 insertions(+), 9 deletions(-) diff --git a/hw/intel-hda.c b/hw/intel-hda.c index 8cb92d3..6e1c5de 100644 --- a/hw/intel-hda.c +++ b/hw/intel-hda.c @@ -1116,9 +1116,7 @@ static void intel_hda_reset(DeviceState *dev) /* reset codecs */ QTAILQ_FOREACH(qdev, &d->codecs.qbus.children, sibling) { cdev = DO_UPCAST(HDACodecDevice, qdev, qdev); - if (qdev_get_info(qdev)->reset) { - qdev_get_info(qdev)->reset(qdev); - } + device_reset(DEVICE(cdev)); d->state_sts |= (1 << cdev->cad); } intel_hda_update_irq(d); diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 9f475e0..3a87171 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -1681,7 +1681,7 @@ static void lsi_reg_writeb(LSIState *s, int offset, uint8_t val) DeviceState *dev; QTAILQ_FOREACH(dev, &s->bus.qbus.children, sibling) { - qdev_get_info(dev)->reset(dev); + device_reset(dev); } s->sstat0 |= LSI_SSTAT0_RST; lsi_script_scsi_interrupt(s, LSI_SIST0_RST, 0); diff --git a/hw/qdev.c b/hw/qdev.c index 9f92c7f..c26b1e7 100644 --- a/hw/qdev.c +++ b/hw/qdev.c @@ -48,7 +48,12 @@ static BusState *qbus_find(const char *path); static void qdev_subclass_init(ObjectClass *klass, void *data) { DeviceClass *dc = DEVICE_CLASS(klass); + dc->info = data; + dc->reset = dc->info->reset; + + /* Poison to try to detect future uses */ + dc->info->reset = NULL; } DeviceInfo *qdev_get_info(DeviceState *dev) @@ -374,8 +379,8 @@ int qdev_init(DeviceState *dev) dev->alias_required_for_version); } dev->state = DEV_STATE_INITIALIZED; - if (dev->hotplugged && qdev_get_info(dev)->reset) { - qdev_get_info(dev)->reset(dev); + if (dev->hotplugged) { + device_reset(dev); } return 0; } @@ -408,9 +413,7 @@ int qdev_unplug(DeviceState *dev) static int qdev_reset_one(DeviceState *dev, void *opaque) { - if (qdev_get_info(dev)->reset) { - qdev_get_info(dev)->reset(dev); - } + device_reset(dev); return 0; } @@ -1556,6 +1559,15 @@ void qdev_machine_init(void) qdev_get_peripheral(); } +void device_reset(DeviceState *dev) +{ + DeviceClass *klass = DEVICE_GET_CLASS(dev); + + if (klass->reset) { + klass->reset(dev); + } +} + static TypeInfo device_type_info = { .name = TYPE_DEVICE, .parent = TYPE_OBJECT, diff --git a/hw/qdev.h b/hw/qdev.h index 329724e..822186e 100644 --- a/hw/qdev.h +++ b/hw/qdev.h @@ -75,6 +75,7 @@ typedef struct DeviceProperty typedef struct DeviceClass { ObjectClass parent_class; DeviceInfo *info; + void (*reset)(DeviceState *dev); } DeviceClass; /* This structure should not be accessed directly. We declare it here @@ -646,4 +647,11 @@ char *qdev_get_type(DeviceState *dev, Error **errp); */ void qdev_machine_init(void); +/** + * @device_reset + * + * Reset a single device (by calling the reset method). + */ +void device_reset(DeviceState *dev); + #endif -- 1.7.4.1