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

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Wed Jul 31 10:38:57 CEST 2024


On 31.07.24 10:34, Sughosh Ganu wrote:
> 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.

Hello Sughosh,

Please, rebase on 
https://source.denx.de/u-boot/custodians/u-boot-efi/-/commits/efi-2024-10-rc2 
for which I will create a pull request. It contains your v1.

Best regards

Heinrich

> 
> -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