diff options
| author | Dean Sanner <dsanner@us.ibm.com> | 2017-01-07 21:01:04 -0600 |
|---|---|---|
| committer | William G. Hoffa <wghoffa@us.ibm.com> | 2017-01-14 09:13:30 -0500 |
| commit | 200e13aff211133665601c5378ec6e09741a6262 (patch) | |
| tree | a7ffc2010efdfee5d740aa927935b84fdcb0f2c2 /src/usr/i2c | |
| parent | 6f5121da6e5b73b3013f46d5acc78344b22d2481 (diff) | |
| download | talos-hostboot-200e13aff211133665601c5378ec6e09741a6262.tar.gz talos-hostboot-200e13aff211133665601c5378ec6e09741a6262.zip | |
I2C diag mode can't be used on master 2, port 0
-This is the typical port where the TPM is connected,
and the extra security features for locality 4 interfere
with the diag mode reset
Change-Id: I5dddc8263d6a75e246ce3e2043037ab9f69555ff
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34544
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/usr/i2c')
| -rwxr-xr-x | src/usr/i2c/i2c.C | 21 |
1 files changed, 19 insertions, 2 deletions
diff --git a/src/usr/i2c/i2c.C b/src/usr/i2c/i2c.C index aca89c7a1..643db7b07 100755 --- a/src/usr/i2c/i2c.C +++ b/src/usr/i2c/i2c.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2011,2016 */ +/* Contributors Listed Below - COPYRIGHT 2011,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -2486,7 +2486,6 @@ errlHndl_t i2cToggleClockLine(TARGETING::Target * i_target, errlHndl_t i2cForceResetAndUnlock( TARGETING::Target * i_target, misc_args_t & i_args) { - errlHndl_t err = NULL; mode_reg_t mode; uint64_t l_speed = I2C_BUS_SPEED_FROM_MRW; @@ -2533,6 +2532,24 @@ errlHndl_t i2cForceResetAndUnlock( TARGETING::Target * i_target, // Need to send slave stop to all ports with a device on the engine for( uint32_t port = 0; port < P8_MASTER_PORTS; port++ ) { + //Check if diag mode should be skipped + TARGETING::TYPE l_type = i_target->getAttr<TARGETING::ATTR_TYPE>(); + + if (l_type == TARGETING::TYPE_PROC) + { + uint8_t l_disable_diag_mode = + i_target->getAttr< + TARGETING::ATTR_DISABLE_I2C_ENGINE2_PORT0_DIAG_MODE>(); + + // P9 engine 2 port 0 has a limitation where the diag mode + // cannot be used. -- skip it if the attribute state it + // should not be used + if (l_disable_diag_mode + && (0 == port) && (2 == i_args.engine)) + { + continue; + } + } // Only send stop to a port if there are devices on it l_speed = speed_array[i_args.engine][port]; |
