From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-it0-x234.google.com (mail-it0-x234.google.com [IPv6:2607:f8b0:4001:c0b::234]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D747821BC6A23 for ; Tue, 28 Mar 2017 06:57:11 -0700 (PDT) Received: by mail-it0-x234.google.com with SMTP id 190so19347864itm.0 for ; Tue, 28 Mar 2017 06:57:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=g6MU5SMUqmZfEAaxGYhuTRSc4X5Hn/i78yIPhaEmqc4=; b=YTGgABd/5lBePzGZZOGaalG4eeObnTXln7RJF+FezerNg3AcHfxup4lb/2sPEXZ3hk CnMEk6l/c49QcQrHVwQ9Og72UBHButjT/N0gjzb3lcE6R+wVbi1EPohqGCas2HwKszYP y6Dgwg2eHwGdjPGSORglFbzyrXPhkjbMTbEvY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=g6MU5SMUqmZfEAaxGYhuTRSc4X5Hn/i78yIPhaEmqc4=; b=t1W/lXmXkqEZjI8shMQVnqq9gkGqXyOIy9DRc+Sb42RhlOYAFxHRGUoB8NnB3uNo3a 11dcHWS+MJTXKs7DXO8UHtvebZVvW5e6mXfFlUULPwKtfAe6yVAzzZwjbEbIA4+tbmxd BQoTTz0IrLfwtL34Fk2wp+CHTnx8iNEZ9VcgOwv181HANfE0UB+uPbwZySFZqoUbsBmA oVydIuWHd6aQJWtu5dwkZDTLG0sI5wD/JOpVZc+4Yq0EXuwGLXkwKiwJlQConpISwiqv lAJe4lZ5T2k5/sOxHv8V/9hxxS9K/zAJxfG1EhYu2KMlIABRuQNo6Jmw+Ba51KKlqlMB l/dg== X-Gm-Message-State: AFeK/H3QCJLJE25stHNJvkh7X389cRMIpQ8MDdtoRWszsu504R+96YrNNVHmk/CtW4qWQ+Z8fVwC+Mof9X0W40QM X-Received: by 10.107.132.155 with SMTP id o27mr25589177ioi.138.1490709431020; Tue, 28 Mar 2017 06:57:11 -0700 (PDT) MIME-Version: 1.0 Received: by 10.107.10.27 with HTTP; Tue, 28 Mar 2017 06:57:10 -0700 (PDT) In-Reply-To: <20170328135033.24238-1-lersek@redhat.com> References: <20170328135033.24238-1-lersek@redhat.com> From: Ard Biesheuvel Date: Tue, 28 Mar 2017 14:57:10 +0100 Message-ID: To: Laszlo Ersek Cc: edk2-devel-01 , Leif Lindholm Subject: Re: [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Mar 2017 13:57:12 -0000 Content-Type: text/plain; charset=UTF-8 On 28 March 2017 at 14:50, Laszlo Ersek wrote: > This is version 4 of the series posted (as v3) previously at > . > > 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 > Cc: Leif Lindholm > > 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 >