diff options
| author | Stephen Glancy <sglancy@us.ibm.com> | 2019-08-06 13:04:44 -0400 |
|---|---|---|
| committer | Christian R Geddes <crgeddes@us.ibm.com> | 2019-08-12 08:36:10 -0500 |
| commit | cfad3552a96071e7e0ea912e9cfa351cdc707d6e (patch) | |
| tree | fb46c4295382e106141ed7634ee6466d6d60f882 /src/import/generic/memory/lib/utils/mcbist | |
| parent | 4b5c78b95f25d054ad9ed80d9f1ac95b7dc056fe (diff) | |
| download | talos-hostboot-cfad3552a96071e7e0ea912e9cfa351cdc707d6e.tar.gz talos-hostboot-cfad3552a96071e7e0ea912e9cfa351cdc707d6e.zip | |
Fixes maintenance pattern load bug
Explorer and Nimbus have different locations within
the buffers for the maintenance patterns. This commit
fixes a bug where only the Nimbus locations were used.
Change-Id: Id8f617191b1ae9a4b09b893a1507d299f240c50f
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81774
Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: ANDRE A MARIN <aamarin@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81915
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/generic/memory/lib/utils/mcbist')
| -rw-r--r-- | src/import/generic/memory/lib/utils/mcbist/gen_mss_mcbist.H | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/import/generic/memory/lib/utils/mcbist/gen_mss_mcbist.H b/src/import/generic/memory/lib/utils/mcbist/gen_mss_mcbist.H index f18aacccf..32f1d79c3 100644 --- a/src/import/generic/memory/lib/utils/mcbist/gen_mss_mcbist.H +++ b/src/import/generic/memory/lib/utils/mcbist/gen_mss_mcbist.H @@ -3009,7 +3009,7 @@ fapi2::ReturnCode load_maint_pattern( const fapi2::Target<T>& i_target, const pa // set ecc mode bit on l_aacr .template writeBit<PT::RMW_WRT_BUFFER_SEL>(mss::states::OFF) - .template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(mss::mcbist::rmw_address::DW0) + .template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(PT::MAINT_DATA_INDEX_START) .template writeBit<PT::RMW_WRT_AUTOINC>(mss::states::ON) .template writeBit<PT::RMW_WRT_ECCGEN>(mss::states::ON); @@ -3021,7 +3021,7 @@ fapi2::ReturnCode load_maint_pattern( const fapi2::Target<T>& i_target, const pa for (const auto& p : l_ports) { - l_aacr.template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(mss::mcbist::rmw_address::DW0); + l_aacr.template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(PT::MAINT_DATA_INDEX_START); for (auto l_num_writes = 0; l_num_writes < 2; ++l_num_writes) { @@ -3049,7 +3049,7 @@ fapi2::ReturnCode load_maint_pattern( const fapi2::Target<T>& i_target, const pa FAPI_TRY( fapi2::putScom(p, PT::RMW_WRT_BUF_ECC_REG, 0) ); } - l_aacr.template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(mss::mcbist::rmw_address::DW8); + l_aacr.template insertFromRight<PT::RMW_WRT_ADDRESS, PT::RMW_WRT_ADDRESS_LEN>(PT::MAINT_DATA_INDEX_END); } } |
