Enable TPM related HDAT sections for all service processors

Change-Id: I302a3c6ce5a6abfefc7f91ae46453e99ab255105
CMVC-Prereq: 1018579
RTC: 171851
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42213
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 7 insertions, 8 deletions

diff --git a/src/usr/runtime/populate_hbruntime.C b/src/usr/runtime/populate_hbruntime.C
index e7c6d1c8b..d0a43ed1d 100644
--- a/src/usr/runtime/populate_hbruntime.C
+++ b/src/usr/runtime/populate_hbruntime.C
@@ -1621,6 +1621,13 @@ errlHndl_t populate_TpmInfoByNode()
         auto i2cAryOff =
             l_pcrd->hdatPcrdIntData[HDAT::HDAT_PCRD_DA_HOST_I2C].hdatOffset;
 
+        // If pointer pair's offset value is 0, advance to next PCRD instance
+        // as this one has no I2C links
+        if(!i2cAryOff)
+        {
+            continue;
+        }
+
         // Convert i2c array header offset to a pointer to the i2c array header
         const auto l_hostI2cPcrdHdrPtr =
            reinterpret_cast<HDAT::hdatHDIFDataArray_t*>(l_pcrdAddr + i2cAryOff);
@@ -1941,14 +1948,6 @@ errlHndl_t populate_hbTpmInfo()
     errlHndl_t l_elog = nullptr;
 
     do {
-        // TODO RTC 171851 Remove FSP restriction when FSP code provides
-        // Node TPM Related Data
-
-        // Skip populating HDAT TPM Node Related Data on FSP systems
-        if (INITSERVICE::spBaseServicesEnabled())
-        {
-            break;
-        }
 
         TRACFCOMP(g_trac_runtime, "Running populate_hbTpmInfo");