Attention is currently required from: Alper Nebi Yasak, ron minnich.
Philipp Hug has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80378?usp=email )
Change subject: mainboard/qemu-riscv: Add PCI support ......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/emulation/qemu-riscv/include/mainboard/addressmap.h:
https://review.coreboot.org/c/coreboot/+/80378/comment/703e2a04_c053e7d9 : PS1, Line 15: #define QEMU_VIRT_PCIE_HIGH_MMIO_BASE 0x300000000ULL this is correct for rv32, but not for rv64, the base depends on the selected amount of memory. I think this would only work with memory size set to 10GB
#define VIRT32_HIGH_PCIE_MMIO_BASE 0x300000000ULL
/* PCIe high mmio for RV64, size is fixed but base depends on top of RAM */ #define VIRT64_HIGH_PCIE_MMIO_SIZE (16 * GiB)
also in virt_machine_init() virt_high_pcie_memmap.base = memmap[VIRT_DRAM].base + machine->ram_size;