From patchwork Wed Feb 20 07:26:05 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Crosthwaite X-Patchwork-Id: 222012 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id AA9582C0082 for ; Wed, 20 Feb 2013 20:19:22 +1100 (EST) Received: from localhost ([::1]:41525 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1U845L-000574-MV for incoming@patchwork.ozlabs.org; Wed, 20 Feb 2013 02:27:23 -0500 Received: from eggs.gnu.org ([208.118.235.92]:36974) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1U8450-0004Xo-V7 for qemu-devel@nongnu.org; Wed, 20 Feb 2013 02:27:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1U844x-00038q-NO for qemu-devel@nongnu.org; Wed, 20 Feb 2013 02:27:02 -0500 Received: from mail-pb0-f52.google.com ([209.85.160.52]:41525) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1U844x-00038c-EC for qemu-devel@nongnu.org; Wed, 20 Feb 2013 02:26:59 -0500 Received: by mail-pb0-f52.google.com with SMTP id ma3so2725947pbc.39 for ; Tue, 19 Feb 2013 23:26:58 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:sender:from:to:cc:subject:date:message-id:x-mailer :in-reply-to:references:in-reply-to:references:x-gm-message-state; bh=mqJpMQng8zGEIA4k6D0l7CrX+9EwJ7O2+Lg2vxGeJYY=; b=M9hOoux8rdX6SWsRHRJhhnKKaVdeq0c2j0OOf7rOuRGazUFHLRb8qYf2ovNJHX3psI 9FpyptQ6yb+g65jswiLEIGP/eWD7D66gY7oyabLnYb3+PXVWr2UqETrdBWCyUk2j62a9 uVxw6THVPkZGlFMchntSoR4mymplNuJ9SVatLOMN2ZrMj7byVAvnYtqy7EzltnE2lKLt xKR/7YHDIOZ4B2uKjCQqPFpDsGD1rwCP1B/mg2lV98NbU5Kqpjk9oXaUs/mGWBkrGYsg mo0tF64AUH3jQ7Y0vpuCFknWI7NBTzfuL5tR8G13VimRpRlZ0d8naTeWS8MH2xQ6+M1o IIqw== X-Received: by 10.68.212.135 with SMTP id nk7mr47021284pbc.120.1361345218666; Tue, 19 Feb 2013 23:26:58 -0800 (PST) Received: from localhost ([1.133.225.158]) by mx.google.com with ESMTPS id o5sm110259019pay.5.2013.02.19.23.26.53 (version=TLSv1.1 cipher=RC4-SHA bits=128/128); Tue, 19 Feb 2013 23:26:57 -0800 (PST) From: Peter Crosthwaite To: qemu-devel@nongnu.org Date: Wed, 20 Feb 2013 17:26:05 +1000 Message-Id: <0daeeb5983452f0739314bfccd888674836f2a7f.1361344859.git.peter.crosthwaite@xilinx.com> X-Mailer: git-send-email 1.7.0.4 In-Reply-To: References: In-Reply-To: References: X-Gm-Message-State: ALoCoQlWYDtV9apRzQlzAPpMffcw4wYgf+ZesX3oOdcxZj17Mh37zBwySK7eYYSIJACVPy3fMbdw X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 209.85.160.52 Cc: edgar.iglesias@gmail.com, Peter Crosthwaite , peter.maydell@linaro.org Subject: [Qemu-devel] [PATCH v1 5/5] xilinx_spips: seperate SPI and QSPI as two classes X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Make SPI and QSPI different classes. QSPIPS is setup as a child of SPIPS. Only QSPI has the LQSPI functionality, so move all that to the child class. Signed-off-by: Peter Crosthwaite --- hw/xilinx_spips.c | 66 ++++++++++++++++++++++++++++++++++++++++++---------- hw/xilinx_zynq.c | 2 +- 2 files changed, 54 insertions(+), 14 deletions(-) diff --git a/hw/xilinx_spips.c b/hw/xilinx_spips.c index 530ef47..46883a5 100644 --- a/hw/xilinx_spips.c +++ b/hw/xilinx_spips.c @@ -149,15 +149,23 @@ typedef struct { uint8_t num_txrx_bytes; uint32_t regs[R_MAX]; +} XilinxSPIPS; + +typedef struct { + XilinxSPIPS parent; uint32_t lqspi_buf[LQSPI_CACHE_SIZE]; hwaddr lqspi_cached_addr; -} XilinxSPIPS; +} XilinxQSPIPS; -#define TYPE_XILINX_SPIPS "xilinx,spips" + +#define TYPE_XILINX_SPIPS "xlnx.ps7-spi" +#define TYPE_XILINX_QSPIPS "xlnx.ps7-qspi" #define XILINX_SPIPS(obj) \ OBJECT_CHECK(XilinxSPIPS, (obj), TYPE_XILINX_SPIPS) +#define XILINX_QSPIPS(obj) \ + OBJECT_CHECK(XilinxQSPIPS, (obj), TYPE_XILINX_QSPIPS) static inline int num_effective_busses(XilinxSPIPS *s) { @@ -436,11 +444,12 @@ static uint64_t lqspi_read(void *opaque, hwaddr addr, unsigned int size) { int i; + XilinxQSPIPS *q = opaque; XilinxSPIPS *s = opaque; - if (addr >= s->lqspi_cached_addr && - addr <= s->lqspi_cached_addr + LQSPI_CACHE_SIZE - 4) { - return s->lqspi_buf[(addr - s->lqspi_cached_addr) >> 2]; + if (addr >= q->lqspi_cached_addr && + addr <= q->lqspi_cached_addr + LQSPI_CACHE_SIZE - 4) { + return q->lqspi_buf[(addr - q->lqspi_cached_addr) >> 2]; } else { int flash_addr = (addr / num_effective_busses(s)); int slave = flash_addr >> LQSPI_ADDRESS_BITS; @@ -484,14 +493,14 @@ lqspi_read(void *opaque, hwaddr addr, unsigned int size) for (i = 0; i < LQSPI_CACHE_SIZE / 4; ++i) { tx_data_bytes(s, 0, 4); xilinx_spips_flush_txfifo(s); - rx_data_bytes(s, &s->lqspi_buf[cache_entry], 4); + rx_data_bytes(s, &q->lqspi_buf[cache_entry], 4); cache_entry++; } s->regs[R_CONFIG] |= CS; xilinx_spips_update_cs_lines(s); - s->lqspi_cached_addr = addr; + q->lqspi_cached_addr = addr; return lqspi_read(opaque, addr, size); } } @@ -511,7 +520,7 @@ static void xilinx_spips_realize(DeviceState *dev, Error **errp) SysBusDevice *sbd = SYS_BUS_DEVICE(dev); int i; - DB_PRINT("inited device model\n"); + DB_PRINT("realized spips\n"); s->spi = g_new(SSIBus *, s->num_busses); for (i = 0; i < s->num_busses; ++i) { @@ -531,17 +540,32 @@ static void xilinx_spips_realize(DeviceState *dev, Error **errp) memory_region_init_io(&s->iomem, &spips_ops, s, "spi", R_MAX*4); sysbus_init_mmio(sbd, &s->iomem); - memory_region_init_io(&s->mmlqspi, &lqspi_ops, s, "lqspi", - (1 << LQSPI_ADDRESS_BITS) * 2); - sysbus_init_mmio(sbd, &s->mmlqspi); - s->irqline = -1; - s->lqspi_cached_addr = ~0ULL; fifo8_create(&s->rx_fifo, RXFF_A); fifo8_create(&s->tx_fifo, TXFF_A); } +static void xilinx_qspips_realize(DeviceState *dev, Error **errp) +{ + XilinxSPIPS *s = XILINX_SPIPS(dev); + XilinxQSPIPS *q = XILINX_SPIPS(dev); + SysBusDevice *sbd = SYS_BUS_DEVICE(dev); + + DB_PRINT("realized qspips\n"); + + s->num_busses = 2; + s->num_cs = 2; + s->num_txrx_bytes = 4; + + xilinx_spips_realize(dev, errp); + memory_region_init_io(&s->mmlqspi, &lqspi_ops, s, "lqspi", + (1 << LQSPI_ADDRESS_BITS) * 2); + sysbus_init_mmio(sbd, &s->mmlqspi); + + q->lqspi_cached_addr = ~0ULL; +} + static int xilinx_spips_post_load(void *opaque, int version_id) { xilinx_spips_update_ixr((XilinxSPIPS *)opaque); @@ -570,6 +594,14 @@ static Property xilinx_spips_properties[] = { DEFINE_PROP_UINT8("num-txrx-bytes", XilinxSPIPS, num_txrx_bytes, 1), DEFINE_PROP_END_OF_LIST(), }; + +static void xilinx_qspips_class_init(ObjectClass *klass, void * data) +{ + DeviceClass *dc = DEVICE_CLASS(klass); + + dc->realize = xilinx_qspips_realize; +} + static void xilinx_spips_class_init(ObjectClass *klass, void *data) { DeviceClass *dc = DEVICE_CLASS(klass); @@ -587,9 +619,17 @@ static const TypeInfo xilinx_spips_info = { .class_init = xilinx_spips_class_init, }; +static const TypeInfo xilinx_qspips_info = { + .name = TYPE_XILINX_QSPIPS, + .parent = TYPE_XILINX_SPIPS, + .instance_size = sizeof(XilinxQSPIPS), + .class_init = xilinx_qspips_class_init, +}; + static void xilinx_spips_register_types(void) { type_register_static(&xilinx_spips_info); + type_register_static(&xilinx_qspips_info); } type_init(xilinx_spips_register_types) diff --git a/hw/xilinx_zynq.c b/hw/xilinx_zynq.c index 311f791..a4909f4 100644 --- a/hw/xilinx_zynq.c +++ b/hw/xilinx_zynq.c @@ -62,7 +62,7 @@ static inline void zynq_init_spi_flashes(uint32_t base_addr, qemu_irq irq, int num_busses = is_qspi ? NUM_QSPI_BUSSES : 1; int num_ss = is_qspi ? NUM_QSPI_FLASHES : NUM_SPI_FLASHES; - dev = qdev_create(NULL, "xilinx,spips"); + dev = qdev_create(NULL, is_qspi ? "xlnx.ps7-qspi" : "xlnx.ps7-spi"); qdev_prop_set_uint8(dev, "num-txrx-bytes", is_qspi ? 4 : 1); qdev_prop_set_uint8(dev, "num-ss-bits", num_ss); qdev_prop_set_uint8(dev, "num-busses", num_busses);