* [U-Boot] [PATCH 1/1] efi_loader: HII protocols: informative error message
@ 2019-02-28 23:02 Heinrich Schuchardt
2019-03-01 1:03 ` AKASHI Takahiro
0 siblings, 1 reply; 2+ messages in thread
From: Heinrich Schuchardt @ 2019-02-28 23:02 UTC (permalink / raw)
To: u-boot
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.
Prefix the messages by "HII: " instead of "\t".
Signed-off-by: Heinrich Schuchardt <xypron.glpk@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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [U-Boot] [PATCH 1/1] efi_loader: HII protocols: informative error message
2019-02-28 23:02 [U-Boot] [PATCH 1/1] efi_loader: HII protocols: informative error message Heinrich Schuchardt
@ 2019-03-01 1:03 ` AKASHI Takahiro
0 siblings, 0 replies; 2+ messages in thread
From: AKASHI Takahiro @ 2019-03-01 1:03 UTC (permalink / raw)
To: u-boot
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@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
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-01 1:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-28 23:02 [U-Boot] [PATCH 1/1] efi_loader: HII protocols: informative error message Heinrich Schuchardt
2019-03-01 1:03 ` AKASHI Takahiro
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.