[PATCH 1/1] efi_loader: use list_count_nodes() in efi_protocols_per_handle()

Sughosh Ganu sughosh.ganu at linaro.org
Wed Jul 31 10:34:44 CEST 2024


On Wed, 31 Jul 2024 at 13:43, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> Simplify the code by using the list_count_nodes() function.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---

I have made changes to the files that you mentioned in your review,
and plan to come up with a v2 once the CI goes through. Thanks.

-sughosh

>  lib/efi_loader/efi_boottime.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
> index eedc5f39549..4f52284b4c6 100644
> --- a/lib/efi_loader/efi_boottime.c
> +++ b/lib/efi_loader/efi_boottime.c
> @@ -2509,16 +2509,12 @@ static efi_status_t EFIAPI efi_protocols_per_handle(
>                 return EFI_EXIT(EFI_INVALID_PARAMETER);
>
>         *protocol_buffer = NULL;
> -       *protocol_buffer_count = 0;
>
>         efiobj = efi_search_obj(handle);
>         if (!efiobj)
>                 return EFI_EXIT(EFI_INVALID_PARAMETER);
>
> -       /* Count protocols */
> -       list_for_each(protocol_handle, &efiobj->protocols) {
> -               ++*protocol_buffer_count;
> -       }
> +       *protocol_buffer_count = list_count_nodes(&efiobj->protocols);
>
>         /* Copy GUIDs */
>         if (*protocol_buffer_count) {
> --
> 2.45.2
>


More information about the U-Boot mailing list