[edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI

Laszlo Ersek posted 12 patches 7 years, 7 months ago
Failed in applying to current master (apply log)
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
[edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI
Posted by Laszlo Ersek 7 years, 7 months ago
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.

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

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI
Posted by Ard Biesheuvel 7 years, 7 months ago
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
>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI
Posted by Laszlo Ersek 7 years, 7 months ago
On 03/28/17 15:57, Ard Biesheuvel wrote:
> 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!
> 

Thank you very much; pushed as 3efa3f3d05ea..89ad870fbff0.

Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel