Add Reset to TPM's I2C Bus for MPIPLs

This commit updates the general I2C reset function to only reset
certain engine(s) of the I2C master.  This new functionality is
then used to reset all of the processor I2C engines that can
drive the TPMs on MPIPLs.

Change-Id: Ie19e93233b5012b69d59bfc3f485ce2914d665da
CQ:SW419116
Backport:release-fips910
Backport:release-fips900
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54840
CI-Ready: Nicholas E. Bofferding <bofferdn@us.ibm.com>
CI-Ready: Marshall J. Wilks <mjwilks@us.ibm.com>
CI-Ready: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

2 files changed, 54 insertions, 12 deletions

diff --git a/src/usr/i2c/i2c.C b/src/usr/i2c/i2c.C
index 95074f82d..3aed2eb02 100755
--- a/src/usr/i2c/i2c.C
+++ b/src/usr/i2c/i2c.C
@@ -3179,7 +3179,8 @@ enum i2cProcessOperation
 errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
                                      i2cProcessOperation i_processOperation,
                                      uint64_t            i_busSpeed,
-                                     bool                i_functional )
+                                     bool                i_functional,
+                                     i2cEngineSelect     i_engineSelect )
 {
     errlHndl_t err = NULL;
     bool error_found = false;
@@ -3193,9 +3194,10 @@ errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
     TARGETING::ATTR_I2C_BUS_SPEED_ARRAY_type speed_array;
 
     TRACFCOMP( g_trac_i2c,
-               ENTER_MRK"i2cProcessActiveMasters(): Type=0x%X, "
-               "Operation=%d Bus Speed=%d Functional=%d",
-               i_processType, i_processOperation, i_busSpeed, i_functional );
+               ENTER_MRK"i2cProcessActiveMasters(): Type=0x%X "
+               "Operation=%d Bus Speed=%d Functional=%d engineSelect=0x%.2X",
+               i_processType, i_processOperation, i_busSpeed, i_functional,
+               i_engineSelect );
     do
     {
 
@@ -3357,7 +3359,7 @@ errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
                 }
             }
 
-            for( size_t engine = 0;
+            for( uint8_t engine = 0;
                  engine < I2C_BUS_ATTR_MAX_ENGINE;
                  engine++ )
             {
@@ -3373,8 +3375,7 @@ errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
                     continue;
                 }
 
-                // Never touch engine 0 for Host -- the SBE owns
-                // it
+                // Never touch engine 0 for Host -- the SBE owns it
                 if ( ( engine == 0 ) &&
                      (io_args.switches.useHostI2C == 1) )
                 {
@@ -3383,6 +3384,16 @@ errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
                     continue;
                 }
 
+                // Only operate on selected engines
+                if ( ! ( i2cEngineToEngineSelect(engine) & i_engineSelect ) )
+                {
+                    TRACFCOMP( g_trac_i2c,INFO_MRK
+                        "Skipping engine %d because i_engineSelect=0x%.2X",
+                        engine, i_engineSelect );
+                    continue;
+                }
+
+
                 // Look for any device on this engine based on speed_array
                 bool skip = true;
                 size_t l_numPorts = I2C_BUS_ATTR_MAX_PORT;
@@ -3647,19 +3658,21 @@ errlHndl_t i2cProcessActiveMasters ( i2cProcessType      i_processType,
  *        Set bus speed from the MRW value.
  */
 errlHndl_t i2cResetActiveMasters ( i2cProcessType i_resetType,
-                                   bool i_functional )
+                                   bool i_functional,
+                                   i2cEngineSelect i_engineSelect )
 {
     errlHndl_t err = NULL;
 
     TRACFCOMP( g_trac_i2c,
                ENTER_MRK"i2cResetActiveMasters(): i2cProcessType=0x%X, "
-               "i_functional=%d",
-               i_resetType, i_functional );
+               "i_functional=%d, i_engineSelect=0x%.2X",
+               i_resetType, i_functional, i_engineSelect );
 
     err = i2cProcessActiveMasters (i_resetType,  // select engines
                                    I2C_OP_RESET, // reset engines
                                    I2C_BUS_SPEED_FROM_MRW,
-                                   i_functional);
+                                   i_functional,
+                                   i_engineSelect);
 
     TRACFCOMP( g_trac_i2c,
                EXIT_MRK"i2cResetActiveMasters(): err rc=0x%X, plid=0x%X",
@@ -3686,7 +3699,8 @@ errlHndl_t i2cSetupActiveMasters ( i2cProcessType i_setupType,
     err = i2cProcessActiveMasters (i_setupType,   // select engines
                                    I2C_OP_SETUP,  // setup engines
                                    I2C_BUS_SPEED_400KHZ,
-                                   i_functional);
+                                   i_functional,
+                                   I2C_ENGINE_SELECT_ALL);
 
     TRACFCOMP( g_trac_i2c,
                EXIT_MRK"i2cSetupActiveMasters(): err rc=0x%X, plid=0x%X",
diff --git a/src/usr/secureboot/trusted/trustedboot.C b/src/usr/secureboot/trusted/trustedboot.C
index 0c1e9e974..43ce1f330 100644
--- a/src/usr/secureboot/trusted/trustedboot.C
+++ b/src/usr/secureboot/trusted/trustedboot.C
@@ -52,6 +52,7 @@
 #include <config.h>
 #include <devicefw/driverif.H>
 #include <i2c/tpmddif.H>
+#include <i2c/i2cif.H>
 #include "trustedboot.H"
 #include "trustedTypes.H"
 #include "trustedbootCmds.H"
@@ -274,6 +275,33 @@ void* host_update_master_tpm( void *io_pArgs )
             if(   hwasState.present
                && hwasState.functional)
             {
+                // If MPIPL do I2C Reset to any processor's I2C engine that is
+                // driving the TPMs
+                TARGETING::Target* sys = nullptr;
+                (void) tS.getTopLevelTarget( sys );
+                assert(sys, "host_update_master_tpm() system target is nullptr");
+
+                if (sys->getAttr<TARGETING::ATTR_IS_MPIPL_HB>())
+                {
+                   err = I2C::i2cResetActiveMasters(
+                             I2C::I2C_PROC_HOST,
+                             true,
+                             I2C::i2cEngineToEngineSelect(tpmData.engine));
+
+                   if (nullptr != err)
+                   {
+                       // Commit log and continue
+                       TRACFCOMP(g_trac_trustedboot,ERR_MRK
+                                 "host_update_master_tpm(): Committing I2C "
+                                 "Reset Fail plid=0x%X but continuing",
+                                 err->plid());
+                       err->collectTrace(TRBOOT_COMP_NAME);
+                       errlCommit(err, TRBOOT_COMP_ID);
+                       err = nullptr;
+                   }
+
+                }
+
                 // API call will set TPM init attempted appropriately
                 tpmInitialize(pPrimaryTpm);
             }