[PATCH 3/3] efi_loader: make efi_add_memory_map_pg() static

[Ilias Apalodimas]

On Sun, 16 Feb 2025 at 13:13, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> The function is only used in the efi_memory.c module.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  include/efi_loader.h        | 15 ---------------
>  lib/efi_loader/efi_memory.c |  1 +
>  2 files changed, 1 insertion(+), 15 deletions(-)
>
> diff --git a/include/efi_loader.h b/include/efi_loader.h
> index db3d20fd753..1d75d97ebbc 100644
> --- a/include/efi_loader.h
> +++ b/include/efi_loader.h
> @@ -852,21 +852,6 @@ efi_status_t efi_get_memory_map(efi_uintn_t *memory_map_size,
>  /* Adds a range into the EFI memory map */
>  efi_status_t efi_add_memory_map(u64 start, u64 size, int memory_type);
>
> -/**
> - * efi_add_memory_map_pg() - add pages to the memory map
> - *
> - * @start:                     start address, must be a multiple of
> - *                             EFI_PAGE_SIZE
> - * @pages:                     number of pages to add
> - * @memory_type:               type of memory added
> - * @overlap_conventional:      region may only overlap free(conventional)
> - *                             memory
> - * Return:                     status code
> - */
> -efi_status_t efi_add_memory_map_pg(u64 start, u64 pages,
> -                                  int memory_type,
> -                                  bool overlap_conventional);
> -
>  /* Called by board init to initialize the EFI drivers */
>  efi_status_t efi_driver_init(void);
>  /* Called when a block device is added */
> diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
> index 11d092dc289..6d00b186250 100644
> --- a/lib/efi_loader/efi_memory.c
> +++ b/lib/efi_loader/efi_memory.c
> @@ -268,6 +268,7 @@ static s64 efi_mem_carve_out(struct efi_mem_list *map,
>   *                             memory
>   * Return:                     status code
>   */
> +static
>  efi_status_t efi_add_memory_map_pg(u64 start, u64 pages,
>                                    int memory_type,
>                                    bool overlap_conventional)
> --
> 2.47.1
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>