Re: [PATCH 2/2] ArmVirtPkg: enable EBC interpreter for AArch64 QEMU

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 17 August 2016 at 15:45, Leif Lindholm <leif.lindholm@linaro.org> 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 <leif.lindholm@linaro.org>
> ---
>  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 <ard.biesheuvel@linaro.org>

> 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
>