[PATCH v2 3/4] efi_loader: correct efi_binary_run_dp() return value

[Ilias Apalodimas]

On Tue, 21 Apr 2026 at 01:03, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> efi_binary_run_dp() is expected to return an efi_status_t value.
>
> Reported-by: Simon Glass <sjg at chromium.org>
> Fixes: 6422820ac3e5 ("efi_loader: split unrelated code from efi_bootmgr.c")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>


> ---
> v2:
>         new patch
> ---
>  lib/efi_loader/efi_bootbin.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/efi_bootbin.c b/lib/efi_loader/efi_bootbin.c
> index b394f0d60ce..144266b785f 100644
> --- a/lib/efi_loader/efi_bootbin.c
> +++ b/lib/efi_loader/efi_bootbin.c
> @@ -227,7 +227,7 @@ static efi_status_t efi_binary_run_dp(void *image, size_t size, void *fdt,
>         if (ret != EFI_SUCCESS) {
>                 log_err("Error: Cannot initialize UEFI sub-system, r = %lu\n",
>                         ret & ~EFI_ERROR_MASK);
> -               return -1;
> +               return ret;
>         }
>
>         ret = efi_install_fdt(fdt);
> --
> 2.53.0
>