From e725dc10bc6113e3ea09f862dad184b85c0c3ca0 Mon Sep 17 00:00:00 2001
From: Matt Derksen <mderkse1@us.ibm.com>
Date: Fri, 30 Jun 2017 09:11:34 -0500
Subject: Undo hbrt errl into pnor workaround

Change-Id: I0c00eb1a400410a72cabb89d57d80e70d0ab3ebe
RTC: 131067
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42657
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
---
 src/usr/errl/runtime/rt_errlmanager.C | 34 ++--------------------------------
 1 file changed, 2 insertions(+), 32 deletions(-)

(limited to 'src/usr')

diff --git a/src/usr/errl/runtime/rt_errlmanager.C b/src/usr/errl/runtime/rt_errlmanager.C
index 14b61622e..f690fded5 100644
--- a/src/usr/errl/runtime/rt_errlmanager.C
+++ b/src/usr/errl/runtime/rt_errlmanager.C
@@ -109,14 +109,6 @@ ErrlManager::ErrlManager() :
         TRACFCOMP( g_trac_errl, ERR_MRK"HOSTSVC_PLID not available" );
     }
 
-#if 1
-// TODO: RTC 131067
-// for now do this. but the real code should be the #else below.
-#ifdef CONFIG_BMC_IPMI
-        // setup so that we can write the error log to PNOR
-        setupPnorInfo();
-#endif
-#else
     // check if there's an FSP. if not, then we write to PNOR
     TARGETING::SpFunctions spfn;
     if (!(sys &&
@@ -131,7 +123,7 @@ ErrlManager::ErrlManager() :
     {
         iv_isSpBaseServices = true;
     }
-#endif
+
 
     TRACFCOMP( g_trac_errl, EXIT_MRK "ErrlManager::ErrlManager constructor." );
 }
@@ -141,22 +133,12 @@ ErrlManager::ErrlManager() :
 ErrlManager::~ErrlManager()
 {
     TRACFCOMP( g_trac_errl, INFO_MRK"ErrlManager::ErrlManager destructor" );
-#if 1
-// TODO: RTC 131067
-// for now do this. but the real code should be the #else below.
-#ifdef CONFIG_BMC_IPMI
-        // if we saved to PNOR, we need to flush
-        TRACFCOMP( g_trac_errl, INFO_MRK"flushing pnor" );
-        PNOR::flush(PNOR::HB_ERRLOGS);
-#endif
-#else
     if (!iv_isSpBaseServices)
     {
         // if we saved to PNOR, we need to flush
         TRACFCOMP( g_trac_errl, INFO_MRK"no baseServices, flushing pnor" );
         PNOR::flush(PNOR::HB_ERRLOGS);
     }
-#endif
 }
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -166,18 +148,6 @@ void ErrlManager::sendMboxMsg ( errlHndl_t& io_err )
 {
     do
     {
-#if 1
-// TODO: RTC 131067
-// for now do this. but the real code should be the #else below.
-#ifdef CONFIG_BMC_IPMI
-        bool l_savedToPnor = saveErrLogToPnor(io_err);
-        if (!l_savedToPnor)
-        {
-            TRACFCOMP( g_trac_errl, ENTER_MRK"saveErrLogToPnor didn't save 0x%X",
-                io_err->eid());
-        }
-#endif
-#else
         if (!iv_isSpBaseServices)
         {
             // save to PNOR
@@ -189,7 +159,7 @@ void ErrlManager::sendMboxMsg ( errlHndl_t& io_err )
                     io_err->eid());
             }
         }
-#endif
+
 
 #ifdef CONFIG_BMC_IPMI
         TRACFCOMP(g_trac_errl,INFO_MRK"Send msg to BMC for errlogId [0x%08x]",
-- 
cgit v1.2.1