X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=blobdiff_plain;f=CorebootModulePkg%2FLibrary%2FCbParseLib%2FCbParseLib.c;h=da227dea5e275104efa92ce3fddb62344d32881e;hp=0909b0f492023b3a029065906006db6bafdcaf6e;hb=271d8cd7df0c346f19803938d98fe7cbb7b63be4;hpb=f75c7478280b96d191e86856c0b1c49ff711c284

diff --git a/CorebootModulePkg/Library/CbParseLib/CbParseLib.c b/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
index 0909b0f492..da227dea5e 100644
--- a/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
+++ b/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
@@ -18,6 +18,7 @@
 #include <Library/BaseMemoryLib.h>
 #include <Library/DebugLib.h>
 #include <Library/PcdLib.h>
+#include <Library/IoLib.h>
 #include <Library/CbParseLib.h>
 
 #include <IndustryStandard/Acpi.h>
@@ -477,6 +478,39 @@ CbParseFadtInfo (
         ASSERT(Fadt->Pm1aEvtBlk != 0);
         ASSERT(Fadt->Gpe0Blk != 0);
 
+        DEBUG_CODE_BEGIN ();
+          BOOLEAN    SciEnabled;
+
+          //
+          // Check the consistency of SCI enabling
+          //
+
+          //
+          // Get SCI_EN value
+          //
+          if (Fadt->Pm1CntLen == 4) {
+            SciEnabled = (IoRead32 (Fadt->Pm1aCntBlk) & BIT0)? TRUE : FALSE;
+          } else {
+            //
+            // if (Pm1CntLen == 2), use 16 bit IO read;
+            // if (Pm1CntLen != 2 && Pm1CntLen != 4), use 16 bit IO read as a fallback
+            //
+            SciEnabled = (IoRead16 (Fadt->Pm1aCntBlk) & BIT0)? TRUE : FALSE;
+          }
+
+          if (!(Fadt->Flags & EFI_ACPI_5_0_HW_REDUCED_ACPI) &&
+              (Fadt->SmiCmd == 0) &&
+              !SciEnabled) {
+            //
+            // The ACPI enabling status is inconsistent: SCI is not enabled but ACPI
+            // table does not provide a means to enable it through FADT->SmiCmd
+            //
+            DEBUG ((DEBUG_ERROR, "ERROR: The ACPI enabling status is inconsistent: SCI is not"
+              " enabled but the ACPI table does not provide a means to enable it through FADT->SmiCmd."
+              " This may cause issues in OS.\n"));
+            ASSERT (FALSE);
+          }
+        DEBUG_CODE_END ();
         return RETURN_SUCCESS;
       }
     }