Re: [PATCH v4 08/45] x86: Show the CPU physical address size with bdinfo

[Bin Meng]

Hi Simon,

On Mon, Jun 19, 2023 at 8:01 PM Simon Glass <s...@chromium.org> wrote:
>
> This is useful information so show it with the bdinfo command.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
>
> (no changes since v1)
>
>  arch/x86/lib/bdinfo.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/x86/lib/bdinfo.c b/arch/x86/lib/bdinfo.c
> index 15390070fe85..0970efa4726f 100644
> --- a/arch/x86/lib/bdinfo.c
> +++ b/arch/x86/lib/bdinfo.c
> @@ -22,6 +22,7 @@ void arch_print_bdinfo(void)
>         bdinfo_print_num_l("vendor", gd->arch.x86_vendor);
>         bdinfo_print_str(" name", cpu_vendor_name(gd->arch.x86_vendor));
>         bdinfo_print_num_l("model", gd->arch.x86_model);
> +       bdinfo_print_num_l("phys_addr", cpu_phys_address_size());

The "phys_addr" is confusing. How about "phy addr in bits"?

>
>         if (IS_ENABLED(CONFIG_EFI_STUB))
>                 efi_show_bdinfo();
> --

Regards,
Bin