From patchwork Fri Feb 15 11:45:04 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Peter Maydell X-Patchwork-Id: 14845 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id 46F2423EAD for ; Fri, 15 Feb 2013 11:45:15 +0000 (UTC) Received: from mail-ve0-f171.google.com (mail-ve0-f171.google.com [209.85.128.171]) by fiordland.canonical.com (Postfix) with ESMTP id 03F5BA18892 for ; Fri, 15 Feb 2013 11:45:14 +0000 (UTC) Received: by mail-ve0-f171.google.com with SMTP id b10so2951835vea.2 for ; Fri, 15 Feb 2013 03:45:14 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:x-forwarded-to:x-forwarded-for:delivered-to:x-received :received-spf:from:to:cc:subject:date:message-id:x-mailer :in-reply-to:references:x-gm-message-state; bh=YH9dM56845rVwW9XBqhgY+16JjcPtyIGwn8sr01VNeI=; b=NwSRnb18w8L4QzyhN+VVZBM0pchc9C1d77kKxKB9P+eonw7AsbGUmbj141pmJS6Idr Nb1BsCWW8a0Gqr5ljkelHj2vr6++RnekUVLerob4FRE0jq15FOnlqqKQAw1RtPM7LmQ/ lfnK6ieRswec7RIfy/8aNsknWYL+82opTiXxta5KnzXSsYR3HUt20cUXs0H9NiGN7M2k yr8Fjsyf0bbUGe/56oBU6Je6sLQ1EQERjM1z+m9CCd3HDOKMKeS7rmVGbtAJHZF+sxCO 69c20sL+V1da/nNbG0ks9MlQ7SIAPl0t2D+W4v+3GfFxBNVg6W7RjzHm+7rip+XUZFjs zC/Q== X-Received: by 10.52.38.163 with SMTP id h3mr2417658vdk.35.1360928714448; Fri, 15 Feb 2013 03:45:14 -0800 (PST) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.221.4.5 with SMTP id oa5csp8012vcb; Fri, 15 Feb 2013 03:45:13 -0800 (PST) X-Received: by 10.68.232.38 with SMTP id tl6mr5097740pbc.153.1360928710497; Fri, 15 Feb 2013 03:45:10 -0800 (PST) Received: from mnementh.archaic.org.uk (1.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.d.1.0.0.b.8.0.1.0.0.2.ip6.arpa. [2001:8b0:1d0::1]) by mx.google.com with ESMTPS id nq5si4351870pbc.56.2013.02.15.03.45.09 (version=TLSv1 cipher=RC4-SHA bits=128/128); Fri, 15 Feb 2013 03:45:10 -0800 (PST) Received-SPF: neutral (google.com: 2001:8b0:1d0::1 is neither permitted nor denied by best guess record for domain of pm215@archaic.org.uk) client-ip=2001:8b0:1d0::1; Authentication-Results: mx.google.com; spf=neutral (google.com: 2001:8b0:1d0::1 is neither permitted nor denied by best guess record for domain of pm215@archaic.org.uk) smtp.mail=pm215@archaic.org.uk Received: from pm215 by mnementh.archaic.org.uk with local (Exim 4.72) (envelope-from ) id 1U6Jj0-0003Os-Ev; Fri, 15 Feb 2013 11:45:06 +0000 From: Peter Maydell To: qemu-devel@nongnu.org Cc: patches@linaro.org, Michael Walle , Jan Kiszka Subject: [PATCH 3/5] milkymist-minimac2: Just expose buffers as a sysbus mmio region Date: Fri, 15 Feb 2013 11:45:04 +0000 Message-Id: <1360928706-13041-4-git-send-email-peter.maydell@linaro.org> X-Mailer: git-send-email 1.7.2.5 In-Reply-To: <1360928706-13041-1-git-send-email-peter.maydell@linaro.org> References: <1360928706-13041-1-git-send-email-peter.maydell@linaro.org> X-Gm-Message-State: ALoCoQnf9Zwe1hU+tHLvm9VAlKYjYrASOr5XEWZnwSaq6Elj5UNEHLPD4wHwBhFkdk4bvzmBT6Mz Just expose the register buffers memory as a standard sysbus mmio region which the creator of the device can map, rather than providing a qdev property which the creator has to set to the base address and then doing the mapping in the device's own init function. Signed-off-by: Peter Maydell Reviewed-by: Andreas Färber Acked-by: Michael Walle --- hw/milkymist-hw.h | 2 +- hw/milkymist-minimac2.c | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/hw/milkymist-hw.h b/hw/milkymist-hw.h index c8bd7e9..21e202b 100644 --- a/hw/milkymist-hw.h +++ b/hw/milkymist-hw.h @@ -193,10 +193,10 @@ static inline DeviceState *milkymist_minimac2_create(hwaddr base, qemu_check_nic_model(&nd_table[0], "minimac2"); dev = qdev_create(NULL, "milkymist-minimac2"); - qdev_prop_set_taddr(dev, "buffers_base", buffers_base); qdev_set_nic_properties(dev, &nd_table[0]); qdev_init_nofail(dev); sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, base); + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 1, buffers_base); sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, rx_irq); sysbus_connect_irq(SYS_BUS_DEVICE(dev), 1, tx_irq); diff --git a/hw/milkymist-minimac2.c b/hw/milkymist-minimac2.c index 9992dcc..b462e90 100644 --- a/hw/milkymist-minimac2.c +++ b/hw/milkymist-minimac2.c @@ -96,7 +96,6 @@ struct MilkymistMinimac2State { NICState *nic; NICConf conf; char *phy_model; - hwaddr buffers_base; MemoryRegion buffers; MemoryRegion regs_region; @@ -475,7 +474,7 @@ static int milkymist_minimac2_init(SysBusDevice *dev) s->rx1_buf = s->rx0_buf + MINIMAC2_BUFFER_SIZE; s->tx_buf = s->rx1_buf + MINIMAC2_BUFFER_SIZE; - sysbus_add_memory(dev, s->buffers_base, &s->buffers); + sysbus_init_mmio(dev, &s->buffers); qemu_macaddr_default_if_unset(&s->conf.macaddr); s->nic = qemu_new_nic(&net_milkymist_minimac2_info, &s->conf, @@ -517,8 +516,6 @@ static const VMStateDescription vmstate_milkymist_minimac2 = { }; static Property milkymist_minimac2_properties[] = { - DEFINE_PROP_TADDR("buffers_base", MilkymistMinimac2State, - buffers_base, 0), DEFINE_NIC_PROPERTIES(MilkymistMinimac2State, conf), DEFINE_PROP_STRING("phy_model", MilkymistMinimac2State, phy_model), DEFINE_PROP_END_OF_LIST(),