From patchwork Fri Feb 24 10:18:21 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?C=C3=A9dric_Le_Goater?= X-Patchwork-Id: 732088 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3vV7DS6dsKz9s7C for ; Fri, 24 Feb 2017 21:50:32 +1100 (AEDT) Received: from localhost ([::1]:36101 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1chDSI-0004Gx-Bh for incoming@patchwork.ozlabs.org; Fri, 24 Feb 2017 05:50:30 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:32892) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1chCza-0006qd-5I for qemu-devel@nongnu.org; Fri, 24 Feb 2017 05:20:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1chCzW-0000PQ-2O for qemu-devel@nongnu.org; Fri, 24 Feb 2017 05:20:50 -0500 Received: from 9.mo5.mail-out.ovh.net ([178.32.96.204]:45158) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1chCzV-0000K6-Rm for qemu-devel@nongnu.org; Fri, 24 Feb 2017 05:20:45 -0500 Received: from player734.ha.ovh.net (b6.ovh.net [213.186.33.56]) by mo5.mail-out.ovh.net (Postfix) with ESMTP id 49ED8DCAC8 for ; Fri, 24 Feb 2017 11:20:42 +0100 (CET) Received: from zorba.kaod.org.com (LFbn-1-10647-27.w90-89.abo.wanadoo.fr [90.89.233.27]) (Authenticated sender: clg@kaod.org) by player734.ha.ovh.net (Postfix) with ESMTPSA id 6CE23280070; Fri, 24 Feb 2017 11:20:35 +0100 (CET) From: =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= To: David Gibson Date: Fri, 24 Feb 2017 11:18:21 +0100 Message-Id: <1487931503-10607-23-git-send-email-clg@kaod.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1487931503-10607-1-git-send-email-clg@kaod.org> References: <1487931503-10607-1-git-send-email-clg@kaod.org> MIME-Version: 1.0 X-Ovh-Tracer-Id: 646829499064159206 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrfeelhedrvddtgdduvdcutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfqggfjpdevjffgvefmvefgnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 178.32.96.204 Subject: [Qemu-devel] [PATCH v3 22/24] ppc/xics: remove the 'xics' backlinks X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" These are not used anymore. Do the cleanups Signed-off-by: Cédric Le Goater --- hw/intc/xics.c | 28 ---------------------------- hw/intc/xics_kvm.c | 10 ---------- hw/ppc/spapr.c | 3 --- include/hw/ppc/xics.h | 3 --- 4 files changed, 44 deletions(-) diff --git a/hw/intc/xics.c b/hw/intc/xics.c index 571063a67125..565b5ec0686a 100644 --- a/hw/intc/xics.c +++ b/hw/intc/xics.c @@ -360,23 +360,6 @@ static void icp_reset(DeviceState *dev) qemu_set_irq(icp->output, 0); } -static void icp_realize(DeviceState *dev, Error **errp) -{ - ICPState *icp = ICP(dev); - Object *obj; - Error *err = NULL; - - obj = object_property_get_link(OBJECT(dev), "xics", &err); - if (!obj) { - error_setg(errp, "%s: required link 'xics' not found: %s", - __func__, error_get_pretty(err)); - return; - } - - icp->xics = XICS_COMMON(obj); -} - - static void icp_class_init(ObjectClass *klass, void *data) { DeviceClass *dc = DEVICE_CLASS(klass); @@ -384,7 +367,6 @@ static void icp_class_init(ObjectClass *klass, void *data) dc->reset = icp_reset; dc->vmsd = &vmstate_icp_server; - dc->realize = icp_realize; ic->print_info = icp_pic_print_info; } @@ -634,16 +616,6 @@ static void ics_simple_initfn(Object *obj) static void ics_simple_realize(DeviceState *dev, Error **errp) { ICSState *ics = ICS_SIMPLE(dev); - Object *obj; - Error *err = NULL; - - obj = object_property_get_link(OBJECT(dev), "xics", &err); - if (!obj) { - error_setg(errp, "%s: required link 'xics' not found: %s", - __func__, error_get_pretty(err)); - return; - } - ics->xics = XICS_COMMON(obj); if (!ics->nr_irqs) { error_setg(errp, "Number of interrupts needs to be greater 0"); diff --git a/hw/intc/xics_kvm.c b/hw/intc/xics_kvm.c index 850777eab913..7c1809873c29 100644 --- a/hw/intc/xics_kvm.c +++ b/hw/intc/xics_kvm.c @@ -323,16 +323,6 @@ static void ics_kvm_reset(DeviceState *dev) static void ics_kvm_realize(DeviceState *dev, Error **errp) { ICSState *ics = ICS_SIMPLE(dev); - Object *obj; - Error *err = NULL; - - obj = object_property_get_link(OBJECT(dev), "xics", &err); - if (!obj) { - error_setg(errp, "%s: required link 'xics' not found: %s", - __func__, error_get_pretty(err)); - return; - } - ics->xics = XICS_COMMON(obj); if (!ics->nr_irqs) { error_setg(errp, "Number of interrupts needs to be greater 0"); diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 032c8ab008d8..cfd0139dd418 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -115,8 +115,6 @@ static XICSState *try_create_xics(sPAPRMachineState *spapr, qdev_set_parent_bus(DEVICE(spapr->ics), sysbus_get_default()); object_property_add_child(OBJECT(spapr), "ics", OBJECT(spapr->ics), NULL); object_property_set_int(OBJECT(spapr->ics), nr_irqs, "nr-irqs", &err); - object_property_add_const_link(OBJECT(spapr->ics), "xics", OBJECT(xics), - NULL); object_property_set_bool(OBJECT(spapr->ics), true, "realized", &local_err); error_propagate(&err, local_err); if (err) { @@ -132,7 +130,6 @@ static XICSState *try_create_xics(sPAPRMachineState *spapr, object_initialize(icp, sizeof(*icp), type_icp); qdev_set_parent_bus(DEVICE(icp), sysbus_get_default()); object_property_add_child(OBJECT(spapr), "icp[*]", OBJECT(icp), NULL); - object_property_add_const_link(OBJECT(icp), "xics", OBJECT(xics), NULL); object_property_set_bool(OBJECT(icp), true, "realized", &err); if (err) { goto error; diff --git a/include/hw/ppc/xics.h b/include/hw/ppc/xics.h index 1ea8d8139678..d1d110051c72 100644 --- a/include/hw/ppc/xics.h +++ b/include/hw/ppc/xics.h @@ -110,8 +110,6 @@ struct ICPState { uint8_t mfrr; qemu_irq output; bool cap_irq_xics_enabled; - - XICSState *xics; }; #define TYPE_ICS_BASE "ics-base" @@ -147,7 +145,6 @@ struct ICSState { uint32_t offset; qemu_irq *qirqs; ICSIRQState *irqs; - XICSState *xics; }; static inline bool ics_valid_irq(ICSState *ics, uint32_t nr)