diff mbox

[edk2] Platforms/ARM/Juno: limit ACPI support to v5.0 and higher

Message ID 1472144947-17736-1-git-send-email-sudeep.holla@arm.com
State New
Headers show

Commit Message

Sudeep Holla Aug. 25, 2016, 5:09 p.m. UTC
The ACPI spec predates the AARCH64 architecture by 5 versions, so there
is no point in supporting anything below v5.0. So set the PCD that
controls the ACPI table generation to the appropriate value.

Based on the commit e0692789058e ("ArmVirtPkg/ArmVirtQemu: limit ACPI
support to v5.0 and higher") in the upstream TianoCore by Ard Biesheuvel

Contributed-under: TianoCore Contribution Agreement 1.0
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Evan Lloyd <Evan.Lloyd@arm.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

---
 Platforms/ARM/Juno/ArmJuno.dsc | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.7.4

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

Comments

Evan Lloyd Aug. 31, 2016, 10:29 a.m. UTC | #1
>-----Original Message-----

>From: Sudeep Holla [mailto:sudeep.holla@arm.com]

>Sent: 25 August 2016 18:09

>To: edk2-devel@lists.01.org; Linaro UEFI; Evan Lloyd; Leif Lindholm

>Cc: Sudeep Holla; ard.biesheuvel@linaro.org

>Subject: [PATCH] Platforms/ARM/Juno: limit ACPI support to v5.0 and higher

>

>The ACPI spec predates the AARCH64 architecture by 5 versions, so there

>is no point in supporting anything below v5.0. So set the PCD that

>controls the ACPI table generation to the appropriate value.

>

>Based on the commit e0692789058e ("ArmVirtPkg/ArmVirtQemu: limit ACPI

>support to v5.0 and higher") in the upstream TianoCore by Ard Biesheuvel

>

>Contributed-under: TianoCore Contribution Agreement 1.0

>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>

>Cc: Leif Lindholm <leif.lindholm@linaro.org>

>Cc: Evan Lloyd <Evan.Lloyd@arm.com>

>Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

>---

> Platforms/ARM/Juno/ArmJuno.dsc | 4 ++++

> 1 file changed, 4 insertions(+)

>

>diff --git a/Platforms/ARM/Juno/ArmJuno.dsc

>b/Platforms/ARM/Juno/ArmJuno.dsc

>index c51d8f2e21ab..3029999bc66f 100644

>--- a/Platforms/ARM/Juno/ArmJuno.dsc

>+++ b/Platforms/ARM/Juno/ArmJuno.dsc

>@@ -183,6 +183,10 @@

>   #

>   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0300

>   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosDocRev|0x0

>+  #

>+  # ACPI Table Version

>+  #

>+  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20

>

> [PcdsPatchableInModule]

>   # Console Resolution (Full HD)

>--

>2.7.4

Reviewed-by: Evan Lloyd <Evan.Lloyd@arm.com>


IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel Aug. 31, 2016, 11:08 a.m. UTC | #2
On 31 August 2016 at 11:29, Evan Lloyd <Evan.Lloyd@arm.com> wrote:
>

>>-----Original Message-----

>>From: Sudeep Holla [mailto:sudeep.holla@arm.com]

>>Sent: 25 August 2016 18:09

>>To: edk2-devel@lists.01.org; Linaro UEFI; Evan Lloyd; Leif Lindholm

>>Cc: Sudeep Holla; ard.biesheuvel@linaro.org

>>Subject: [PATCH] Platforms/ARM/Juno: limit ACPI support to v5.0 and higher

>>

>>The ACPI spec predates the AARCH64 architecture by 5 versions, so there

>>is no point in supporting anything below v5.0. So set the PCD that

>>controls the ACPI table generation to the appropriate value.

>>

>>Based on the commit e0692789058e ("ArmVirtPkg/ArmVirtQemu: limit ACPI

>>support to v5.0 and higher") in the upstream TianoCore by Ard Biesheuvel

>>

>>Contributed-under: TianoCore Contribution Agreement 1.0

>>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>

>>Cc: Leif Lindholm <leif.lindholm@linaro.org>

>>Cc: Evan Lloyd <Evan.Lloyd@arm.com>

>>Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

>>---

>> Platforms/ARM/Juno/ArmJuno.dsc | 4 ++++

>> 1 file changed, 4 insertions(+)

>>

>>diff --git a/Platforms/ARM/Juno/ArmJuno.dsc

>>b/Platforms/ARM/Juno/ArmJuno.dsc

>>index c51d8f2e21ab..3029999bc66f 100644

>>--- a/Platforms/ARM/Juno/ArmJuno.dsc

>>+++ b/Platforms/ARM/Juno/ArmJuno.dsc

>>@@ -183,6 +183,10 @@

>>   #

>>   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0300

>>   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosDocRev|0x0

>>+  #

>>+  # ACPI Table Version

>>+  #

>>+  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20

>>

>> [PcdsPatchableInModule]

>>   # Console Resolution (Full HD)

>>--

>>2.7.4

> Reviewed-by: Evan Lloyd <Evan.Lloyd@arm.com>

>


Pushed, thanks.
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
diff mbox

Patch

diff --git a/Platforms/ARM/Juno/ArmJuno.dsc b/Platforms/ARM/Juno/ArmJuno.dsc
index c51d8f2e21ab..3029999bc66f 100644
--- a/Platforms/ARM/Juno/ArmJuno.dsc
+++ b/Platforms/ARM/Juno/ArmJuno.dsc
@@ -183,6 +183,10 @@ 
   #
   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0300
   gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosDocRev|0x0
+  #
+  # ACPI Table Version
+  #
+  gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20
 
 [PcdsPatchableInModule]
   # Console Resolution (Full HD)