On Mon, Aug 20, 2018 at 5:02 AM, Emanuele Giuseppe Esposito <e.emanuelegiuse...@gmail.com> wrote: > Add xlnx-zcu102 machine to the graph. This machine contains generic-sdhci, so > its constructor must take care of setting it properly when called. > > Signed-off-by: Emanuele Giuseppe Esposito <e.emanuelegiuse...@gmail.com>
Acked-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > tests/Makefile.include | 1 + > tests/libqos/xlnx-zcu102-machine.c | 83 ++++++++++++++++++++++++++++++ > tests/sdhci-test.c | 51 ------------------ > 3 files changed, 84 insertions(+), 51 deletions(-) > create mode 100644 tests/libqos/xlnx-zcu102-machine.c > > diff --git a/tests/Makefile.include b/tests/Makefile.include > index d559218b1c..bad4de3d61 100644 > --- a/tests/Makefile.include > +++ b/tests/Makefile.include > @@ -765,6 +765,7 @@ libqgraph-arm-machines-obj-y += > tests/libqos/xilinx-zynq-a9-machine.o > libqgraph-machines-obj-y = $(libqgraph-arm-machines-obj-y) > libqgraph-machines-obj-y += tests/libqos/x86_64_pc-machine.o > libqgraph-machines-obj-y += tests/libqos/ppc64_pseries-machine.o > +libqgraph-machines-obj-y += tests/libqos/xlnx-zcu102-machine.o > > libqgraph-virtio-obj-y = tests/libqos/virtio-serial.o > libqgraph-virtio-obj-y += tests/libqos/virtio-9p.o > diff --git a/tests/libqos/xlnx-zcu102-machine.c > b/tests/libqos/xlnx-zcu102-machine.c > new file mode 100644 > index 0000000000..9932cf39a8 > --- /dev/null > +++ b/tests/libqos/xlnx-zcu102-machine.c > @@ -0,0 +1,83 @@ > +/* > + * libqos driver framework > + * > + * Copyright (c) 2018 Emanuele Giuseppe Esposito > <e.emanuelegiuse...@gmail.com> > + * > + * This library is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License version 2 as published by the Free Software Foundation. > + * > + * This library is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this library; if not, see > <http://www.gnu.org/licenses/> > + */ > + > +#include "qemu/osdep.h" > +#include "libqtest.h" > +#include "libqos/malloc.h" > +#include "libqos/qgraph.h" > +#include "sdhci.h" > + > +typedef struct QXlnx_zcu102Machine QXlnx_zcu102Machine; > + > +struct QXlnx_zcu102Machine { > + QOSGraphObject obj; > + QGuestAllocator *alloc; > + QSDHCI_MemoryMapped sdhci; > +}; > + > +static void xlnx_zcu102_destructor(QOSGraphObject *obj) > +{ > + g_free(obj); > +} > + > +static void *xlnx_zcu102_get_driver(void *object, const char *interface) > +{ > + QXlnx_zcu102Machine *machine = object; > + if (!g_strcmp0(interface, "memory")) { > + return &machine->alloc; > + } > + > + fprintf(stderr, "%s not present in aarch64/xlnx-zcu102\n", interface); > + g_assert_not_reached(); > +} > + > +static QOSGraphObject *xlnx_zcu102_get_device(void *obj, const char *device) > +{ > + QXlnx_zcu102Machine *machine = obj; > + if (!g_strcmp0(device, "generic-sdhci")) { > + return &machine->sdhci.obj; > + } > + > + fprintf(stderr, "%s not present in aarch64/xlnx-zcu102\n", device); > + g_assert_not_reached(); > +} > + > +static void *qos_create_machine_aarch64_xlnx_zcu102(void) > +{ > + QXlnx_zcu102Machine *machine = g_new0(QXlnx_zcu102Machine, 1); > + > + machine->obj.get_device = xlnx_zcu102_get_device; > + machine->obj.get_driver = xlnx_zcu102_get_driver; > + machine->obj.destructor = xlnx_zcu102_destructor; > + /* Datasheet: UG1085 (v1.7) */ > + qos_init_sdhci_smm(&machine->sdhci, 0xff160000, &(QSDHCIProperties) { > + .version = 3, > + .baseclock = 0, > + .capab.sdma = true, > + .capab.reg = 0x280737ec6481 > + }); > + return &machine->obj; > +} > + > +static void xlnx_zcu102_register_nodes(void) > +{ > + qos_node_create_machine("aarch64/xlnx-zcu102", > qos_create_machine_aarch64_xlnx_zcu102); > + qos_node_contains("aarch64/xlnx-zcu102", "generic-sdhci", NULL); > +} > + > +libqos_init(xlnx_zcu102_register_nodes); > diff --git a/tests/sdhci-test.c b/tests/sdhci-test.c > index 0a8efd8554..deb42be493 100644 > --- a/tests/sdhci-test.c > +++ b/tests/sdhci-test.c > @@ -22,57 +22,6 @@ FIELD(SDHC_CAPAB, SDR, 32, 3); /* > since v3 */ > FIELD(SDHC_CAPAB, DRIVER, 36, 3); /* since v3 */ > #define SDHC_HCVER 0xFE > > -/** > - * Old sdhci_t structure: > - * > - struct sdhci_t { > - const char *arch, *machine; > - struct { > - uintptr_t addr; > - uint8_t version; > - uint8_t baseclock; > - struct { > - bool sdma; > - uint64_t reg; > - } capab; > - } sdhci; > - struct { > - uint16_t vendor_id, device_id; > - } pci; > - } > - * > - * implemented drivers: > - * > - PC via PCI > - { "x86_64", "pc", > - {-1, 2, 0, {1, 0x057834b4} }, > - .pci = { PCI_VENDOR_ID_REDHAT, PCI_DEVICE_ID_REDHAT_SDHCI } }, > - > - BCM2835 > - { "arm", "raspi2", > - {0x3f300000, 3, 52, {0, 0x052134b4} } }, > - > - Exynos4210 > - { "arm", "smdkc210", > - {0x12510000, 2, 0, {1, 0x5e80080} } }, > - > - i.MX 6 > - { "arm", "sabrelite", > - {0x02190000, 3, 0, {1, 0x057834b4} } }, > - > - Zynq-7000 > - { "arm", "xilinx-zynq-a9", Datasheet: UG585 (v1.12.1) > - {0xe0100000, 2, 0, {1, 0x69ec0080} } }, > - > - * > - * FIXME: the following drivers are missing: > - * > - > - ZynqMP > - { "aarch64", "xlnx-zcu102", Datasheet: UG1085 (v1.7) > - {0xff160000, 3, 0, {1, 0x280737ec6481} } }, > - */ > - > static void check_specs_version(QSDHCI *s, uint8_t version) > { > uint32_t v; > -- > 2.17.1 > >