[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 2/5] hw/riscv/virt: Connect the gpex PCIe
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v6 2/5] hw/riscv/virt: Connect the gpex PCIe |
Date: |
Mon, 5 Nov 2018 11:47:17 -0800 |
On Mon, Nov 5, 2018 at 5:24 AM Bin Meng <address@hidden> wrote:
>
> Hi,
>
> On Wed, Oct 31, 2018 at 6:22 AM Alistair Francis
> <address@hidden> wrote:
> >
> > Connect the gpex PCIe device based on the device tree included in the
> > HiFive Unleashed ROM.
> >
> > Signed-off-by: Alistair Francis <address@hidden>
> > ---
> > default-configs/riscv32-softmmu.mak | 6 +-
> > default-configs/riscv64-softmmu.mak | 6 +-
> > hw/riscv/virt.c | 111 ++++++++++++++++++++++++++++
> > include/hw/riscv/virt.h | 8 +-
> > 4 files changed, 127 insertions(+), 4 deletions(-)
> >
> > diff --git a/default-configs/riscv32-softmmu.mak
> > b/default-configs/riscv32-softmmu.mak
> > index 7937c69e22..3e3d195f37 100644
> > --- a/default-configs/riscv32-softmmu.mak
> > +++ b/default-configs/riscv32-softmmu.mak
> > @@ -1,7 +1,11 @@
> > # Default configuration for riscv-softmmu
> >
> > +include pci.mak
> > +
> > CONFIG_SERIAL=y
> > CONFIG_VIRTIO_MMIO=y
> > -include virtio.mak
> >
> > CONFIG_CADENCE=y
> > +
> > +CONFIG_PCI_GENERIC=y
> > +CONFIG_PCI_XILINX=y
> > diff --git a/default-configs/riscv64-softmmu.mak
> > b/default-configs/riscv64-softmmu.mak
> > index 7937c69e22..3e3d195f37 100644
> > --- a/default-configs/riscv64-softmmu.mak
> > +++ b/default-configs/riscv64-softmmu.mak
> > @@ -1,7 +1,11 @@
> > # Default configuration for riscv-softmmu
> >
> > +include pci.mak
> > +
> > CONFIG_SERIAL=y
> > CONFIG_VIRTIO_MMIO=y
> > -include virtio.mak
> >
> > CONFIG_CADENCE=y
> > +
> > +CONFIG_PCI_GENERIC=y
> > +CONFIG_PCI_XILINX=y
> > diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> > index 4a137a503c..2fbe58ba4b 100644
> > --- a/hw/riscv/virt.c
> > +++ b/hw/riscv/virt.c
> > @@ -39,6 +39,8 @@
> > #include "sysemu/arch_init.h"
> > #include "sysemu/device_tree.h"
> > #include "exec/address-spaces.h"
> > +#include "hw/pci/pci.h"
> > +#include "hw/pci-host/gpex.h"
> > #include "elf.h"
> >
> > #include <libfdt.h>
> > @@ -55,6 +57,10 @@ static const struct MemmapEntry {
> > [VIRT_UART0] = { 0x10000000, 0x100 },
> > [VIRT_VIRTIO] = { 0x10001000, 0x1000 },
> > [VIRT_DRAM] = { 0x80000000, 0x0 },
> > + [VIRT_PCIE_MMIO] = { 0x2000000000, 0x4000000 },
>
> Does this work with RV32?
That's a good point, probably not. This is based on the HiFive
unleashed values to be as similar as possible.
>
> > + [VIRT_PCIE_PIO] = { 0x2010000, 0x40000000 },
> > + [VIRT_PCIE_ECAM] = { 0x40000000, 0x20000000 },
> > +
> > };
> >
> > static uint64_t load_kernel(const char *kernel_filename)
> > @@ -98,6 +104,37 @@ static hwaddr load_initrd(const char *filename,
> > uint64_t mem_size,
> > return *start + size;
> > }
> >
> > +#define INTERREUPT_MAP_WIDTH 7
> > +
> > +static void create_pcie_irq_map(void *fdt, char *nodename,
> > + uint32_t plic_phandle)
> > +{
> > + int pin;
> > + uint32_t full_irq_map[GPEX_NUM_IRQS * INTERREUPT_MAP_WIDTH] = { 0 };
> > + uint32_t *irq_map = full_irq_map;
> > +
> > + for (pin = 0; pin < GPEX_NUM_IRQS; pin++) {
> > + int irq_nr = PCIE_IRQ + (pin % PCI_NUM_PINS);
> > + int i;
> > +
> > + uint32_t map[] = {
> > + 0, 0, 0,
> > + pin + 1, plic_phandle, 0, irq_nr};
> > +
> > + /* Convert map to big endian */
> > + for (i = 0; i < INTERREUPT_MAP_WIDTH; i++) {
> > + irq_map[i] = cpu_to_be32(map[i]);
> > + }
> > + irq_map += INTERREUPT_MAP_WIDTH;
> > + }
> > +
> > + qemu_fdt_setprop(fdt, nodename, "interrupt-map",
> > + full_irq_map, sizeof(full_irq_map));
> > +
> > + qemu_fdt_setprop_cells(fdt, nodename, "interrupt-map-mask",
> > + 0, 0, 0, 0x7);
> > +}
> > +
> > static void *create_fdt(RISCVVirtState *s, const struct MemmapEntry
> > *memmap,
> > uint64_t mem_size, const char *cmdline)
> > {
> > @@ -233,6 +270,31 @@ static void *create_fdt(RISCVVirtState *s, const
> > struct MemmapEntry *memmap,
> > g_free(nodename);
> > }
> >
> > + nodename = g_strdup_printf("/address@hidden",
> > + (long) memmap[VIRT_PCIE_MMIO].base);
> > + qemu_fdt_add_subnode(fdt, nodename);
> > + qemu_fdt_setprop_cells(fdt, nodename, "#address-cells", 0x3);
> > + qemu_fdt_setprop_cells(fdt, nodename, "#interrupt-cells", 0x1);
> > + qemu_fdt_setprop_cells(fdt, nodename, "#size-cells", 0x2);
> > + qemu_fdt_setprop_string(fdt, nodename, "compatible",
> > + "pci-host-ecam-generic");
> > + qemu_fdt_setprop_string(fdt, nodename, "device_type", "pci");
> > + qemu_fdt_setprop_cell(fdt, nodename, "linux,pci-domain", 0);
> > + qemu_fdt_setprop_cells(fdt, nodename, "bus-range", 0,
> > + memmap[VIRT_PCIE_ECAM].base /
> > + PCIE_MMCFG_SIZE_MIN - 1);
> > + qemu_fdt_setprop(fdt, nodename, "dma-coherent", NULL, 0);
> > + qemu_fdt_setprop_cells(fdt, nodename, "reg", 0x20, 0,
> > + 0, memmap[VIRT_PCIE_ECAM].size);
> > + qemu_fdt_setprop_cells(fdt, nodename, "ranges",
> > + memmap[VIRT_PCIE_PIO].base,
> > + 0, memmap[VIRT_PCIE_PIO].size,
> > + 0, memmap[VIRT_PCIE_MMIO].base,
> > + 0, memmap[VIRT_PCIE_MMIO].size);
>
> This does not conform with the PCI bus ranges encoding.
Do you know what should?
I have tried so many different combinations here and nothing seems to work.
Alistair
>
> > + qemu_fdt_setprop_cells(fdt, nodename, "interrupt-parent",
> > plic_phandle);
> > + qemu_fdt_setprop_cells(fdt, nodename, "interrupts", PCIE_IRQ);
> > + create_pcie_irq_map(fdt, nodename, plic_phandle);
> > +
> > nodename = g_strdup_printf("/address@hidden",
> > (long)memmap[VIRT_TEST].base);
> > qemu_fdt_add_subnode(fdt, nodename);
> > @@ -262,6 +324,47 @@ static void *create_fdt(RISCVVirtState *s, const
> > struct MemmapEntry *memmap,
> > return fdt;
> > }
> >
>
> [snip]
>
> Regards,
> Bin