diff options
author | Louis Stermole <stermole@us.ibm.com> | 2017-12-15 13:05:58 -0600 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2017-12-22 15:50:35 -0500 |
commit | 956df8db0fd1720bf6caafaa06451e3eafd65dfb (patch) | |
tree | ff996d2659271fe1a2abd615b6edd91587b96de8 /src/import | |
parent | 6b43bcadea886d324aa94971130f3567fbeb60b1 (diff) | |
download | talos-hostboot-956df8db0fd1720bf6caafaa06451e3eafd65dfb.tar.gz talos-hostboot-956df8db0fd1720bf6caafaa06451e3eafd65dfb.zip |
Change ZQ cal fail action to deconfigure MCBIST instead of MCA
ZQ calibration is started on a per-MCBIST basis. If any port fails
ZQ calibration due to the magic MCA0, then it is the result of a bad
resistor. The resistor is on a per-MCBIST basis. As such, the MCBIST
should be deconfigured, while the port is additionally called out.
Change-Id: I20475f7924ecc82a4435648cb8f3a63f230d6460
CQ:SW411690
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/51028
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/51039
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: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/phy/ddr_phy.C | 2 | ||||
-rw-r--r-- | src/import/chips/p9/procedures/xml/error_info/p9_memory_mss_ddr_phy_reset.xml | 5 |
2 files changed, 4 insertions, 3 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/phy/ddr_phy.C b/src/import/chips/p9/procedures/hwp/memory/lib/phy/ddr_phy.C index 66c1e9af5..564ed5c54 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/phy/ddr_phy.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/phy/ddr_phy.C @@ -190,7 +190,7 @@ fapi2::ReturnCode enable_zctl( const fapi2::Target<TARGET_TYPE_MCBIST>& i_target }); FAPI_ASSERT(l_poll, - fapi2::MSS_ZCNTL_FAILED_TO_COMPLETE().set_MCA_IN_ERROR(p), + fapi2::MSS_ZCNTL_FAILED_TO_COMPLETE().set_MCA_IN_ERROR(p).set_MCBIST_TARGET(i_target), "zctl enable failed: %s", mss::c_str(p)); } diff --git a/src/import/chips/p9/procedures/xml/error_info/p9_memory_mss_ddr_phy_reset.xml b/src/import/chips/p9/procedures/xml/error_info/p9_memory_mss_ddr_phy_reset.xml index a64dfde9f..8a1294af6 100644 --- a/src/import/chips/p9/procedures/xml/error_info/p9_memory_mss_ddr_phy_reset.xml +++ b/src/import/chips/p9/procedures/xml/error_info/p9_memory_mss_ddr_phy_reset.xml @@ -304,6 +304,7 @@ <description> ZCAL failed to complete </description> + <ffdc>MCBIST_TARGET</ffdc> <collectRegisterFfdc> <id>REG_FFDC_ZCNTL_REGS</id> <target>MCA_IN_ERROR</target> @@ -314,10 +315,10 @@ <priority>HIGH</priority> </callout> <deconfigure> - <target>MCA_IN_ERROR</target> + <target>MCBIST_TARGET</target> </deconfigure> <gard> - <target>MCA_IN_ERROR</target> + <target>MCBIST_TARGET</target> </gard> </hwpError> |