[U-Boot] [PATCH 0/2] efi_loader: Fix inconsistencies in efi_add_memory_map usage

Bryan O'Donoghue pure.logic at nexus-software.ie
Mon Jul 8 23:10:55 UTC 2019


These two patches fix some inconsistent usage around efi_add_memory_map().

The first patch fixes the case where there is a mapping for an address
starting at 0 as is the case on RPI3. We should not print an error for
this. efi_add_memory_map(start = 0, ...) succeeds but
efi_carve_out_dt_rsv() does not properly parse the result code.

The second patch fixes the result code returned by efi_add_memory_map() in
two instances. Returing zero is the same as returning EFI_SUCCESS, we
should return one of the error codes from include/efi.h only, not zero to
indicate failure.

Bryan O'Donoghue (2):
  efi_loader: Check the result of efi_add_memory_map against input addr
  efi_loader: Return non-zero for error in efi_add_memory_map()

 cmd/bootefi.c               | 4 ++--
 lib/efi_loader/efi_memory.c | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.22.0



More information about the U-Boot mailing list