Hi Bin, On Tue, 19 Sept 2023 at 01:20, Bin Meng <bmeng...@gmail.com> wrote: > > On Thu, Aug 24, 2023 at 5:11 AM Simon Glass <s...@chromium.org> wrote: > > > > Make a note of where coreboot installed the SMBIOS tables so that we can > > pass this on to EFI. > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > --- > > > > (no changes since v2) > > > > Changes in v2: > > - Add new patch > > > > arch/x86/lib/coreboot/cb_sysinfo.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/x86/lib/coreboot/cb_sysinfo.c > > b/arch/x86/lib/coreboot/cb_sysinfo.c > > index dfbc80c430e..f7fd9ea5bcb 100644 > > --- a/arch/x86/lib/coreboot/cb_sysinfo.c > > +++ b/arch/x86/lib/coreboot/cb_sysinfo.c > > @@ -471,6 +471,7 @@ int get_coreboot_info(struct sysinfo_t *info) > > return -ENOENT; > > gd->arch.coreboot_table = addr; > > gd_set_acpi_start(map_to_sysmem(info->rsdp)); > > + gd_set_smbios_start(info->smbios_start); > > use map_to_sysmem() too?
Actually that field is a u64, so already an address. > > > gd->flags |= GD_FLG_SKIP_LL_INIT; > > > > return 0; > > -- > > Reviewed-by: Bin Meng <bmeng...@gmail.com> Regards, Simon