[PATCH v4 14/27] lmb: bootm: remove superfluous lmb stub functions

[Ilias Apalodimas]

On Mon, 26 Aug 2024 at 15:01, Sughosh Ganu <sughosh.ganu at 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 at linaro.org>
> Reviewed-by: Simon Glass <sjg at 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 at linaro.org>