Hello ron minnich, Paul Menzel, build bot (Jenkins), Philipp Hug,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38904
to look at the new patch set (#6).
Change subject: qemu-riscv: Fix discovered DDR size ......................................................................
qemu-riscv: Fix discovered DDR size
qemu-riscv value for DDR is incorrect. It was set to 32768 which in turn is being used as 32768MB (32+GB), instead of the 32MB it is intended to represent.
Change value so it is a select option based in menuconfig based on what you want coreboot to "discover" on boot.
Four options are : 256MB 512MB 1024MB 2048MB
NOTE: You must specify greater than or equal to what is chosen from the above items on the qemu cmdline.
This value is used in both romstage and ramstage to set the amount of DDR present. This will be the above value minus the size of the boot rom.
Fixes: https://ticket.coreboot.org/issues/254
TEST = qemu-system-riscv64 -M virt -m 1024M -nographic \ -bios build/coreboot.rom
TEST = qemu-system-riscv64 -M virt -m 1024M -nographic \ -kernel build/coreboot.elf
Signed-off-by: Marc Karasek mkarasek@cryptocoretech.com Change-Id: Idef44eb8baf3e89d7c74fa452fc60b7beefa2c48 --- M src/mainboard/emulation/qemu-riscv/Kconfig M src/mainboard/emulation/qemu-riscv/mainboard.c M src/soc/ucb/riscv/cbmem.c 3 files changed, 36 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/38904/6