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

[Bryan O'Donoghue]

V3:

- Takes addition of function description and makes it a separate patch
- Subtracts dangling "uint64_t r" in V2 patch

Heinrich if you want to take this set as is add a SOB to
"efi_loader: Add a method description to efi_add_memory_map"

and if not please take in my change for "efi_add_runtime_mmio()" to your
single patch version.

V2:

Following on from a discussion with Heinrich Schuchardt, please find a
reworked set of patches updating efi_add_memory_map() to

- Return efi_status_t
- Return EFI_SUCCESS where appropriate
- Return EFI_NO_MAPPING in two cases where zero was returned to indicate an
  error
- Updating of users of efi_add_memory_map() to parse for
  EFI_SUCCESS/efi_status_t

I've opted to maintain other returned status codes propogated by functions
that call efi_add_memory_map(). For example efi_add_runtime_mmio()
continues return EFI_OUT_OF_RESOURCES instead of directly returning the
result code of efi_add_memory_map(). The idea being that other users of the
EFI layer such as Linux or grub would not be affected by this internal
u-boot change.

V1:

https://patchwork.ozlabs.org/patch/1129402/
https://patchwork.ozlabs.org/patch/1129403/

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 (7):
  efi_loader: Change return type of efi_add_memory_map()
  efi_loader: Change efi_add_memory_map() to return EFI_SUCCESS
  efi_loader: Return non-zero for error in efi_add_memory_map()
  efi_loader: Update efi_allocate_pages() to new efi_add_memory_map()
  efi_loader: Update efi_free_pages() to new efi_add_memory_map()
  efi_loader: Treat the result of efi_add_memory_map as efi_status_t
  efi_loader: Capture efi_add_memory_map() result efi_add_runtime_mmio()

Heinrich Schuchardt (1):
  efi_loader: Add a method description to efi_add_memory_map

 cmd/bootefi.c                |  4 ++--
 include/efi_loader.h         |  4 ++--
 lib/efi_loader/efi_memory.c  | 34 ++++++++++++++++++++--------------
 lib/efi_loader/efi_runtime.c |  6 +++---
 4 files changed, 27 insertions(+), 21 deletions(-)

-- 
2.22.0