On 08/17/16 15:47, Ard Biesheuvel wrote: > On 17 August 2016 at 15:45, Leif Lindholm <[email protected]> wrote: >> Since we now have EBC support for AArch64, enable it by default >> on the QEMU platform. >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Leif Lindholm <[email protected]> >> --- >> ArmVirtPkg/ArmVirtQemu.dsc | 1 + >> ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 5 +++++ >> 2 files changed, 6 insertions(+) >> >> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc >> index 9f88786..fb851e7 100644 >> --- a/ArmVirtPkg/ArmVirtQemu.dsc >> +++ b/ArmVirtPkg/ArmVirtQemu.dsc >> @@ -401,3 +401,4 @@ [Components.AARCH64] >> <LibraryClasses> >> NULL|ArmVirtPkg/Library/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf >> } >> + MdeModulePkg/Universal/EbcDxe/EbcDxe.inf > > Please add it to ArmVirtPkg/ArmVirtQemuKernel.dsc as well: > ArmVirtQemuFvMain.fdf.inc is shared between them. > > With that, > > Reviewed-by: Ard Biesheuvel <[email protected]>
Any reason for not adding it to the Xen platform as well? What kind of modules do we expect to come in EBC? Option ROMs perhaps? In that case, Xen may indeed be irrelevant here (I have no clue about adding oproms to emulated or assigned devices under Xen, so I can't make an argument either way.) If we expect general stuff (for example, even UEFI_APPLICATION modules) to arrive in EBC, then I think Xen would be a justified addition. Thanks Laszlo >> diff --git a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc >> b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc >> index ad7037f..0a5b899 100644 >> --- a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc >> +++ b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc >> @@ -146,6 +146,11 @@ [FV.FvMain] >> # >> INF MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf >> INF OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpiPlatformDxe.inf >> + >> + # >> + # EBC support >> + # >> + INF MdeModulePkg/Universal/EbcDxe/EbcDxe.inf >> !endif >> >> # >> -- >> 2.1.4 >> _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
