[edk2-devel] [Patch V3 7/9] MdeModulePkg: Add new structure for the Universal Payload ACPI Table Info Hob

Zhiguang Liu posted 9 patches 4 years, 8 months ago
There is a newer version of this series
[edk2-devel] [Patch V3 7/9] MdeModulePkg: Add new structure for the Universal Payload ACPI Table Info Hob
Posted by Zhiguang Liu 4 years, 8 months ago
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>

Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
---
 MdeModulePkg/Include/UniversalPayload/AcpiTable.h | 28 ++++++++++++++++++++++++++++
 MdeModulePkg/MdeModulePkg.dec                     |  3 +++
 2 files changed, 31 insertions(+)

diff --git a/MdeModulePkg/Include/UniversalPayload/AcpiTable.h b/MdeModulePkg/Include/UniversalPayload/AcpiTable.h
new file mode 100644
index 0000000000..a85c555d9c
--- /dev/null
+++ b/MdeModulePkg/Include/UniversalPayload/AcpiTable.h
@@ -0,0 +1,28 @@
+/** @file
+ Define the structure for the Payload APCI table.
+
+Copyright (c) 2021, Intel Corporation. All rights reserved.<BR>
+SPDX-License-Identifier: BSD-2-Clause-Patent
+
+**/
+
+#ifndef _PLD_ACPI_TABLE_H_
+#define _PLD_ACPI_TABLE_H_
+
+#include <Uefi.h>
+#include <UniversalPayload/UniversalPayload.h>
+
+#pragma pack(1)
+
+typedef struct {
+  PLD_GENERIC_HEADER   PldHeader;
+  EFI_PHYSICAL_ADDRESS Rsdp;
+} PLD_ACPI_TABLE;
+
+#pragma pack()
+
+#define PLD_ACPI_TABLE_REVISION 1
+
+extern GUID gPldAcpiTableGuid;
+
+#endif //_PLD_ACPI_TABLE_H_
diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec
index 45324d2e20..4da472d7b9 100644
--- a/MdeModulePkg/MdeModulePkg.dec
+++ b/MdeModulePkg/MdeModulePkg.dec
@@ -416,6 +416,9 @@
   ## Include/UniversalPayload/SmbiosTable.h
   gPldSmbiosTableGuid = { 0x590a0d26, 0x06e5, 0x4d20, { 0x8a, 0x82, 0x59, 0xea, 0x1b, 0x34, 0x98, 0x2d } }
 
+  ## Include/UniversalPayload/AcpiTable.h
+  gPldAcpiTableGuid = { 0x9f9a9506, 0x5597, 0x4515, { 0xba, 0xb6, 0x8b, 0xcd, 0xe7, 0x84, 0xba, 0x87 } }
+
 [Ppis]
   ## Include/Ppi/AtaController.h
   gPeiAtaControllerPpiGuid       = { 0xa45e60d1, 0xc719, 0x44aa, { 0xb0, 0x7a, 0xaa, 0x77, 0x7f, 0x85, 0x90, 0x6d }}
-- 
2.30.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#76062): https://edk2.groups.io/g/devel/message/76062
Mute This Topic: https://groups.io/mt/83304949/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [Patch V3 7/9] MdeModulePkg: Add new structure for the Universal Payload ACPI Table Info Hob
Posted by Wu, Hao A 4 years, 8 months ago
> -----Original Message-----
> From: Liu, Zhiguang <zhiguang.liu@intel.com>
> Sent: Friday, June 4, 2021 5:42 PM
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Wu,
> Hao A <hao.a.wu@intel.com>
> Subject: [Patch V3 7/9] MdeModulePkg: Add new structure for the Universal
> Payload ACPI Table Info Hob
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> 
> Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
>  MdeModulePkg/Include/UniversalPayload/AcpiTable.h | 28
> ++++++++++++++++++++++++++++
>  MdeModulePkg/MdeModulePkg.dec                     |  3 +++
>  2 files changed, 31 insertions(+)
> 
> diff --git a/MdeModulePkg/Include/UniversalPayload/AcpiTable.h
> b/MdeModulePkg/Include/UniversalPayload/AcpiTable.h
> new file mode 100644
> index 0000000000..a85c555d9c
> --- /dev/null
> +++ b/MdeModulePkg/Include/UniversalPayload/AcpiTable.h
> @@ -0,0 +1,28 @@
> +/** @file
> 
> + Define the structure for the Payload APCI table.
> 
> +
> 
> +Copyright (c) 2021, Intel Corporation. All rights reserved.<BR>
> 
> +SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> +


Could you help to add the specification reference information in the header file description comment?

With this handled:
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>

Best Regards,
Hao Wu


> 
> +**/
> 
> +
> 
> +#ifndef _PLD_ACPI_TABLE_H_
> 
> +#define _PLD_ACPI_TABLE_H_
> 
> +
> 
> +#include <Uefi.h>
> 
> +#include <UniversalPayload/UniversalPayload.h>
> 
> +
> 
> +#pragma pack(1)
> 
> +
> 
> +typedef struct {
> 
> +  PLD_GENERIC_HEADER   PldHeader;
> 
> +  EFI_PHYSICAL_ADDRESS Rsdp;
> 
> +} PLD_ACPI_TABLE;
> 
> +
> 
> +#pragma pack()
> 
> +
> 
> +#define PLD_ACPI_TABLE_REVISION 1
> 
> +
> 
> +extern GUID gPldAcpiTableGuid;
> 
> +
> 
> +#endif //_PLD_ACPI_TABLE_H_
> 
> diff --git a/MdeModulePkg/MdeModulePkg.dec
> b/MdeModulePkg/MdeModulePkg.dec
> index 45324d2e20..4da472d7b9 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -416,6 +416,9 @@
>    ## Include/UniversalPayload/SmbiosTable.h
> 
>    gPldSmbiosTableGuid = { 0x590a0d26, 0x06e5, 0x4d20, { 0x8a, 0x82, 0x59,
> 0xea, 0x1b, 0x34, 0x98, 0x2d } }
> 
> 
> 
> +  ## Include/UniversalPayload/AcpiTable.h
> 
> +  gPldAcpiTableGuid = { 0x9f9a9506, 0x5597, 0x4515, { 0xba, 0xb6, 0x8b, 0xcd,
> 0xe7, 0x84, 0xba, 0x87 } }
> 
> +
> 
>  [Ppis]
> 
>    ## Include/Ppi/AtaController.h
> 
>    gPeiAtaControllerPpiGuid       = { 0xa45e60d1, 0xc719, 0x44aa, { 0xb0, 0x7a,
> 0xaa, 0x77, 0x7f, 0x85, 0x90, 0x6d }}
> 
> --
> 2.30.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#76185): https://edk2.groups.io/g/devel/message/76185
Mute This Topic: https://groups.io/mt/83304949/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-