[coreboot-gerrit] Change in coreboot[master]: lib/imd_cbmem.c: Add a helper function to indicate that cbmem is ready

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38736 )

Change subject: lib/imd_cbmem.c: Add a helper function to indicate that cbmem is ready
......................................................................

lib/imd_cbmem.c: Add a helper function to indicate that cbmem is ready

This can be used in romstage in particular to know if dram is ready.

Change-Id: I0231ab9c0b78a69faa762e0a97378bf0b50eebaf
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38736
Tested-by: build bot (Jenkins) 
Reviewed-by: Julius Werner 
Reviewed-by: Angel Pons 
---
M src/include/cbmem.h
M src/lib/imd_cbmem.c
2 files changed, 20 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Julius Werner: Looks good to me, approved
  Angel Pons: Looks good to me, approved

diff --git a/src/include/cbmem.h b/src/include/cbmem.h
index b548cd9..7e1da17 100644
--- a/src/include/cbmem.h
+++ b/src/include/cbmem.h
@@ -155,4 +155,18 @@
    return 1;
 }

+/* Returns 1 after running cbmem init hooks, 0 otherwise. */
+static inline int cbmem_online(void)
+{
+	extern int cbmem_initialized;
+
+	if (!cbmem_possibly_online())
+		return 0;
+
+	if (ENV_ROMSTAGE)
+		return cbmem_initialized;
+
+	return 1;
+}
+
 #endif /* _CBMEM_H_ */
diff --git a/src/lib/imd_cbmem.c b/src/lib/imd_cbmem.c
index 7638e0e..4b7c412 100644
--- a/src/lib/imd_cbmem.c
+++ b/src/lib/imd_cbmem.c
@@ -30,6 +30,8 @@
    dead_code();
 }

+int cbmem_initialized;
+
 static inline const struct cbmem_entry *imd_to_cbmem(const struct imd_entry *e)
 {
    return (const struct cbmem_entry *)e;
@@ -79,6 +81,8 @@

/* Complete migration to CBMEM. */
    cbmem_run_init_hooks(no_recovery);
+
+	cbmem_initialized = 1;
 }

int cbmem_initialize(void)
@@ -112,6 +116,8 @@
    /* Complete migration to CBMEM. */
    cbmem_run_init_hooks(recovery);

+	cbmem_initialized = 1;
+
    /* Recovery successful. */
    return 0;
 }