diff options
author | Zane Shelley <zshelle@us.ibm.com> | 2018-05-15 21:17:35 -0500 |
---|---|---|
committer | Zane C. Shelley <zshelle@us.ibm.com> | 2018-05-23 15:11:16 -0400 |
commit | 5064efd9ee552ace99fb051ce98bb896202933ab (patch) | |
tree | a162ded1c13ee10a2e9edcdadd243ed961970f89 /src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C | |
parent | 0535232e93da36701b20fd52022f02af6fdfc7f4 (diff) | |
download | talos-hostboot-5064efd9ee552ace99fb051ce98bb896202933ab.tar.gz talos-hostboot-5064efd9ee552ace99fb051ce98bb896202933ab.zip |
PRD: Simplify templates in memory CE functions
Change-Id: I8182ce256b247009b198220f937aef58343a61bf
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/58879
Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59227
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>
Diffstat (limited to 'src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C')
-rw-r--r-- | src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C index 410ef7ac8..d4e29723c 100644 --- a/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C +++ b/src/usr/diag/prdf/plat/mem/prdfMemTdCtlr.C @@ -218,7 +218,7 @@ uint32_t MemTdCtlr<T>::handleCmdComplete( STEP_CODE_DATA_STRUCT & io_sc ) // some way to change the template to use the MCA. It is also a local function // because this is only for MemTdCtlr internal use and it didn't make much sense // to create a public function. -template<TARGETING::TYPE T, typename D> +template<TARGETING::TYPE T> uint32_t __checkEcc( ExtensibleChip * i_chip, TdQueue & io_queue, const MemAddr & i_addr, bool & o_errorsFound, STEP_CODE_DATA_STRUCT & io_sc ); @@ -272,11 +272,8 @@ uint32_t __analyzeCmdComplete<TYPE_MCBIST>( ExtensibleChip * i_chip, for ( auto & mcaChip : portList ) { bool errorsFound; - uint32_t l_rc = __checkEcc<TYPE_MCA, McaDataBundle *>( mcaChip, - io_queue, - i_addr, - errorsFound, - io_sc ); + uint32_t l_rc = __checkEcc<TYPE_MCA>( mcaChip, io_queue, i_addr, + errorsFound, io_sc ); if ( SUCCESS != l_rc ) { PRDF_ERR( PRDF_FUNC "__checkEcc<TYPE_MCA>(0x%08x) failed", @@ -307,8 +304,7 @@ uint32_t __analyzeCmdComplete<TYPE_MBA>( ExtensibleChip * i_chip, o_stoppedRank = __getStopRank<TYPE_MBA>( i_chip, i_addr ); // Check the MBA for ECC errors. - return __checkEcc<TYPE_MBA, MbaDataBundle *>( i_chip, io_queue, i_addr, - o_errorsFound, io_sc ); + return __checkEcc<TYPE_MBA>(i_chip, io_queue, i_addr, o_errorsFound, io_sc); } //------------------------------------------------------------------------------ |