Re: [PATCH v3 0/2] Add Ramdisk support to ArmVirtPkg platforms

["Sajjan, Vikas C" <vikas.cha.sajjan@hpe.com>]

-----Original Message-----
From: Laszlo Ersek [mailto:lersek@redhat.com] 
Sent: Friday, August 19, 2016 6:58 PM
To: Sajjan, Vikas C <vikas.cha.sajjan@hpe.com>; edk2-devel@ml01.01.org; ard.biesheuvel@linaro.org; leif.lindholm@linaro.org
Cc: Wei, Dong (Office of the CTO) <dong.wei@hpe.com>
Subject: Re: [PATCH v3 0/2] Add Ramdisk support to ArmVirtPkg platforms

On 08/19/16 08:55, Vikas C Sajjan wrote:
> This series adds ramdisk driver support to ArmVirtPkg platforms.
> It actually ports OvmfPkg commit 259d87146b07 to ArmVirtPkg.
> 
> changes since v2:
>    - Made changes as suggested by Laszlo Ersek <lersek@redhat.com>
> 	a] Moved inclusion of AcpiTableDxe.inf to end of file
> 	b] Moved gEfiMdeModulePkgTokenSpaceGuid.PcdInstallAcpiSdtProtocol to proper location
>         c] fixed minor nits
> 
> changes since v1:
>    - Made changes as suggested by Laszlo Ersek <lersek@redhat.com>
>       a] clarified in the commit message (including the subject line)
>          that this patch is actually about adding the RAM Disk driver to the
>          ArmVirtPkg platforms, not just the NFIT reporting feature of the driver
>       b] enabled feature for all of ArmVirtQemu, ArmVirtQemuKernel, and ArmVirtXen.
>       c] prepended a patch to the series that extracts
>          "MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf"
>          from all of the DSC files, into "ArmVirt.dsc.inc"
> 
> 
> Vikas C Sajjan (2):
>   ArmVirtPkg: Move inclusion of AcpiTableDxe.inf to ArmVirt.dsc.inc
>   ArmVirtPkg: Add Ramdisk support to ArmVirtPkg platforms
> 
>  ArmVirtPkg/ArmVirt.dsc.inc           | 17 +++++++++++++++++
>  ArmVirtPkg/ArmVirtQemu.dsc           |  1 -
>  ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc |  5 +++++
>  ArmVirtPkg/ArmVirtQemuKernel.dsc     |  1 -
>  ArmVirtPkg/ArmVirtRules.fdf.inc      |  2 ++
>  ArmVirtPkg/ArmVirtXen.dsc            |  1 -
>  ArmVirtPkg/ArmVirtXen.fdf            |  5 +++++
>  7 files changed, 29 insertions(+), 3 deletions(-)
> 

Series
Reviewed-by: Laszlo Ersek <lersek@redhat.com>

Thank you, Laszlo.

Ard, can you please commit the series for Vikas? My brain is fried, I need to get out. :)

Thanks!
Laszlo