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

[Simon Glass]

Hi Bin,

On Wed, 12 Jul 2023 at 08:02, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Mon, Jun 19, 2023 at 8:01 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > This is useful information so show it with the bdinfo command.
> >
> > Signed-off-by: Simon Glass <sjg at 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"?

I've updated it, although it exceeds the 12 chars allow for the field.
We can always adjust it later if needed.


>
> >
> >         if (IS_ENABLED(CONFIG_EFI_STUB))
> >                 efi_show_bdinfo();
> > --
Regards,
Simon