[coreboot-gerrit] New patch to review for coreboot: soc/lowrisc: Place CBMEM at top of autodetected RAM

[Jonathan Neuschäfer]

Jonathan Neuschäfer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17708

-gerrit

commit 8ad15834db54b90bd140035870435531e56f3a4d
Author: Jonathan Neuschäfer 
Date:   Thu Nov 24 08:26:03 2016 +0100

soc/lowrisc: Place CBMEM at top of autodetected RAM

Change-Id: I9f169db330d1792128db7a6308ed3afbe5803c03
    Signed-off-by: Jonathan Neuschäfer 
---
 src/soc/lowrisc/lowrisc/cbmem.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/soc/lowrisc/lowrisc/cbmem.c b/src/soc/lowrisc/lowrisc/cbmem.c
index ea7ad70..44ffa26 100644
--- a/src/soc/lowrisc/lowrisc/cbmem.c
+++ b/src/soc/lowrisc/lowrisc/cbmem.c
@@ -11,13 +11,15 @@
  * GNU General Public License for more details.
  */

-#include <console/console.h>
 #include <cbmem.h>
+#include <commonlib/configstring.h>

void *cbmem_top(void)
 {
-	// TODO: find out how the lowrisc SOC decides to provide
-	// this information, when they know.
-	printk(BIOS_SPEW, "Returning hard-coded 128M; fix me\n");
-	return (void *)((uintptr_t)(2ULL*GiB+128*MiB));
+	uintptr_t base;
+	size_t size;
+
+	query_mem(configstring(), &base, &size);
+
+	return (void *)(base + size);
 }