[coreboot-gerrit] Change in coreboot[master]: lib/ramtest: Adjust address for RAM test

Werner Zeh (Code Review) gerrit at coreboot.org
Mon Oct 15 08:02:26 CEST 2018


Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/29106


Change subject: lib/ramtest: Adjust address for RAM test
......................................................................

lib/ramtest: Adjust address for RAM test

In patch 83bd46e (selfboot: remove bounce buffers) the address for
RAMBASE has been changed from 1 MiB to 14 MiB. As the address for the
quick RAM test was unfortunately hard coded it was overseen in the
mentioned patch.
Use now the value of CONFIG_RAMBASE for the quick RAM test so that the
address that will be used to load the ramstage is tested.

Change-Id: Ibc57e684a677ac64225ea54cb43e6fbb68930354
Signed-off-by: Werner Zeh <werner.zeh at siemens.com>
---
M src/lib/ramtest.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/29106/1

diff --git a/src/lib/ramtest.c b/src/lib/ramtest.c
index 461a028..1344c95 100644
--- a/src/lib/ramtest.c
+++ b/src/lib/ramtest.c
@@ -214,5 +214,5 @@
 
 void quick_ram_check(void)
 {
-	__quick_ram_check(0x100000);
+	__quick_ram_check(CONFIG_RAMBASE);
 }

-- 
To view, visit https://review.coreboot.org/29106
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc57e684a677ac64225ea54cb43e6fbb68930354
Gerrit-Change-Number: 29106
Gerrit-PatchSet: 1
Gerrit-Owner: Werner Zeh <werner.zeh at siemens.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181015/d1e71f34/attachment.html>


More information about the coreboot-gerrit mailing list