Currently the console is connected before EndOfDxe causing OptionsROMs to be loaded, but their drivers aren't used and thus no GOP is installed. To make use of 3rdparty OptionROMs connect the console

Hi Sunny, none of the other packages are doing this before EndOfDxe. And there's no point in having trusted console as earlier as possible, as nothing is displayed in PlatformBootManagerBeforeConsole(

The default EfiSmbiosProtocol operates on an empty SMBIOS table. The SMBIOS tables are provided by the bootloader on UefiPayloadPkg. Scan for existing tables in SmbiosDxe and load them if they seem va

From: Tim Crawford <tcrawford@...> Use Microsoft's recommended positioning [1] for the boot logo. > We recommend that the logo is placed with its center at 38.2% from the > screen's top edge.

Hi, Please find the issue created here: https://bugzilla.tianocore.org/show_bug.cgi?id=3226

This patch breaks booting on master. In CpuDxe.efi / InitGlobalDescriptorTable as the GDT pointer is casted to 32bits. Regards, Patrick

Hi Guo, I tested on 078400ee15e7b250e4dfafd840c2e0c19835e16b and run it in QEMU. The problem seems to be here, as gdt is allocated > 4GiB: gdtPtr.Base = (UINT32)(UINTN)(VOID*) gdt; Regards, Patrick

Hi Guo, I tested it and it works fine with your patch. The crash is gone. BTW: Are there are other possible side effects with placing memory above 4GB? Like 32bit UEFI drivers or Option ROMs malfuncti

The default EfiSmbiosProtocol operates on an empty SMBIOS table. The SMBIOS tables are provided by the bootloader on UefiPayloadPkg. Scan for existing tables in SmbiosDxe and load them if they seem va

Hi Ray, thanks for your feedback. Currently a single HOB containing all the SMBIOS table is exported by coreboot. As coreboot doesn't support multiple HOBs with the same ID, #2 isn't a solution. I'll

Hi Zhiguang, this is still on my todo-list, but I'm quite busy right now. If you got time please take care of that patch. Regards, Patrick Rudolph

This is not correct, it should be if (SmbiosTable->EntryPointLength != 0x1E && SmbiosTable->EntryPointLength != sizeof (SMBIOS_TABLE_ENTRY_POINT)) { otherwise the table wouldn't be recognized as valid

Note that AcpiTableDxe.inf is currently not part of the FV on UefipayloadPkg, so there won't be any tables installed after all. DEBUG ((DEBUG_INFO, "Create acpi table ...

That doesn't compile as Status is written, but never read.

Tested the patch series on Intel Coffee Lake Platform using latest coreboot master. Everything seems to work fine. Thanks Patrick

Currently several DXE crash due to invalid memory resource settings. coreboot and slimbootloader provide an e820 compatible memory map, which doesn't work well with EDK2 as the e820 spec is missing MM

Hi Guo, The PciHostBridgeDxe expects the PCI Aperature and MMCONF to be marked as EfiMemoryMappedIO, however as the bootloader provides an e820 compatible memory map, it's actually marked as EfiReserv

Currently several DXE crash due to invalid memory resource settings. The PciHostBridgeDxe which expects the MMCONF and PCI Aperature to be EfiMemoryMappedIO, but currently those regions are (partly) m

Hi Maurice, I've implemented the requested changes. It now also accepts ACPI_NVS as usable DRAM. Thanks, Patrick

Currently several DXE crash due to invalid memory resource settings. The PciHostBridgeDxe which expects the MMCONF and PCI Aperature to be EfiMemoryMappedIO, but currently those regions are (partly) m