Check for targeting before adding version info in errl commit

addVersionInfo function calls spBaseServicesEnabled, which
accesses targeting data.  Moved the function call after the
check for targeting loaded.

Change-Id: Iaee25d71ce046821b7805f9fdb5a80bfa452f3fe
CQ:SW449232
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67730
Tested-by: Jenkins Server <pfd-jenkins+hostboot@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: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/src/usr/errl/errlentry.C b/src/usr/errl/errlentry.C
index 072a31ac1..ad95bc4dd 100644
--- a/src/usr/errl/errlentry.C
+++ b/src/usr/errl/errlentry.C
@@ -1000,9 +1000,6 @@ void ErrlEntry::commit( compId_t  i_committerComponent )
     // Add the Hostboot Build ID to the error log
     addHbBuildId();
 
-    // Add the version info to the error log for OpenPOWER systems
-    addVersionInfo();
-
     // check to see if we should skip info and recoverable errors?
     checkHiddenLogsEnable();
 
@@ -1010,6 +1007,9 @@ void ErrlEntry::commit( compId_t  i_committerComponent )
     // serial numbers
     if(Util::isTargetingLoaded() && TARGETING::targetService().isInitialized())
     {
+        // Add the version info to the error log for OpenPOWER systems
+        addVersionInfo();
+
         // If this error was a hardware callout, add the serial and part numbers
         // to the log. FSP provides this data so if there is no FSP, get them here.
         if(!INITSERVICE::spBaseServicesEnabled())