commit 79f4f6f0c99c2cc8b03304d0fec9bd657b313312
author Leahy, Leroy P <leroy.p.leahy@intel.com> Wed Feb 24 18:53:31 2016 -0800
committer Prince Agyeman <prince.agyeman@intel.com> Fri Feb 26 10:02:25 2016 -0800
tree de51ab1a8c4089521ad4af1025753e8a0a78bb75
parent ffbb5ae3ba7da2ece8dbf116b1eb0718c346d19b

CorebootModulePkg-CbParseLib: Add ACPI table verification

Verify the register address in the FADT.

TEST=Build and run on Galileo Gen2 when the FADT was not present.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com>
Reviewed-by: Erik Bjorge <erik.c.bjorge@intel.com>
Reviewed-by: Prince Agyeman <prince.agyeman@intel.com>

CorebootModulePkg/Library/CbParseLib/CbParseLib.c

diff --git a/CorebootModulePkg/Library/CbParseLib/CbParseLib.c b/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
index 8509bd2..d9717f4 100644
--- a/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
+++ b/CorebootModulePkg/Library/CbParseLib/CbParseLib.c
@@ -478,6 +478,15 @@
           DEBUG ((EFI_D_INFO, "PmGpeEn Reg 0x%x\n", *pPmGpeEnReg));

         }

 

+        //
+        // Verify values for proper operation
+        //
+        ASSERT(Fadt->Pm1aCntBlk != 0);
+        ASSERT(Fadt->PmTmrBlk != 0);
+        ASSERT(Fadt->ResetReg.Address != 0);
+        ASSERT(Fadt->Pm1aEvtBlk != 0);
+        ASSERT(Fadt->Gpe0Blk != 0);
+
         return RETURN_SUCCESS;

       }

     }