Re: [PATCH] UefiPayloadPkg: Enhance UEFI payload for coreboot and Slim Bootloader

["Ni, Ray" <ray.ni@intel.com>]

Guo,
Just to double confirm: UefiPayloadPkg will not require Legacy8254 timer support.
The old packages Coreboot*Pkgs will be removed.
Which means now only QEMU/OVMF needs the Legacy8254 support.

Laszlo,
Now since QEMU/OVMF is the only consumer of the Legacy8254 driver, do you
agree that the Legacy8254 is moved to OvmfPkg?
Note: We agreed that Legacy8259 will be moved to OvmfPkg/Csm directory and
that decision is not going to be changed by this new situation.

Thanks,
Ray

> -----Original Message-----
> From: edk2-devel <edk2-devel-bounces@lists.01.org> On Behalf Of Guo
> Dong
> Sent: Friday, March 29, 2019 8:34 AM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH] UefiPayloadPkg: Enhance UEFI payload for coreboot
> and Slim Bootloader
> 
> CorebootModulePkg and CorebootPayloadPkg originally supports coreboot
> only.
> In order to support other bootloaders, such as Slim Bootloader, they need
> be updated to be more generic.
> UEFI Payload (UefiPayloadPkg) a converged package from
> CorebootModulePkg
> and CorebootPayloadPkg with following updates:
> a. Support both coreboot and Slim Bootloader
> b. Removed SataControllerDxe and BaseSerialPortLib16550 to use EDK2
> modules
> c. Support passing bootloader parameter to UEFI payload, e.g. coreboot
>    table from coreboot or HOB list from Slim Bootloader
> d. Using GraphicsOutputDxe from EDK2 with minor change instead of FbGop
> e. Remove the dependency to IntelFrameworkPkg and
> IntelFrameworkModulePkg
>    and QuarkSocPkg
> f. Use BaseDebugLibSerialPort library as DebugLib
> g. Use HPET timer, drop legacy 8254 timer support
> h. Use BaseXApicX2ApicLib instead of BaseXApicLib
> i. Other clean ups
> 
> On how UefiPayloadPkg could work with coreboot/Slim Bootloader, please
> refer UefiPayloadPkg/BuildAndIntegrationInstructions.txt
> 
> Once UefiPayloadPkg is checked-in, CorebootModulePkg and
> CorebootPayloadPkg
> could be retired.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Guo Dong <guo.dong@intel.com>