[U-Boot] [PATCH 1/1] efi_loader: HII protocols: informative error message
AKASHI Takahiro
takahiro.akashi at linaro.org
Fri Mar 1 01:03:02 UTC 2019
On Fri, Mar 01, 2019 at 12:02:37AM +0100, Heinrich Schuchardt wrote:
> When package types are not supported an error message is displayed.
> Unfortunately there is no hint in the messages that they concern the HII
> protocols. Furthermore they are indent by tab for no obvious reason.
I think that I intended to use EFI_PRINT() here, but
printf() is fine for showing informative error messages.
> Prefix the messages by "HII: " instead of "\t".
"EFI HII" would be better?
Thanks,
-Takahiro Akashi
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> lib/efi_loader/efi_hii.c | 36 ++++++++++++++++++------------------
> 1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/lib/efi_loader/efi_hii.c b/lib/efi_loader/efi_hii.c
> index d63d2d84184..6ee386596a5 100644
> --- a/lib/efi_loader/efi_hii.c
> +++ b/lib/efi_loader/efi_hii.c
> @@ -387,7 +387,7 @@ add_packages(struct efi_hii_packagelist *hii,
> (struct efi_hii_guid_package *)package);
> break;
> case EFI_HII_PACKAGE_FORMS:
> - printf("\tForm package not supported\n");
> + printf("HII: Form package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_STRINGS:
> @@ -395,19 +395,19 @@ add_packages(struct efi_hii_packagelist *hii,
> (struct efi_hii_strings_package *)package);
> break;
> case EFI_HII_PACKAGE_FONTS:
> - printf("\tFont package not supported\n");
> + printf("HII: Font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_IMAGES:
> - printf("\tImage package not supported\n");
> + printf("HII: Image package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_SIMPLE_FONTS:
> - printf("\tSimple font package not supported\n");
> + printf("HII: Simple font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_DEVICE_PATH:
> - printf("\tDevice path package not supported\n");
> + printf("HII: Device path package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_KEYBOARD_LAYOUT:
> @@ -415,7 +415,7 @@ add_packages(struct efi_hii_packagelist *hii,
> (struct efi_hii_keyboard_package *)package);
> break;
> case EFI_HII_PACKAGE_ANIMATIONS:
> - printf("\tAnimation package not supported\n");
> + printf("HII: Animation package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_END:
> @@ -522,33 +522,33 @@ update_package_list(const struct efi_hii_database_protocol *this,
> remove_guid_package(hii);
> break;
> case EFI_HII_PACKAGE_FORMS:
> - printf("\tForm package not supported\n");
> + printf("HII: Form package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_STRINGS:
> remove_strings_package(hii);
> break;
> case EFI_HII_PACKAGE_FONTS:
> - printf("\tFont package not supported\n");
> + printf("HII: Font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_IMAGES:
> - printf("\tImage package not supported\n");
> + printf("HII: Image package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_SIMPLE_FONTS:
> - printf("\tSimple font package not supported\n");
> + printf("HII: Simple font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_DEVICE_PATH:
> - printf("\tDevice path package not supported\n");
> + printf("HII: Device path package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_KEYBOARD_LAYOUT:
> remove_keyboard_package(hii);
> break;
> case EFI_HII_PACKAGE_ANIMATIONS:
> - printf("\tAnimation package not supported\n");
> + printf("HII: Animation package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> break;
> case EFI_HII_PACKAGE_END:
> @@ -609,7 +609,7 @@ list_package_lists(const struct efi_hii_database_protocol *this,
> break;
> continue;
> case EFI_HII_PACKAGE_FORMS:
> - printf("\tForm package not supported\n");
> + printf("HII: Form package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_STRINGS:
> @@ -617,19 +617,19 @@ list_package_lists(const struct efi_hii_database_protocol *this,
> break;
> continue;
> case EFI_HII_PACKAGE_FONTS:
> - printf("\tFont package not supported\n");
> + printf("HII: Font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_IMAGES:
> - printf("\tImage package not supported\n");
> + printf("HII: Image package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_SIMPLE_FONTS:
> - printf("\tSimple font package not supported\n");
> + printf("HII: Simple font package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_DEVICE_PATH:
> - printf("\tDevice path package not supported\n");
> + printf("HII: Device path package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_KEYBOARD_LAYOUT:
> @@ -637,7 +637,7 @@ list_package_lists(const struct efi_hii_database_protocol *this,
> break;
> continue;
> case EFI_HII_PACKAGE_ANIMATIONS:
> - printf("\tAnimation package not supported\n");
> + printf("HII: Animation package not supported\n");
> ret = EFI_INVALID_PARAMETER;
> continue;
> case EFI_HII_PACKAGE_END:
> --
> 2.20.1
>
More information about the U-Boot
mailing list