[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: |
Bin Meng |
Subject: |
Re: [Qemu-devel] [PATCH v6 2/5] hw/riscv/virt: Connect the gpex PCIe |
Date: |
Tue, 6 Nov 2018 14:45:36 +0800 |
Hi Alistair,
On Tue, Nov 6, 2018 at 3:47 AM Alistair Francis <address@hidden> wrote:
>
> 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.
>
Please specifying a 32-bit address to make it work for both 32-bit and 64-bit.
> >
> > > + [VIRT_PCIE_PIO] = { 0x2010000, 0x40000000 },
> > > + [VIRT_PCIE_ECAM] = { 0x40000000, 0x20000000 },
Forgot to mention: the maximum size of ECAM is 0x10000000 by spec.
> > > +
> > > };
> > >
> > > 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.
>
qemu_fdt_setprop_sized_cells(fdt, nodename, "ranges",
1, FDT_PCI_RANGE_IOPORT, 2, 0,
2, memmap[VIRT_PCIE_PIO].base, 2, memmap[VIRT_PCIE_PIO].size,
1, FDT_PCI_RANGE_MMIO_64BIT,
2, memmap[VIRT_PCIE_MMIO].base,
2, memmap[VIRT_PCIE_MMIO].base, 2, memmap[VIRT_PCIE_MMIO].size);
Note if we are using 32-bit address for the MMIO,
FDT_PCI_RANGE_MMIO_64BIT should be FDT_PCI_RANGE_MMIO.
Regards,
Bin