diff options
author | Mike Baiocchi <mbaiocch@us.ibm.com> | 2017-07-28 10:08:20 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-08-04 10:30:53 -0400 |
commit | c7ec40b932a3ebcfcdefe0cc9606278c1852b74c (patch) | |
tree | 0255af1e9899324aa2962255231ede4025cea539 /src/usr/sbeio/sbe_psudd.C | |
parent | 2c17373f30bf1ee14a565488b15bc3004f63393c (diff) | |
download | talos-hostboot-c7ec40b932a3ebcfcdefe0cc9606278c1852b74c.tar.gz talos-hostboot-c7ec40b932a3ebcfcdefe0cc9606278c1852b74c.zip |
Enable SBEIO PSU calls to Open/Close Unsecure Memory Regions
This commit enables the SBEIO PSU calls to the SBE to open and
close Unsecure Memory Regions now that the SBE team supports that
API. Their support, however, does not actually whitelist/blacklist
the memory at this time. This commit also gracefully handles the
errors from the SBE saying this support is not there.
Change-Id: Ia9367b17e403bf759d778de5f4bc7fca3544397b
RTC:174970
CMVC-Prereq: 1030098
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44018
Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Stephen M. Cprek <smcprek@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>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/sbeio/sbe_psudd.C')
-rw-r--r-- | src/usr/sbeio/sbe_psudd.C | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/usr/sbeio/sbe_psudd.C b/src/usr/sbeio/sbe_psudd.C index 7c24cf327..7796cd55b 100644 --- a/src/usr/sbeio/sbe_psudd.C +++ b/src/usr/sbeio/sbe_psudd.C @@ -164,7 +164,11 @@ errlHndl_t SbePsu::performPsuChipOp(TARGETING::Target * i_target, mutex_unlock(&l_psuOpMux); - if( errl && (SBEIO_PSU == errl->moduleId()) ) + if( errl && (SBEIO_PSU == errl->moduleId()) + // For this special case pass back errl without commiting or + // collecting FFDC/shutting down + && (SBE_PSU_SET_UNSECURE_MEMORY_REGION_CMD != i_pPsuRequest->command) + ) { SBE_TRACF( "Forcing shutdown for FSP to collect FFDC" ); @@ -217,7 +221,11 @@ errlHndl_t SbePsu::writeRequest(TARGETING::Target * i_target, { // assign sequence ID and save to check that response matches i_pPsuRequest->seqID = ++l_seqID; - SBE_TRACF("Sending Req = %.16X", i_pPsuRequest->mbxReg0); + SBE_TRACF("Sending Req = %.16X %.16X %.16X %.16X", + i_pPsuRequest->mbxReg0, + i_pPsuRequest->mbxReg1, + i_pPsuRequest->mbxReg2, + i_pPsuRequest->mbxReg3); // Read SBE doorbell to confirm ready to accept command. // Since the device driver single threads the requests, we should |