PRD: reduced error path on memory threshold functions

Change-Id: I12ed4cca3b5b10d7d5505af0a0a23971e29634d0
RTC: 169729
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36675
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Caleb N. Palmer <cnpalmer@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/37012
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>

1 files changed, 8 insertions, 13 deletions

diff --git a/src/usr/diag/prdf/plat/mem/prdfMemIplCeStats.H b/src/usr/diag/prdf/plat/mem/prdfMemIplCeStats.H
index 04cfaaba7..846df1669 100755
--- a/src/usr/diag/prdf/plat/mem/prdfMemIplCeStats.H
+++ b/src/usr/diag/prdf/plat/mem/prdfMemIplCeStats.H
@@ -90,11 +90,9 @@ class MemIplCeStats
     /**
      * @brief  Analyzes the maintenance statistics that were gathered, making
      *         the appropriate callouts.
-     * @param  o_callOutsDone TRUE if PRD made a hardware callout, FALSE
-     *         otherwise.
-     * @return Non-SUCCESS if an internal function fails, SUCCESS otherwise.
+     * @return TRUE if PRD made a hardware callout, FALSE otherwise.
      */
-    int32_t analyzeStats ( bool & o_callOutsDone );
+    bool analyzeStats();
 
     /** @brief  Calls out all symbols with hard CEs.
      *  @param  i_stopRank The rank the maintenance command stopped on.
@@ -176,23 +174,20 @@ class MemIplCeStats
   private: // functions
 
     /** @brief  Calls out all symbols on a dram that has exceeded threshold.
-     *  @param  o_callOutsMade TRUE if a callout was made, FALSE othewise.
-     *  @return Non-SUCCESS if an internal function fails, SUCCESS othewise.
+     *  @return TRUE if a callout was made, FALSE othewise.
      */
-    int32_t calloutCePerDram( bool & o_callOutsMade );
+    bool calloutCePerDram();
 
     /** @brief  Calls out all symbols on a rank that has exceeded threshold.
-     *  @param  o_callOutsMade TRUE if a callout was made, FALSE othewise.
-     *  @return Non-SUCCESS if an internal function fails, SUCCESS othewise.
+     *  @return TRUE if a callout was made, FALSE othewise.
      */
-    int32_t calloutCePerRank( bool & o_callOutsMade );
+    bool calloutCePerRank();
 
     /** @brief  Calls out all symbols on a dimm select that has exceeded
      *          threshold.
-     *  @param  o_callOutsMade TRUE if a callout was made, FALSE othewise.
-     *  @return Non-SUCCESS if an internal function fails, SUCCESS othewise.
+     *  @return TRUE if a callout was made, FALSE othewise.
      */
-    int32_t calloutCePerDs( bool & o_callOutsMade );
+    bool calloutCePerDs();
 
     /**
      * @brief Add MemoryMru callout to error log and commit it.