[PATCH v3 18/27] lmb: remove the unused board_lmb_reserve() function

Simon Glass sjg at chromium.org
Fri Aug 23 22:47:27 CEST 2024


On Wed, 21 Aug 2024 at 05:00, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> The board_lmb_reserve() function is not being used, and currently
> there is only an empty weak function defined. Remove this unused
> function.
>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
> Changes since V2: New patch
>
>  include/lmb.h | 2 --
>  lib/lmb.c     | 6 ------
>  2 files changed, 8 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

One less weak function!


>
> diff --git a/include/lmb.h b/include/lmb.h
> index 6e435bd35c..79bd2457d3 100644
> --- a/include/lmb.h
> +++ b/include/lmb.h
> @@ -107,8 +107,6 @@ long lmb_free(phys_addr_t base, phys_size_t size);
>  void lmb_dump_all(void);
>  void lmb_dump_all_force(void);
>
> -void board_lmb_reserve(void);
> -
>  #if CONFIG_IS_ENABLED(UNIT_TEST)
>  struct lmb *lmb_get(void);
>  int lmb_push(struct lmb *store);
> diff --git a/lib/lmb.c b/lib/lmb.c
> index 1116b2b868..11959760b6 100644
> --- a/lib/lmb.c
> +++ b/lib/lmb.c
> @@ -221,7 +221,6 @@ static void lmb_reserve_uboot_region(void)
>
>  static void lmb_reserve_common(void *fdt_blob)
>  {
> -       board_lmb_reserve();
>         lmb_reserve_uboot_region();
>
>         if (CONFIG_IS_ENABLED(OF_LIBFDT) && fdt_blob)
> @@ -688,11 +687,6 @@ int lmb_is_reserved_flags(phys_addr_t addr, int flags)
>         return 0;
>  }
>
> -__weak void board_lmb_reserve(void)
> -{
> -       /* please define platform specific board_lmb_reserve() */
> -}
> -
>  static int lmb_setup(void)
>  {
>         bool ret;
> --
> 2.34.1
>


More information about the U-Boot mailing list