Marc Karasek has uploaded this change for review.

View Change

Fixed configuration for DDR Size discovered by Coreboot

For qemu-riscv value was incorrect. It was set to 32768 which
in turn was being used as 32768MB (32+GB), instead of the 32MB
it was intended to represent.

Change value so it is a select option based on in menuconfig
based on what you want coreboot to "discover" on boot.

Current three options are :
128MB
256MB
512MB

NOTE: You must specify greater than what is chosen from
the above items on the qemu cmdline.

Signed-off-by: Marc Karasek <mkarasek@cryptocoretech.com>
Change-Id: Idef44eb8baf3e89d7c74fa452fc60b7beefa2c48
---
M src/mainboard/emulation/qemu-riscv/Kconfig
1 file changed, 27 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/38904/1
diff --git a/src/mainboard/emulation/qemu-riscv/Kconfig b/src/mainboard/emulation/qemu-riscv/Kconfig
index 4d4c900..2a119a3 100644
--- a/src/mainboard/emulation/qemu-riscv/Kconfig
+++ b/src/mainboard/emulation/qemu-riscv/Kconfig
@@ -54,9 +54,35 @@
int
default 1

+choice
+ prompt "DRAM SIZE MB"
+ help
+ Select the size of the DDR coreboot will find.
+ Note: You must specify a -m <size> greater than
+ what is chosen here.
+
+ config COREBOOT_DDR_SIZE_128M
+ bool "128 MB"
+ help
+ Choose this option if you want coreboot to find 127MB DDR
+
+ config COREBOOT_DDR_SIZE_256M
+ bool "256 MB"
+ help
+ Choose this option if you want coreboot to find 255MB DDR
+
+ config COREBOOT_DDR_SIZE_512M
+ bool "512 MB"
+ help
+ Choose this option if you want coreboot to find 512MB DDR
+endchoice
+
config DRAM_SIZE_MB
int
- default 32768
+ default 32
+ default 32 if COREBOOT_DDR_SIZE_128M
+ default 64 if COREBOOT_DDR_SIZE_256M
+ default 128 if COREBOOT_DDR_SIZE_512M

config OPENSBI_PLATFORM
string

To view, visit change 38904. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Idef44eb8baf3e89d7c74fa452fc60b7beefa2c48
Gerrit-Change-Number: 38904
Gerrit-PatchSet: 1
Gerrit-Owner: Marc Karasek <marckarasek@gmail.com>
Gerrit-MessageType: newchange