<div dir="ltr"><div dir="ltr"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 24, 2021 at 9:13 AM Zhiguang Liu <<a href="mailto:zhiguang.liu@intel.com">zhiguang.liu@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From SysTableInfo Hob, get ACPI table address, and creat gPldAcpiTableGuid Hob<br>
to store it. Remove diretly adding ACPI table to ConfigurationTable.<br>
Dxe ACPI driver will parse it and install ACPI table from Guid Hob.<br>
<br>
Cc: Maurice Ma <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>><br>
Cc: Guo Dong <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>><br>
Cc: Benjamin You <<a href="mailto:benjamin.you@intel.com" target="_blank">benjamin.you@intel.com</a>><br>
Cc: Ray Ni <<a href="mailto:ray.ni@intel.com" target="_blank">ray.ni@intel.com</a>><br>
Signed-off-by: Zhiguang Liu <<a href="mailto:zhiguang.liu@intel.com" target="_blank">zhiguang.liu@intel.com</a>><br>
---<br>
UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c | 17 -----------------<br>
UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h | 5 +----<br>
UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf | 1 -<br>
UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 11 +++++++++++<br>
UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h | 2 +-<br>
UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf | 1 +<br>
6 files changed, 14 insertions(+), 23 deletions(-)<br>
<br>
diff --git a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c<br>
index 56b85b8e6d..ffd3427fb3 100644<br>
--- a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c<br>
+++ b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c<br>
@@ -99,7 +99,6 @@ BlDxeEntryPoint (<br>
{<br>
EFI_STATUS Status;<br>
EFI_HOB_GUID_TYPE *GuidHob;<br>
- SYSTEM_TABLE_INFO *SystemTableInfo;<br>
EFI_PEI_GRAPHICS_INFO_HOB *GfxInfo;<br>
ACPI_BOARD_INFO *AcpiBoardInfo;<br>
<br>
@@ -113,22 +112,6 @@ BlDxeEntryPoint (<br>
Status = ReserveResourceInGcd (TRUE, EfiGcdMemoryTypeMemoryMappedIo, 0xFED00000, SIZE_1KB, 0, ImageHandle); // HPET<br>
ASSERT_EFI_ERROR (Status);<br>
<br>
- //<br>
- // Find the system table information guid hob<br>
- //<br>
- GuidHob = GetFirstGuidHob (&gUefiSystemTableInfoGuid);<br>
- ASSERT (GuidHob != NULL);<br>
- SystemTableInfo = (SYSTEM_TABLE_INFO *)GET_GUID_HOB_DATA (GuidHob);<br>
-<br>
- //<br>
- // Install Acpi Table<br>
- //<br>
- if (SystemTableInfo->AcpiTableBase != 0 && SystemTableInfo->AcpiTableSize != 0) {<br>
- DEBUG ((DEBUG_ERROR, "Install Acpi Table at 0x%lx, length 0x%x\n", SystemTableInfo->AcpiTableBase, SystemTableInfo->AcpiTableSize));<br>
- Status = gBS->InstallConfigurationTable (&gEfiAcpiTableGuid, (VOID *)(UINTN)SystemTableInfo->AcpiTableBase);<br>
- ASSERT_EFI_ERROR (Status);<br>
- }<br>
-<br></blockquote><div><br></div><div>Note that AcpiTableDxe.inf is currently not part of the FV on UefipayloadPkg, so there won't be any tables installed after all.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
//<br>
// Find the frame buffer information and update PCDs<br>
//<br>
diff --git a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h<br>
index 512105fafd..3332a30eae 100644<br>
--- a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h<br>
+++ b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h<br>
@@ -1,7 +1,7 @@<br>
/** @file<br>
The header file of bootloader support DXE.<br>
<br>
-Copyright (c) 2014, Intel Corporation. All rights reserved.<BR><br>
+Copyright (c) 2021, Intel Corporation. All rights reserved.<BR><br>
SPDX-License-Identifier: BSD-2-Clause-Patent<br>
<br>
**/<br>
@@ -19,12 +19,9 @@ SPDX-License-Identifier: BSD-2-Clause-Patent<br>
#include <Library/IoLib.h><br>
#include <Library/HobLib.h><br>
<br>
-#include <Guid/Acpi.h><br>
#include <Guid/SmBios.h><br>
#include <Guid/SystemTableInfoGuid.h><br>
#include <Guid/AcpiBoardInfoGuid.h><br>
#include <Guid/GraphicsInfoHob.h><br>
<br>
-#include <IndustryStandard/Acpi.h><br>
-<br>
#endif<br>
diff --git a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf<br>
index 30f41f8c39..1ccb250991 100644<br>
--- a/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf<br>
+++ b/UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf<br>
@@ -42,7 +42,6 @@<br>
HobLib<br>
<br>
[Guids]<br>
- gEfiAcpiTableGuid<br>
gUefiSystemTableInfoGuid<br>
gUefiAcpiBoardInfoGuid<br>
gEfiGraphicsInfoHobGuid<br>
diff --git a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c<br>
index 7b71d37f94..14b7a732da 100644<br>
--- a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c<br>
+++ b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c<br>
@@ -235,6 +235,7 @@ BuildHobFromBl (<br>
EFI_PEI_GRAPHICS_DEVICE_INFO_HOB GfxDeviceInfo;<br>
EFI_PEI_GRAPHICS_DEVICE_INFO_HOB *NewGfxDeviceInfo;<br>
PLD_SMBIOS_TABLE *SmBiosTableHob;<br>
+ PLD_ACPI_TABLE *AcpiTableHob;<br>
<br>
//<br>
// Parse memory info and build memory HOBs<br>
@@ -287,6 +288,16 @@ BuildHobFromBl (<br>
SmBiosTableHob->SmBiosEntryPoint = SysTableInfo.SmbiosTableBase;<br>
DEBUG ((DEBUG_INFO, "Create smbios table gPldSmbiosTableGuid guid hob\n"));<br>
<br>
+ // <br>
+ // Creat ACPI table Hob<br>
+ //<br>
+ AcpiTableHob = BuildGuidHob (&gPldAcpiTableGuid, sizeof (PLD_ACPI_TABLE));<br>
+ ASSERT (AcpiTableHob != NULL);<br>
+ AcpiTableHob->PldHeader.Revision = PLD_ACPI_TABLE_REVISION;<br>
+ AcpiTableHob->PldHeader.Length = sizeof (PLD_ACPI_TABLE);<br>
+ AcpiTableHob->Rsdp = SysTableInfo.AcpiTableBase;<br>
+ DEBUG ((DEBUG_INFO, "Create smbios table gPldAcpiTableGuid guid hob\n"));<br>
+<br></blockquote><div> </div><div> DEBUG ((DEBUG_INFO, "Create acpi table ...<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
//<br>
// Create guid hob for acpi board information<br>
//<br>
diff --git a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h<br>
index e7d0d15118..a4c9da128e 100644<br>
--- a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h<br>
+++ b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h<br>
@@ -32,7 +32,7 @@<br>
#include <Guid/AcpiBoardInfoGuid.h><br>
#include <Guid/GraphicsInfoHob.h><br>
#include <UniversalPayload/SmbiosTable.h><br>
-<br>
+#include <UniversalPayload/AcpiTable.h><br>
<br>
#define LEGACY_8259_MASK_REGISTER_MASTER 0x21<br>
#define LEGACY_8259_MASK_REGISTER_SLAVE 0xA1<br>
diff --git a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf<br>
index 444f39acf3..01388b8831 100644<br>
--- a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf<br>
+++ b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf<br>
@@ -65,6 +65,7 @@<br>
gEfiGraphicsDeviceInfoHobGuid<br>
gUefiAcpiBoardInfoGuid<br>
gPldSmbiosTableGuid<br>
+ gPldAcpiTableGuid<br>
<br>
[FeaturePcd.IA32]<br>
gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode ## CONSUMES<br>
-- <br>
2.30.0.windows.2<br>
<br>
<br>
<br>
------------<br>
Groups.io Links: You receive all messages sent to this group.<br>
View/Reply Online (#75493): <a href="https://edk2.groups.io/g/devel/message/75493" rel="noreferrer" target="_blank">https://edk2.groups.io/g/devel/message/75493</a><br>
Mute This Topic: <a href="https://groups.io/mt/83045527/2917327" rel="noreferrer" target="_blank">https://groups.io/mt/83045527/2917327</a><br>
Group Owner: <a href="mailto:devel%2Bowner@edk2.groups.io" target="_blank">devel+owner@edk2.groups.io</a><br>
Unsubscribe: <a href="https://edk2.groups.io/g/devel/unsub" rel="noreferrer" target="_blank">https://edk2.groups.io/g/devel/unsub</a> [<a href="mailto:patrick.rudolph@9elements.com" target="_blank">patrick.rudolph@9elements.com</a>]<br>
------------<br>
<br>
<br>
</blockquote></div></div>
<div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links:<p> You receive all messages sent to this group. <p> <a target="_blank" href="https://edk2.groups.io/g/devel/message/75662">View/Reply Online (#75662)</a> | | <a target="_blank" href="https://groups.io/mt/83045527/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com]<br> <div width="1" style="color:white;clear:both">_._,_._,_</div>