On Mon, 26 Aug 2024 at 15:01, Sughosh Ganu sughosh.ganu@linaro.org wrote:
Remove a couple of superfluous LMB stub functions, and instead put a check for calling the lmb_reserve() function.
Signed-off-by: Sughosh Ganu sughosh.ganu@linaro.org Reviewed-by: Simon Glass sjg@chromium.org
Changes since V3: None
boot/bootm.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/boot/bootm.c b/boot/bootm.c index ddede08979..a61bbcfb45 100644 --- a/boot/bootm.c +++ b/boot/bootm.c @@ -239,11 +239,6 @@ static int boot_get_kernel(const char *addr_fit, struct bootm_headers *images, return 0; }
-#if !CONFIG_IS_ENABLED(LMB) -#define lmb_reserve(base, size) -static inline void boot_start_lmb(void) { } -#endif
static int bootm_start(void) { memset((void *)&images, 0, sizeof(images)); @@ -700,7 +695,9 @@ static int bootm_load_os(struct bootm_headers *images, int boot_progress) images->os.end = relocated_addr + image_size; }
lmb_reserve(images->os.load, (load_end - images->os.load));
if (CONFIG_IS_ENABLED(LMB))lmb_reserve(images->os.load, (load_end - images->os.load));return 0;}
-- 2.34.1
Reviewed-by: Ilias Apalodimas ilias.apalodimas@linaro.org