On Fri, 7 Jun 2024 at 21:54, Sughosh Ganu <[email protected]> wrote: > > The changes in the EFI memory map are now notified to the LMB > module. There is therefore no need to explicitly get the efi memory > map and set aside the EFI allocated memory. > > Signed-off-by: Sughosh Ganu <[email protected]> > --- > lib/lmb.c | 35 ----------------------------------- > 1 file changed, 35 deletions(-) > > diff --git a/lib/lmb.c b/lib/lmb.c > index 3059609aea..c9f6ca692e 100644 > --- a/lib/lmb.c > +++ b/lib/lmb.c > @@ -204,38 +204,6 @@ void arch_lmb_reserve_generic(ulong sp, ulong end, ulong > align) > } > } > > -/** > - * efi_lmb_reserve() - add reservations for EFI memory > - * > - * Add reservations for all EFI memory areas that are not > - * EFI_CONVENTIONAL_MEMORY. > - * > - * Return: 0 on success, 1 on failure > - */ > -static __maybe_unused int efi_lmb_reserve(void) > -{ > - struct efi_mem_desc *memmap = NULL, *map; > - efi_uintn_t i, map_size = 0; > - efi_status_t ret; > - > - ret = efi_get_memory_map_alloc(&map_size, &memmap); > - if (ret != EFI_SUCCESS) > - return 1; > - > - for (i = 0, map = memmap; i < map_size / sizeof(*map); ++map, ++i) { > - if (map->type != EFI_CONVENTIONAL_MEMORY) { > - lmb_reserve_flags(map_to_sysmem((void *)(uintptr_t) > - map->physical_start), > - map->num_pages * EFI_PAGE_SIZE, > - map->type == > EFI_RESERVED_MEMORY_TYPE > - ? LMB_NOMAP : LMB_NONE); > - } > - } > - efi_free_pool(memmap); > - > - return 0; > -} > - > /** > * lmb_reserve_common() - Reserve memory region occupied by U-Boot image > * @fdt_blob: pointer to the FDT blob > @@ -255,9 +223,6 @@ void lmb_reserve_common(void *fdt_blob) > > if (CONFIG_IS_ENABLED(OF_LIBFDT) && fdt_blob) > boot_fdt_add_mem_rsv_regions(fdt_blob); > - > - if (CONFIG_IS_ENABLED(EFI_LOADER)) > - efi_lmb_reserve(); > } > > /** > -- > 2.34.1 >
Reviewed-by: Ilias Apalodimas <[email protected]>
