[RFC PATCH 3/4] lmb: Split init into two functions
Simon Glass
sjg at chromium.org
Tue Jul 30 16:40:34 CEST 2024
The normal case for initing lmb is to set up all the memory regions. Put
that in a separate function, called lmb_init(). The other function is
now only used for testing.
Note that tests never set up the memory regions, so we can drop that
code.
Signed-off-by: Simon Glass <sjg at chromium.org>
---
common/board_r.c | 4 +--
include/lmb.h | 13 +++++++
lib/lmb.c | 88 +++++++++++++++++++++++++-----------------------
3 files changed, 59 insertions(+), 46 deletions(-)
diff --git a/common/board_r.c b/common/board_r.c
index eaf9b40ec02..9ce1b8cdd2b 100644
--- a/common/board_r.c
+++ b/common/board_r.c
@@ -611,9 +611,7 @@ static init_fnc_t init_sequence_r[] = {
#ifdef CONFIG_CLOCKS
set_cpu_clk_info, /* Setup clock information */
#endif
-#if CONFIG_IS_ENABLED(LMB)
- initr_lmb,
-#endif
+ lmb_init,
#ifdef CONFIG_EFI_LOADER
efi_memory_init,
#endif
diff --git a/include/lmb.h b/include/lmb.h
index 19231439c34..a2236a419fd 100644
--- a/include/lmb.h
+++ b/include/lmb.h
@@ -96,6 +96,19 @@ void board_lmb_reserve(void);
void arch_lmb_reserve(void);
void arch_lmb_reserve_generic(ulong sp, ulong end, ulong align);
+#if CONFIG_IS_ENABLED(LMB)
+/**
+ * lmb_init() - Initialise the LMB memory
+ *
+ * Initialise the LMB-subsystem-related data structures.
+ *
+ * Return: 0 if OK, -ve on failure.
+ */
+int lmb_init(void);
+#else
+static int lmb_init(void) { return 0; }
+#endif
+
/**
* lmb_mem_regions_init() - Initialise the LMB memory
* @mem_lst: Pointer to store location of free memory list
diff --git a/lib/lmb.c b/lib/lmb.c
index 83b060a2f4d..34cbaeaafd5 100644
--- a/lib/lmb.c
+++ b/lib/lmb.c
@@ -718,6 +718,47 @@ __weak void arch_lmb_reserve(void)
arch_lmb_reserve_generic(rsv_start, gd->ram_top, 16384);
}
+static int lmb_setup(struct lmb *lmb)
+{
+ int ret;
+
+ ret = alist_init(&lmb->free_mem, sizeof(struct lmb_region),
+ (uint)LMB_ALIST_INITIAL_SIZE);
+ if (!ret) {
+ log_debug("Unable to initialise the list for LMB free memory\n");
+ return -ENOMEM;
+ }
+
+ ret = alist_init(&lmb->used_mem, sizeof(struct lmb_region),
+ (uint)LMB_ALIST_INITIAL_SIZE);
+ if (!ret) {
+ log_debug("Unable to initialise the list for LMB used memory\n");
+ return -ENOMEM;
+ }
+
+ return 0;
+}
+
+int lmb_init(void)
+{
+ bool ret;
+
+ ret = lmb_setup(&lmb);
+ if (ret) {
+ log_debug("Unable to init LMB\n");
+ return ret;
+ }
+ lmb_add_memory();
+
+ /* Reserve the U-Boot image region once U-Boot has relocated */
+ if (spl_phase() == PHASE_SPL)
+ lmb_reserve_common_spl();
+ else if (spl_phase() == PHASE_BOARD_R)
+ lmb_reserve_common((void *)gd->fdt_blob);
+
+ return 0;
+}
+
/**
* lmb_mem_regions_init() - Initialise the LMB memory
* @mem_lst: Pointer to store location of free memory list
@@ -740,18 +781,10 @@ int lmb_mem_regions_init(struct alist **mem_lst, struct alist **used_lst,
{
bool ret;
- ret = alist_init(&lmb.free_mem, sizeof(struct lmb_region),
- (uint)LMB_ALIST_INITIAL_SIZE);
- if (!ret) {
- log_debug("Unable to initialise the list for LMB free memory\n");
- return -1;
- }
-
- ret = alist_init(&lmb.used_mem, sizeof(struct lmb_region),
- (uint)LMB_ALIST_INITIAL_SIZE);
- if (!ret) {
- log_debug("Unable to initialise the list for LMB used memory\n");
- return -1;
+ ret = lmb_setup(&lmb);
+ if (ret) {
+ log_debug("Unable to init LMB\n");
+ return ret;
}
if (mem_lst)
@@ -760,17 +793,6 @@ int lmb_mem_regions_init(struct alist **mem_lst, struct alist **used_lst,
if (used_lst)
*used_lst = &lmb.used_mem;
- if (!add_rsv_mem)
- return 0;
-
- lmb_add_memory();
-
- /* Reserve the U-Boot image region once U-Boot has relocated */
- if (spl_phase() == PHASE_SPL)
- lmb_reserve_common_spl();
- else if (spl_phase() == PHASE_BOARD_R)
- lmb_reserve_common((void *)gd->fdt_blob);
-
return 0;
}
@@ -790,26 +812,6 @@ void __maybe_unused lmb_mem_regions_uninit(struct alist *mem_lst,
alist_uninit(used_lst);
}
-/**
- * initr_lmb() - Initialise the LMB lists
- *
- * Initialise the LMB lists needed for keeping the memory map. There
- * are two lists, in form of alloced list data structure. One for the
- * available memory, and one for the used memory.
- *
- * Return: 0 on success, -ve on error
- */
-int initr_lmb(void)
-{
- int ret;
-
- ret = lmb_mem_regions_init(NULL, NULL, true);
- if (ret)
- printf("Unable to initialise the LMB data structures\n");
-
- return ret;
-}
-
struct lmb *lmb_get(void)
{
return &lmb;
--
2.34.1
More information about the U-Boot
mailing list