diff options
author | Chris Phan <cphan@us.ibm.com> | 2013-05-31 15:27:14 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-06-11 09:23:31 -0500 |
commit | 0b3d775296fb211aaa6b58031f479399da79f808 (patch) | |
tree | 9e45f7012baaa7eb6b2dfa00a4191136a35a87ee /src/usr/diag/prdf/plat | |
parent | cb9f045d4caaf7a5c8cbdd2ea709bc647ba6c3cc (diff) | |
download | talos-hostboot-0b3d775296fb211aaa6b58031f479399da79f808.tar.gz talos-hostboot-0b3d775296fb211aaa6b58031f479399da79f808.zip |
PRD: update to use new hwsv enums and function
Change-Id: I2e0020e4b0934dd68ad4a48b9ea0007f7eab0a2c
RTC: 72578
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/4785
Tested-by: Jenkins Server
Reviewed-by: Zane Shelley <zshelle@us.ibm.com>
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/4940
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/diag/prdf/plat')
-rw-r--r-- | src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C | 36 |
1 files changed, 10 insertions, 26 deletions
diff --git a/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C b/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C index 4e58f4f2f..b2a64ef8e 100644 --- a/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C +++ b/src/usr/diag/prdf/plat/pegasus/prdfDramRepairs.C @@ -52,13 +52,12 @@ bool validSymbol(uint8_t i_symbol) return i_symbol != INVALID_SYMBOL; } -bool commitRestoreCallout( bool (*i_func)(errlHndl_t &, void *), void * i_data, +void commitRestoreCallout( void (*i_func)(errlHndl_t &, void *), void * i_data, TargetHandle_t i_mba ) { PRDF_DENTER("commitRestoreCallout"); errlHndl_t err = NULL; - bool o_term = false; PRDF_HW_CREATE_ERRL( err, @@ -70,15 +69,12 @@ bool commitRestoreCallout( bool (*i_func)(errlHndl_t &, void *), void * i_data, FSP_DEFAULT_REFCODE, PRDF_DETECTED_FAIL_HARDWARE_PROBABLE, 0, 0, 0, 0, // user data - HOM_SYS_NO_TERMINATE, + HWSV_SYS_NO_TERMINATE, false); // no pld check // add the callout - if((*i_func)(err, i_data)) - { - o_term = true; - } + (*i_func)(err, i_data); bool term = false; @@ -89,7 +85,7 @@ bool commitRestoreCallout( bool (*i_func)(errlHndl_t &, void *), void * i_data, err, HWSV::HWSV_DECONFIG_DEFER, ERRL_ACTION_REPORT, - HOM_CONTINUE); + HWSV_CONTINUE); if(term) { @@ -98,16 +94,12 @@ bool commitRestoreCallout( bool (*i_func)(errlHndl_t &, void *), void * i_data, PRDF_COMMIT_ERRL(err, ERRL_ACTION_REPORT); } - - return o_term; } -bool addMemMruCallout(errlHndl_t & io_log, void * i_memMru) +void addMemMruCallout(errlHndl_t & io_log, void * i_memMru) { PRDF_DENTER("addMemMruCallout"); - bool o_term = false; - if ( NULL != i_memMru ) { MemoryMru *memMru = static_cast<MemoryMru *>(i_memMru); @@ -117,11 +109,10 @@ bool addMemMruCallout(errlHndl_t & io_log, void * i_memMru) it != partList.end(); it++ ) { PRDF_HW_ADD_CALLOUT( - o_term, *it, SRCI_PRIORITY_HIGH, - HWSV::HOM_DECONFIG, - HWSV::HOM_DECONFIG_GARD, + HWSV::HWSV_DECONFIG, + HWSV::HWSV_DECONFIG_GARD, io_log, false, // don't write src to vpd GARD_Predictive, @@ -129,29 +120,22 @@ bool addMemMruCallout(errlHndl_t & io_log, void * i_memMru) false); // don't update hcdb } } - - return o_term; } -bool addDimmCallout(errlHndl_t & io_log, void * i_dimm) +void addDimmCallout(errlHndl_t & io_log, void * i_dimm) { PRDF_DENTER("addDimmCallout"); - bool o_term = false; - PRDF_HW_ADD_CALLOUT( - o_term, static_cast<TargetHandle_t>(i_dimm), SRCI_PRIORITY_HIGH, - HWSV::HOM_DECONFIG, - HWSV::HOM_DECONFIG_GARD, + HWSV::HWSV_DECONFIG, + HWSV::HWSV_DECONFIG_GARD, io_log, false, // don't write src to vpd GARD_Predictive, ERRL_SEV_PREDICTIVE, false); // don't update hcdb - - return o_term; } bool processRepairedRanks( TargetHandle_t i_mba, uint8_t i_repairedRankMask ) |