Re: [PATCH v4 1/6] EmbeddedPkg: SiI3132: Note that ARM is using this Dxe

[Leif Lindholm <leif.lindholm@linaro.org>]

Well, the subject doesn't really match the content.

MdeModulePkg: add ARM/AARCH64 to AtaBusDxe VALID_ARCHITECTURES

Is a better match.

And since it's both non-functional and an obvious fix, I'd say you
could send this off to edk2-devel on its own if you change the
subject. (And add MdeModulePkg maintainers on cc.)

/
    Leif

On Tue, Mar 07, 2017 at 04:15:06PM -0600, Jeremy Linton wrote:
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
>  MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf
> index 4aab75b..fcd2044 100644
> --- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf
> +++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf
> @@ -29,7 +29,7 @@
>  #
>  # The following information is for reference only and not required by the build tools.
>  #
> -#  VALID_ARCHITECTURES           = IA32 X64 IPF EBC
> +#  VALID_ARCHITECTURES           = IA32 X64 IPF EBC ARM AARCH64
>  #
>  #  DRIVER_BINDING                =  gAtaBusDriverBinding
>  #  COMPONENT_NAME                =  gAtaBusComponentName
> -- 
> 2.9.3
>