public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: Laszlo Ersek <lersek@redhat.com>
Cc: edk2-devel-01 <edk2-devel@lists.01.org>,
	Leif Lindholm <leif.lindholm@linaro.org>
Subject: Re: [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI
Date: Tue, 28 Mar 2017 14:57:10 +0100	[thread overview]
Message-ID: <CAKv+Gu9qRBHTdiEMQ0v4VP1xc13c_tecrUktz=bk4HrTfA9JqA@mail.gmail.com> (raw)
In-Reply-To: <20170328135033.24238-1-lersek@redhat.com>

On 28 March 2017 at 14:50, Laszlo Ersek <lersek@redhat.com> wrote:
> This is version 4 of the series posted (as v3) previously at
> <https://lists.01.org/pipermail/edk2-devel/2017-March/009000.html>.
>
> Main changes in this version:
>
> * move gEdkiiPlatformHasAcpiGuid from MdeModulePkg to EmbeddedPkg (based
>   on IRC discussion),
>
> * move PlatformHasAcpiLib from ArmPkg to EmbeddedPkg (based on IRC
>   discussion),
>
> * the only two packages being touched are ArmVirtPkg and EmbeddedPkg
>   now,
>
> * retest with a Fedora 25 guest (4.8.7-300.fc25.aarch64).
>
> Updates are also noted in detail on each patch individually.
>
> The series is fully reviewed by Ard (thanks!), I'm sending this out for
> a final ACK before pushing.
>

Looks good to me. Thanks for spending the time and effort!

-- 
Ard.


> Repo:   https://github.com/lersek/edk2.git
> Branch: dt_acpi_v4
> RHBZ:   https://bugzilla.redhat.com/show_bug.cgi?id=1430262
>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
>
> Thanks,
> Laszlo
>
> Laszlo Ersek (12):
>   Revert "ArmVirtPkg/FdtClientDxe: make DT table installation !ACPI
>     dependent"
>   Revert "ArmVirtPkg/FdtClientDxe: install DT configuration table at
>     ReadyToBoot"
>   ArmVirtPkg/XenAcpiPlatformDxe: don't cast UINT64 to pointer directly
>   EmbeddedPkg: introduce EDKII Platform Has ACPI GUID
>   EmbeddedPkg: introduce PlatformHasAcpiLib
>   EmbeddedPkg: introduce EDKII Platform Has Device Tree GUID
>   ArmVirtPkg: add PlatformHasAcpiDtDxe
>   ArmVirtPkg: add XenPlatformHasAcpiDtDxe
>   ArmVirtPkg: enable AcpiTableDxe and EFI_ACPI_TABLE_PROTOCOL
>     dynamically
>   ArmVirtPkg/FdtClientDxe: install DT as sysconfig table in protocol
>     notify
>   ArmVirtPkg/PlatformHasAcpiDtDxe: don't expose DT if QEMU provides ACPI
>   ArmVirtPkg: remove PURE_ACPI_BOOT_ENABLE and PcdPureAcpiBoot
>
>  ArmVirtPkg/ArmVirt.dsc.inc                                     |   9 +-
>  ArmVirtPkg/ArmVirtQemu.dsc                                     |   1 +
>  ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc                           |   1 +
>  ArmVirtPkg/ArmVirtQemuKernel.dsc                               |   1 +
>  ArmVirtPkg/ArmVirtXen.dsc                                      |   1 +
>  ArmVirtPkg/ArmVirtXen.fdf                                      |   1 +
>  ArmVirtPkg/FdtClientDxe/FdtClientDxe.c                         | 116 ++++++++++++++------
>  ArmVirtPkg/FdtClientDxe/FdtClientDxe.inf                       |   4 +-
>  ArmVirtPkg/PlatformHasAcpiDtDxe/PlatformHasAcpiDtDxe.c         |  86 +++++++++++++++
>  ArmVirtPkg/PlatformHasAcpiDtDxe/PlatformHasAcpiDtDxe.inf       |  44 ++++++++
>  ArmVirtPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.c             |   3 +-
>  ArmVirtPkg/XenPlatformHasAcpiDtDxe/XenPlatformHasAcpiDtDxe.c   |  72 ++++++++++++
>  ArmVirtPkg/XenPlatformHasAcpiDtDxe/XenPlatformHasAcpiDtDxe.inf |  42 +++++++
>  EmbeddedPkg/EmbeddedPkg.dec                                    |   6 +
>  EmbeddedPkg/Include/Guid/PlatformHasAcpi.h                     |  35 ++++++
>  EmbeddedPkg/Include/Guid/PlatformHasDeviceTree.h               |  35 ++++++
>  EmbeddedPkg/Library/PlatformHasAcpiLib/PlatformHasAcpiLib.c    |  36 ++++++
>  EmbeddedPkg/Library/PlatformHasAcpiLib/PlatformHasAcpiLib.inf  |  40 +++++++
>  18 files changed, 495 insertions(+), 38 deletions(-)
>  create mode 100644 ArmVirtPkg/PlatformHasAcpiDtDxe/PlatformHasAcpiDtDxe.c
>  create mode 100644 ArmVirtPkg/PlatformHasAcpiDtDxe/PlatformHasAcpiDtDxe.inf
>  create mode 100644 ArmVirtPkg/XenPlatformHasAcpiDtDxe/XenPlatformHasAcpiDtDxe.c
>  create mode 100644 ArmVirtPkg/XenPlatformHasAcpiDtDxe/XenPlatformHasAcpiDtDxe.inf
>  create mode 100644 EmbeddedPkg/Include/Guid/PlatformHasAcpi.h
>  create mode 100644 EmbeddedPkg/Include/Guid/PlatformHasDeviceTree.h
>  create mode 100644 EmbeddedPkg/Library/PlatformHasAcpiLib/PlatformHasAcpiLib.c
>  create mode 100644 EmbeddedPkg/Library/PlatformHasAcpiLib/PlatformHasAcpiLib.inf
>
> --
> 2.9.3
>


  parent reply	other threads:[~2017-03-28 13:57 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-28 13:50 [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 01/12] Revert "ArmVirtPkg/FdtClientDxe: make DT table installation !ACPI dependent" Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 02/12] Revert "ArmVirtPkg/FdtClientDxe: install DT configuration table at ReadyToBoot" Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 03/12] ArmVirtPkg/XenAcpiPlatformDxe: don't cast UINT64 to pointer directly Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 04/12] EmbeddedPkg: introduce EDKII Platform Has ACPI GUID Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 05/12] EmbeddedPkg: introduce PlatformHasAcpiLib Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 06/12] EmbeddedPkg: introduce EDKII Platform Has Device Tree GUID Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 07/12] ArmVirtPkg: add PlatformHasAcpiDtDxe Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 08/12] ArmVirtPkg: add XenPlatformHasAcpiDtDxe Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 09/12] ArmVirtPkg: enable AcpiTableDxe and EFI_ACPI_TABLE_PROTOCOL dynamically Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 10/12] ArmVirtPkg/FdtClientDxe: install DT as sysconfig table in protocol notify Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 11/12] ArmVirtPkg/PlatformHasAcpiDtDxe: don't expose DT if QEMU provides ACPI Laszlo Ersek
2017-03-28 13:50 ` [PATCH v4 12/12] ArmVirtPkg: remove PURE_ACPI_BOOT_ENABLE and PcdPureAcpiBoot Laszlo Ersek
2017-03-28 13:57 ` Ard Biesheuvel [this message]
2017-03-28 14:09   ` [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI Laszlo Ersek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAKv+Gu9qRBHTdiEMQ0v4VP1xc13c_tecrUktz=bk4HrTfA9JqA@mail.gmail.com' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox