diff options
author | Mike Baiocchi <mbaiocch@us.ibm.com> | 2018-05-21 12:20:44 -0500 |
---|---|---|
committer | Nicholas E. Bofferding <bofferdn@us.ibm.com> | 2018-05-22 09:11:25 -0400 |
commit | d0eaecced89b9a8467cd40e76169f6c88db0e56d (patch) | |
tree | 4d87a72f0e4680fbc7358bdef743a20d83f367bd /src/usr/i2c | |
parent | 5db6a94e4472c3d83fee078d02e914684d2957e0 (diff) | |
download | talos-hostboot-d0eaecced89b9a8467cd40e76169f6c88db0e56d.tar.gz talos-hostboot-d0eaecced89b9a8467cd40e76169f6c88db0e56d.zip |
Skip Diagnostic Mode for I2C Resets over FSI In All Cases
This commit restricts running any Diagnostic Mode I2C Resets
over FSI for all cases. Previously, the mode was only skipped
for FSP-based I2C resets if security was enabled.
Change-Id: Ia2313c01aba4fd4a97b5b1d185786f25a93b4569
CQ:SW430079
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59135
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Disable-CI: Michael Baiocchi <mbaiocch@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Diffstat (limited to 'src/usr/i2c')
-rwxr-xr-x | src/usr/i2c/i2c.C | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/usr/i2c/i2c.C b/src/usr/i2c/i2c.C index 5749ba115..3c30a79ee 100755 --- a/src/usr/i2c/i2c.C +++ b/src/usr/i2c/i2c.C @@ -2635,12 +2635,11 @@ errlHndl_t i2cForceResetAndUnlock( TARGETING::Target * i_target, // does not allow diagnostic mode when Secure Boot is enabled. // Note that because I2C is needed before presence detect, we // cannot check the security state of the processor, so we use - // the master secure mode as a proxy. The effectiveness of this - // approach assumes nobody enables Secure Boot in hardware but - // then loads code with without Secure Boot compiled in, and - // that the processors' secure access bits (SABs) all match. - else if( (SECUREBOOT::enabled()) - && (i_args.switches.useFsiI2C)) // FSI engine 0 + // the master secure mode as a proxy. However, we are unable + // to assume that the processors' secure access bits (SABs) + // all match early in the IPL. Therefore we are disabling + // diagnostic mode for all FSI-based resets. + else if(i_args.switches.useFsiI2C) // FSI engine 0 { skipDiagMode = true; } |