colin/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

efi_loader: remove unused code from copy_fdt()

Browse Source 
There is logic in the copy_fdt() function which is iterating over the
platform's DRAM banks and setting the fdt_ram_start variable. However,
this variable is not used subsequently in the function. Remove this
superfluous code.

Signed-off-by: Sughosh Ganu <sughosh.ganu@linaro.org>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
This commit is contained in:
Sughosh Ganu
2025-03-17 14:03:55 +05:30
committed by Ilias Apalodimas
parent 042c8f0cb1
commit b267ab2c53
1 changed files with 1 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lib/efi_loader/efi_helper.c
View File

@@ -454,23 +454,11 @@ efi_status_t efi_env_set_load_options(efi_handle_t handle,
*/ */
static efi_status_t copy_fdt(void **fdtp) static efi_status_t copy_fdt(void **fdtp)
{ {
unsigned long fdt_ram_start = -1L, fdt_pages; unsigned long fdt_pages;
efi_status_t ret = 0; efi_status_t ret = 0;
void *fdt, *new_fdt; void *fdt, *new_fdt;
u64 new_fdt_addr; u64 new_fdt_addr;
uint fdt_size; uint fdt_size;
int i;
for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
u64 ram_start = gd->bd->bi_dram[i].start;
u64 ram_size = gd->bd->bi_dram[i].size;
if (!ram_size)
continue;
if (ram_start < fdt_ram_start)
fdt_ram_start = ram_start;
}
/* /*
* Give us at least 12 KiB of breathing room in case the device tree * Give us at least 12 KiB of breathing room in case the device tree