[U-Boot] [PATCH 1/4] x86: fsp: Compact the output of hob command

Bin Meng bmeng.cn at gmail.com
Sat Oct 10 13:01:49 CEST 2015


On Sat, Oct 10, 2015 at 4:47 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
> Compact hob command output, especially by making hob type string a
> little bit shorter so that we can leave room for future extension.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> ---
>
>  arch/x86/lib/cmd_hob.c | 28 ++++++++++++++--------------
>  1 file changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/arch/x86/lib/cmd_hob.c b/arch/x86/lib/cmd_hob.c
> index 915746a..6ff321a 100644
> --- a/arch/x86/lib/cmd_hob.c
> +++ b/arch/x86/lib/cmd_hob.c
> @@ -14,16 +14,16 @@ DECLARE_GLOBAL_DATA_PTR;
>  static char *hob_type[] = {
>         "reserved",
>         "Hand-off",
> -       "Memory Allocation",
> -       "Resource Descriptor",
> -       "GUID Extension",
> -       "Firmware Volume",
> +       "Mem Alloc",
> +       "Res Desc",
> +       "GUID Ext",
> +       "FV",
>         "CPU",
> -       "Memory Pool",
> +       "Mem Pool",
>         "reserved",
> -       "Firmware Volume 2",
> -       "Load PEIM Unused",
> -       "UEFI Capsule",
> +       "FV2",
> +       "Load PEIM",
> +       "Capsule",
>  };
>
>  int do_hob(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> @@ -37,20 +37,20 @@ int do_hob(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>
>         printf("HOB list address: 0x%08x\n\n", (unsigned int)hdr);
>
> -       printf("No. | Address  | Type                | Length in Bytes\n");
> -       printf("----|----------|---------------------|----------------\n");
> +       printf("#  | Address  | Type      | Len\n");
> +       printf("---|----------|-----------|-----\n");
>         while (!end_of_hob(hdr)) {
> -               printf("%-3d | %08x | ", i, (unsigned int)hdr);
> +               printf("%-2d | %08x | ", i, (unsigned int)hdr);
>                 type = hdr->type;
>                 if (type == HOB_TYPE_UNUSED)
>                         desc = "*Unused*";
>                 else if (type == HOB_TYPE_EOH)
> -                       desc = "*END OF HOB*";
> +                       desc = "*EOH*";
>                 else if (type >= 0 && type <= ARRAY_SIZE(hob_type))
>                         desc = hob_type[type];
>                 else
> -                       desc = "*Invalid Type*";
> -               printf("%-19s | %-15d\n", desc, hdr->len);
> +                       desc = "*Invalid*";
> +               printf("%-9s | %-4d\n", desc, hdr->len);

Looks like I should change %-4d to %-5d as on BayTrail some HOB length
would be quite large.

>                 hdr = get_next_hob(hdr);
>                 i++;
>         }
> --

Regards,
Bin


More information about the U-Boot mailing list