efi_st_printf() does not support format code %ld. Anyway the format code
for size_t would be %zu which isn't supported either.

We do not want any divisions to avoid invalid references to integer
arithmetic routines, cf.
https://gcc.gnu.org/onlinedocs/gccint/Integer-library-routines.html.

As a simple remedy remove the noisy messages from the output.
They are not relevant for automated testing.

Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
 lib/efi_selftest/efi_selftest_hii.c | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/lib/efi_selftest/efi_selftest_hii.c 
b/lib/efi_selftest/efi_selftest_hii.c
index 16d7b60829..e38af7dbf8 100644
--- a/lib/efi_selftest/efi_selftest_hii.c
+++ b/lib/efi_selftest/efi_selftest_hii.c
@@ -205,8 +205,6 @@ static int test_hii_database_list_package_lists(void)
                             (unsigned int)ret);
                goto out;
        }
-       efi_st_printf("list_package_lists returned %ld handles\n",
-                     handles_size / sizeof(*handles));
        free(handles);
 
        /* STRINGS */
@@ -236,8 +234,6 @@ static int test_hii_database_list_package_lists(void)
                ret = EFI_ST_FAILURE;
                goto out;
        }
-       efi_st_printf("list_package_lists returned %ld strings handles\n",
-                     handles_size / sizeof(*handles));
        free(handles);
 
        /* GUID */
@@ -267,8 +263,6 @@ static int test_hii_database_list_package_lists(void)
                ret = EFI_ST_FAILURE;
                goto out;
        }
-       efi_st_printf("list_package_lists returned %ld guid handles\n",
-                     handles_size / sizeof(*handles));
        free(handles);
 
        /* KEYBOARD_LAYOUT */
@@ -298,8 +292,6 @@ static int test_hii_database_list_package_lists(void)
                ret = EFI_ST_FAILURE;
                goto out;
        }
-       efi_st_printf("list_package_lists returned %ld keyboard layout 
handles\n",
-                     handles_size / sizeof(*handles));
        free(handles);
 
        result = EFI_ST_SUCCESS;
@@ -420,9 +412,6 @@ static int test_hii_database_find_keyboard_layouts(void)
        }
        free(guids);
 
-       efi_st_printf("find_keyboard_layouts returned %ld guids\n",
-                     guids_size / sizeof(*guids));
-
        result = EFI_ST_SUCCESS;
 
 out:
-- 
2.20.1

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to