Hi Bin, On Wed, 12 Jul 2023 at 08:02, Bin Meng <[email protected]> wrote: > > Hi Simon, > > On Mon, Jun 19, 2023 at 8:01 PM Simon Glass <[email protected]> wrote: > > > > This is useful information so show it with the bdinfo command. > > > > Signed-off-by: Simon Glass <[email protected]> > > --- > > > > (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
