diff options
| author | Brian Silver <bsilver@us.ibm.com> | 2016-09-07 07:25:08 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-09-25 13:21:50 -0400 |
| commit | 5af4383ee2147812a056e4c355b43b34ce138d39 (patch) | |
| tree | e029f018a4043d6ec95fab08607bd11d33833668 /src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C | |
| parent | 754955318cdc833def3c8f74037c89e5115324ac (diff) | |
| download | talos-hostboot-5af4383ee2147812a056e4c355b43b34ce138d39.tar.gz talos-hostboot-5af4383ee2147812a056e4c355b43b34ce138d39.zip | |
Change the polling in memdiags so it's for sim only
RTC: 153951
Change-Id: I33a1f2735651668c259d21662971db95869a67a4
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29311
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29313
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C b/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C index 505197ecd..f101d6d3f 100644 --- a/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C +++ b/src/import/chips/p9/procedures/hwp/memory/p9_mss_memdiag.C @@ -43,6 +43,7 @@ #include <lib/mcbist/mcbist.H> using fapi2::TARGET_TYPE_MCBIST; +using fapi2::TARGET_TYPE_SYSTEM; extern "C" { @@ -63,10 +64,16 @@ extern "C" return fapi2::FAPI2_RC_SUCCESS; } + uint8_t is_sim = false; + FAPI_TRY( FAPI_ATTR_GET(fapi2::ATTR_IS_SIMULATION, fapi2::Target<TARGET_TYPE_SYSTEM>(), is_sim) ); + + // We start the sf_init (write 0's) and it'll tickle the MCBIST complete FIR. PRD will see that + // and start a background scrub. FAPI_TRY( memdiags::sf_init(i_target, mss::mcbist::PATTERN_0) ); - // TODO RTC:153951 - // Remove the polling when the attention bits are hooked up + // If we're in the sim, we want to poll for the FIR bit. I don't think this ever really happens + // unless we're expressly testing this API. + if (is_sim) { // Poll for the fir bit. We expect this to be set ... fapi2::buffer<uint64_t> l_status; |
