Add unmask errors back to cen_framelock

Change-Id: Ia05fadcd6676f2c076bad14002d6afc26953aaf9
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/56027
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Dev-Ready: LUCAS W. MULKEY <lwmulkey@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57063
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

src/import/chips/p9/procedures/hwp/perv/p9_cen_framelock.C

@@ -43,6 +43,7 @@
 #include <p9_cen_framelock.H>
 #include <p9_mc_scom_addresses.H>
 #include <cen_gen_scom_addresses.H>
+#include <p9c_mss_unmask_errors.H>
 
 fapi2::ReturnCode p9_cen_framelock_cloned(const fapi2::Target<fapi2::TARGET_TYPE_DMI>& i_pu_target,
         const fapi2::Target<fapi2::TARGET_TYPE_MEMBUF_CHIP>& i_mem_target,
@@ -1497,7 +1498,6 @@ fapi_try_exit:
 //
 //
 
-
 //------------------------------------------------------------------------------
 fapi2::ReturnCode p9_cen_framelock(const fapi2::Target<fapi2::TARGET_TYPE_DMI>& i_pu_target,
                                    const fapi2::Target<fapi2::TARGET_TYPE_MEMBUF_CHIP>& i_mem_target,
@@ -1509,7 +1509,7 @@ fapi2::ReturnCode p9_cen_framelock(const fapi2::Target<fapi2::TARGET_TYPE_DMI>&
     // it will commit the passed in rc (if non-zero), and return it's own bad rc.
     // Else if mss_unmask_inband_errors runs clean,
     // it will just return the passed in rc.
-    //l_rc = mss_unmask_inband_errors(i_mem_target, l_rc);
+    FAPI_TRY(mss_unmask_inband_errors(i_mem_target));
 
 fapi_try_exit:
     return fapi2::current_err;

src/import/chips/p9/procedures/hwp/perv/p9_cen_framelock.mk

@@ -5,7 +5,7 @@
 #
 # OpenPOWER HostBoot Project
 #
-# Contributors Listed Below - COPYRIGHT 2017
+# Contributors Listed Below - COPYRIGHT 2017,2018
 # [+] International Business Machines Corp.
 #
 #
@@ -22,6 +22,16 @@
 # permissions and limitations under the License.
 #
 # IBM_PROLOG_END_TAG
+CEN_MSS_INCLUDES := $(GENPATH)
+CEN_MSS_INCLUDES += $(ROOTPATH)
+CEN_MSS_INCLUDES += $(ROOTPATH)/chips/centaur/common/include
+CEN_MSS_INCLUDES += $(ROOTPATH)/chips/centaur/procedures/hwp/memory
+CEN_MSS_INCLUDES += $(ROOTPATH)/chips/centaur/procedures/hwp/memory/lib
+CEN_MSS_INCLUDES += $(ROOTPATH)/chips/centaur/procedures/hwp/memory/lib/shared
+CEN_MSS_INCLUDES += $(ROOTPATH)/chips/centaur/procedures/hwp/initfiles/
+
 PROCEDURE=p9_cen_framelock
+lib${PROCEDURE}_DEPLIBS=p9c_mss_unmask_errors
 $(call ADD_MODULE_INCDIR,$(PROCEDURE),$(ROOTPATH)/chips/centaur/common/include)
+$(eval $(call __ADD_MODULE_INCDIR,$(PROCEDURE),$(CEN_MSS_INCLUDES)))
 $(call BUILD_PROCEDURE)

src/usr/isteps/istep12/makefile

@@ -80,6 +80,9 @@ OBJS += call_cen_set_inband_addr.o
 # HWP objects needed to set inband addr in istep 12
 OBJS+=p9c_set_inband_addr.o
 
+# HWP objects need to unmask inband errors during framelock
+OBJS+=p9c_mss_unmask_errors.o
+
 #Required before all the <procedure>.mk are included
 include ${ROOTPATH}/procedure.rules.mk