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

["Wu, Hao A" <hao.a.wu@intel.com>]

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Monday, April 01, 2019 10:05 PM
> To: Ni, Ray; Dong, Guo; edk2-devel@lists.01.org
> Cc: Wu, Hao A; Kinney, Michael D; Ma, Maurice; David Woodhouse
> Subject: Re: [edk2] [PATCH] UefiPayloadPkg: Enhance UEFI payload for
> coreboot and Slim Bootloader
> 
> (+David)
> 
> On 03/29/19 06:15, Ni, Ray wrote:
> >
> > 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.
> 
> The last status (up to & including comment 12) in
> <https://bugzilla.tianocore.org/show_bug.cgi?id=1496> was that:
> 
> (1) 8254TimerDxe would remain a generic driver, with the minimal
> interrupt controller programming that it needs flattened into it (no
> dependency on gEfiLegacy8259ProtocolGuid)
> 
> (2) the 8259InterruptControllerDxe driver (producing
> gEfiLegacy8259ProtocolGuid) would move under OvmfPkg/Csm.
> 
> Now, if 8254TimerDxe becomes specific to OvmfPkg, then I think it's not
> necessary to modify it -- it can continue depending on
> gEfiLegacy8259ProtocolGuid. Simply move both drivers from PcAtChipsetPkg
> to OvmfPkg. (Note: *not* to OvmfPkg/Csm).
> 
> Hao: can you please update
> <https://bugzilla.tianocore.org/show_bug.cgi?id=1496> accordingly? (If
> you agree, that is.)

Sure.
I will double confirm the latest direction and update the above BZ.

Best Regards,
Hao Wu

> 
> Thanks
> Laszlo