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

[Laszlo Ersek <lersek@redhat.com>]

(+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.)

Thanks
Laszlo