diff options
author | Prem Shanker Jha <premjha2@in.ibm.com> | 2014-01-15 10:16:18 -0600 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2014-01-24 22:28:57 -0600 |
commit | 87f89f151885654e56a6d04fd4861fcd68d6d88b (patch) | |
tree | 0233ef5ad3c56a09b6a00289e18d08197212763c /src/usr/diag/prdf/common/framework/service | |
parent | 46361e5fcd49e5c14213a6e5d83f2c4975913dc0 (diff) | |
download | talos-hostboot-87f89f151885654e56a6d04fd4861fcd68d6d88b.tar.gz talos-hostboot-87f89f151885654e56a6d04fd4861fcd68d6d88b.zip |
PRD:Addressed FIXME in RAS Services
Change-Id: I602037e8496724be89ba3e43e277b6316b86f6e2
RTC:66038
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/8176
Tested-by: Jenkins Server
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Christopher T. Phan <cphan@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Reviewed-by: Zane Shelley <zshelle@us.ibm.com>
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/8337
Diffstat (limited to 'src/usr/diag/prdf/common/framework/service')
3 files changed, 0 insertions, 69 deletions
diff --git a/src/usr/diag/prdf/common/framework/service/prdfRasServices_common.C b/src/usr/diag/prdf/common/framework/service/prdfRasServices_common.C index 798ce62d6..74dc80713 100644 --- a/src/usr/diag/prdf/common/framework/service/prdfRasServices_common.C +++ b/src/usr/diag/prdf/common/framework/service/prdfRasServices_common.C @@ -773,66 +773,6 @@ errlHndl_t ErrDataService::GenerateSrcPfa( ATTENTION_TYPE i_attnType, ErrlVer1, ErrlSectPFA5_1 ); //************************************************************** - // Check for Manufacturing AVP mode - // If needed: Add the AVP mode data to Error Log User Data - //************************************************************** - - /* FIXME - The MDIA component is being removed from FSP code. This means this MDIA registry variable -will also be removed. Need to confirm if this code is required anymore. - if ( PlatServices::avpMode() ) - { - //Get the AVP Test Case Number from the AVP Test Case Data Registry. The Test Case Number is the first four bytes. - uint32_t avpTCNumber = 0; - size_t sz_uint32 = sizeof(uint32_t); - errlHndl_t errorLog = UtilReg::read ("mdia/AvpTestCaseData", &avpTCNumber, sz_uint32); - // printf("AVP Test Case Number from registry: %.8x \n", avpTCNumber); - if (errorLog) - { - errorLog->commit(PRDF_COMP_ID,ERRL_ACTION_REPORT); - PRDF_ERR( "PRDTRACE: RasServices Failure in AVP Test Case Registry read" ); - delete errorLog; - errorLog = NULL; - } - else - { - //Add Test Case Number to Error Log User Data - UtilMem l_membuf; - l_membuf << avpTCNumber; - o_errl->addUsrDtls(l_membuf.base(),l_membuf.size(),PRDF_COMP_ID,ErrlVer1,ErrlAVPData_1); - } - } -*/ - -#ifndef __HOSTBOOT_MODULE - // FIXME: do we need support for AVP in Hostboot? probably not - if ( PlatServices::hdatAvpMode() ) - { - //Get the AVP Test Case Data from the AVP Test Case Data Registry. - uint8_t avpTCData[64]; - size_t sz_avpTCData = sizeof(avpTCData); - errlHndl_t errorLog = UtilReg::read ("hdat/AvpTestCaseData", avpTCData, sz_avpTCData); - - if (errorLog) - { - PRDF_ERR( PRDF_FUNC"Failure in hdat AVP Test Case Registry read" ); - PRDF_COMMIT_ERRL(errorLog, ERRL_ACTION_REPORT); - } - else - { - //Add Test Case Data to Error Log User Data - const size_t sz_usrDtlsTCData = 29; - uint8_t usrDtlsTCData[sz_usrDtlsTCData]; - memcpy(usrDtlsTCData, avpTCData, 4); - memcpy(&usrDtlsTCData[4], &avpTCData[40], 4); - memcpy(&usrDtlsTCData[8], &avpTCData[37], 1); - memcpy(&usrDtlsTCData[9], &avpTCData[44], 20); - PRDF_ADD_FFDC( o_errl, (const char*)usrDtlsTCData, sz_usrDtlsTCData, - ErrlVer1, ErrlAVPData_2 ); - } - } -#endif // if not __HOSTBOOT_MODULE - - //************************************************************** // Add SDC Capture data to Error Log User Data //************************************************************** // Pulled some code out to incorporate into AddCapData diff --git a/src/usr/diag/prdf/common/framework/service/prdfTargetServices.C b/src/usr/diag/prdf/common/framework/service/prdfTargetServices.C index 78b1690b5..24f475fc4 100755 --- a/src/usr/diag/prdf/common/framework/service/prdfTargetServices.C +++ b/src/usr/diag/prdf/common/framework/service/prdfTargetServices.C @@ -1379,9 +1379,6 @@ bool isMnfgFlagSet( uint32_t i_flag ) bool mfgMode() { return isMnfgFlagSet( MNFG_FLAG_BIT_MNFG_THRESHOLDS ); } -bool hdatAvpMode() -{ return isMnfgFlagSet( MNFG_FLAG_BIT_MNFG_HDAT_AVP_ENABLE ); } - bool mnfgTerminate() { return isMnfgFlagSet( MNFG_FLAG_BIT_MNFG_SRC_TERM ); } diff --git a/src/usr/diag/prdf/common/framework/service/prdfTargetServices.H b/src/usr/diag/prdf/common/framework/service/prdfTargetServices.H index a924facf7..2bcbda38d 100755 --- a/src/usr/diag/prdf/common/framework/service/prdfTargetServices.H +++ b/src/usr/diag/prdf/common/framework/service/prdfTargetServices.H @@ -409,12 +409,6 @@ TARGETING::TargetHandle_t getClockId(TARGETING::TargetHandle_t bool mfgMode(); /** - * @brief Returns the state of the MNFG_HDAT_AVP_ENABLE policy flag. - * @return TRUE if MNFG_HDAT_AVP_ENABLE is set, FALSE otherwise. - */ -bool hdatAvpMode(); - -/** * @brief Returns status of MNFG_SRC_TERM manufacturing policy flag. * @param None. * @return TRUE if MNFG_SRC_TERM policy flag is set, FALSE |