diff options
| author | Stephen Glancy <sglancy@us.ibm.com> | 2019-03-14 09:20:24 -0400 |
|---|---|---|
| committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2019-04-15 09:57:05 -0500 |
| commit | 745094dc3a9fbdb241ee4973eec4049f6ea7550b (patch) | |
| tree | 05ec46d75ac657ffab57b628c81daa0a816cfed3 /src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C | |
| parent | 4cfa805d328b0168e7874b025bf7133fd355121e (diff) | |
| download | talos-hostboot-745094dc3a9fbdb241ee4973eec4049f6ea7550b.tar.gz talos-hostboot-745094dc3a9fbdb241ee4973eec4049f6ea7550b.zip | |
Moves blame a FIR API to generic
Change-Id: I4486acdd2fa9ee6a5d48808f52b43e2203967830
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/73325
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/73352
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C')
| -rw-r--r-- | src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C index 1b59893bb..685ec22e7 100644 --- a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C +++ b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_draminit.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2018 */ +/* Contributors Listed Below - COPYRIGHT 2018,2019 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -103,12 +103,12 @@ extern "C" fapi_try_exit: FAPI_INF("Draminit training - %s %s", - (fapi2::current_err == fapi2::FAPI2_RC_SUCCESS ? "success" : "errors reported"), + (fapi2::current_err == fapi2::FAPI2_RC_SUCCESS ? "success" : "errors reported - trying to blame FIR's"), mss::c_str(i_target)); - // TK update this to check FIR's or PLL failures - that will be updated in another commit - // For now, return current_err - return fapi2::current_err; - + // Due to the RAS/PRD requirements, we need to check for FIR's + // If any FIR's have lit up, this draminit fail could have been caused by the FIR, rather than bad hardware + // So, let PRD retrigger this step to see if we can resolve the issue + return mss::check::fir_or_pll_fail<mss::mc_type::EXPLORER>(i_target, fapi2::current_err); } } |
