From cfad3552a96071e7e0ea912e9cfa351cdc707d6e Mon Sep 17 00:00:00 2001 From: Stephen Glancy Date: Tue, 6 Aug 2019 13:04:44 -0400 Subject: 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 Tested-by: FSP CI Jenkins Tested-by: Jenkins Server Dev-Ready: STEPHEN GLANCY Tested-by: Hostboot CI Tested-by: HWSV CI Tested-by: PPE CI Reviewed-by: ANDRE A MARIN Reviewed-by: Jennifer A Stofer Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81915 Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Christian R Geddes --- .../memory/lib/utils/mcbist/gen_mss_mcbist.H | 6 +++--- .../memory/lib/utils/shared/mss_generic_consts.H | 21 --------------------- 2 files changed, 3 insertions(+), 24 deletions(-) (limited to 'src/import/generic') 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& i_target, const pa // set ecc mode bit on l_aacr .template writeBit(mss::states::OFF) - .template insertFromRight(mss::mcbist::rmw_address::DW0) + .template insertFromRight(PT::MAINT_DATA_INDEX_START) .template writeBit(mss::states::ON) .template writeBit(mss::states::ON); @@ -3021,7 +3021,7 @@ fapi2::ReturnCode load_maint_pattern( const fapi2::Target& i_target, const pa for (const auto& p : l_ports) { - l_aacr.template insertFromRight(mss::mcbist::rmw_address::DW0); + l_aacr.template insertFromRight(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& i_target, const pa FAPI_TRY( fapi2::putScom(p, PT::RMW_WRT_BUF_ECC_REG, 0) ); } - l_aacr.template insertFromRight(mss::mcbist::rmw_address::DW8); + l_aacr.template insertFromRight(PT::MAINT_DATA_INDEX_END); } } diff --git a/src/import/generic/memory/lib/utils/shared/mss_generic_consts.H b/src/import/generic/memory/lib/utils/shared/mss_generic_consts.H index 1b916c902..2a80655c6 100644 --- a/src/import/generic/memory/lib/utils/shared/mss_generic_consts.H +++ b/src/import/generic/memory/lib/utils/shared/mss_generic_consts.H @@ -397,27 +397,6 @@ enum broadcast_timebase TB_COUNT_128 = 0b1111111, }; -enum rmw_address -{ - // 32B block addresses into the maint portion of the rmw buffer - DW0 = 0b111110000, - DW1 = 0b111110001, - DW2 = 0b111110010, - DW3 = 0b111110011, - DW4 = 0b111110100, - DW5 = 0b111110101, - DW6 = 0b111110110, - DW7 = 0b111110111, - DW8 = 0b111111000, - DW9 = 0b111111001, - DWA = 0b111111010, - DWB = 0b111111011, - DWC = 0b111111100, - DWD = 0b111111101, - DWE = 0b111111110, - DWF = 0b111111111, -}; - enum data_rotate_mode { // MCBIST data rotate modes refer to register MCBDRCR bits 0:3 -- cgit v1.2.1