[PATCH v3 07/15] efi_memory: do not add U-Boot memory to the memory map

[Simon Glass]

Hi Sughosh,

On Sun, 13 Oct 2024 at 04:56, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> The memory region occupied by U-Boot is reserved by LMB, and gets
> added to the EFI memory map through a call from the LMB module. Remove
> this superfluous addition to the EFI memory map.

Without the patch before (which so far I don't see the need for), this
code is needed.

>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
> Changes since V2: New patch
>
>  lib/efi_loader/efi_memory.c | 10 ----------
>  1 file changed, 10 deletions(-)
>
> diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
> index 41501e9d41..16e64987af 100644
> --- a/lib/efi_loader/efi_memory.c
> +++ b/lib/efi_loader/efi_memory.c
> @@ -870,16 +870,6 @@ static void add_u_boot_and_runtime(void)
>  {
>         unsigned long runtime_start, runtime_end, runtime_pages;
>         unsigned long runtime_mask = EFI_PAGE_MASK;
> -       unsigned long uboot_start, uboot_pages;
> -       unsigned long uboot_stack_size = CONFIG_STACK_SIZE;
> -
> -       /* Add U-Boot */
> -       uboot_start = ((uintptr_t)map_sysmem(gd->start_addr_sp, 0) -
> -                      uboot_stack_size) & ~EFI_PAGE_MASK;
> -       uboot_pages = ((uintptr_t)map_sysmem(gd->ram_top - 1, 0) -
> -                      uboot_start + EFI_PAGE_MASK) >> EFI_PAGE_SHIFT;
> -       efi_add_memory_map_pg(uboot_start, uboot_pages, EFI_BOOT_SERVICES_CODE,
> -                             false);
>
>  #if defined(__aarch64__)
>         /*
> --
> 2.34.1
>

Regards,
Simon