[edk2-devel][edk2-platforms][PATCH V1 1/1] WhitleyOpenBoardPkg/BoardAcpiLib: Fix GP Fault in ACPI Enable SMI

[Oram, Isaac W]

Fixed AcpiSmm library BoardAcpiLib to use the correct SMM services
table and DynamicSiliconLibrarySmmProtocol.

Cc: Nate DeSimone <nathaniel.l.desim...@intel.com>
Cc: Chasel Chiu <chasel.c...@intel.com>
Signed-off-by: Isaac Oram <isaac.w.o...@intel.com>
---
 
Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmBoardAcpiEnableLib.inf
 |  4 ++--
 
Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmSiliconAcpiEnableLib.c
 | 16 ++++++++--------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git 
a/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmBoardAcpiEnableLib.inf
 
b/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmBoardAcpiEnableLib.inf
index fcbc94cc50..19d29ed40f 100644
--- 
a/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmBoardAcpiEnableLib.inf
+++ 
b/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmBoardAcpiEnableLib.inf
@@ -42,7 +42,7 @@
   SmmBoardAcpiEnableLib.c
 
 [Protocols]
-  gDynamicSiLibraryProtocolGuid                 ## CONSUMES
+  gDynamicSiLibrarySmmProtocolGuid                 ## CONSUMES
 
 [Depex]
-  gDynamicSiLibraryProtocolGuid
+  gDynamicSiLibrarySmmProtocolGuid
diff --git 
a/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmSiliconAcpiEnableLib.c
 
b/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmSiliconAcpiEnableLib.c
index 2b8a35c7e8..484311811b 100644
--- 
a/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmSiliconAcpiEnableLib.c
+++ 
b/Platform/Intel/WhitleyOpenBoardPkg/Library/BoardAcpiLib/SmmSiliconAcpiEnableLib.c
@@ -16,8 +16,8 @@
 #include <Library/PcdLib.h>
 #include <Library/DebugLib.h>
 #include <PchAccess.h>
-#include <Protocol/DynamicSiLibraryProtocol.h>
-#include <Library/UefiBootServicesTableLib.h>
+#include <Protocol/DynamicSiLibrarySmmProtocol.h>
+#include <Library/SmmServicesTableLib.h>
 
 /**
   Clear Port 80h
@@ -61,9 +61,9 @@ SiliconEnableAcpi (
   UINT16                           Pm1Cnt;
   UINT16                           PchPmBase;
   EFI_STATUS                       Status;
-  DYNAMIC_SI_LIBARY_PROTOCOL       *DynamicSiLibraryProtocol = NULL;
+  DYNAMIC_SI_LIBARY_SMM_PROTOCOL   *DynamicSiLibrarySmmProtocol = NULL;
 
-  Status = gBS->LocateProtocol (&gDynamicSiLibraryProtocolGuid, NULL, 
&DynamicSiLibraryProtocol);
+  Status = gSmst->SmmLocateProtocol (&gDynamicSiLibrarySmmProtocolGuid, NULL, 
&DynamicSiLibrarySmmProtocol);
   if (EFI_ERROR (Status)) {
     ASSERT_EFI_ERROR (Status);
     return Status;
@@ -72,7 +72,7 @@ SiliconEnableAcpi (
   //
   // Init Power Management I/O Base aka ACPI Base
   //
-  PchPmBase = DynamicSiLibraryProtocol->PmcGetAcpiBase ();
+  PchPmBase = DynamicSiLibrarySmmProtocol->PmcGetAcpiBase ();
 
   SmiEn = IoRead32 (PchPmBase + R_ACPI_IO_SMI_EN);
 
@@ -112,9 +112,9 @@ SiliconDisableAcpi (
   UINT16                           Pm1Cnt;
   UINT16                           PchPmBase;
   EFI_STATUS                       Status;
-  DYNAMIC_SI_LIBARY_PROTOCOL       *DynamicSiLibraryProtocol = NULL;
+  DYNAMIC_SI_LIBARY_SMM_PROTOCOL   *DynamicSiLibrarySmmProtocol = NULL;
 
-  Status = gBS->LocateProtocol (&gDynamicSiLibraryProtocolGuid, NULL, 
&DynamicSiLibraryProtocol);
+  Status = gSmst->SmmLocateProtocol (&gDynamicSiLibrarySmmProtocolGuid, NULL, 
&DynamicSiLibrarySmmProtocol);
   if (EFI_ERROR (Status)) {
     ASSERT_EFI_ERROR (Status);
     return Status;
@@ -123,7 +123,7 @@ SiliconDisableAcpi (
   //
   // Init Power Management I/O Base aka ACPI Base
   //
-  PchPmBase = DynamicSiLibraryProtocol->PmcGetAcpiBase ();
+  PchPmBase = DynamicSiLibrarySmmProtocol->PmcGetAcpiBase ();
 
   Pm1Cnt = IoRead16 (PchPmBase + R_ACPI_IO_PM1_CNT);
 
-- 
2.27.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79123): https://edk2.groups.io/g/devel/message/79123
Mute This Topic: https://groups.io/mt/84829392/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-