summaryrefslogtreecommitdiffstats
path: root/src/import/generic/memory/lib/utils
diff options
context:
space:
mode:
authorAndre Marin <aamarin@us.ibm.com>2019-07-11 13:01:31 -0400
committerDaniel M Crowell <dcrowell@us.ibm.com>2019-08-09 07:56:28 -0500
commit1061da0271bf85fe29fedff6f242181b3dc6d5ed (patch)
treef586bdbba79f3ee934073e1bbd49981d0d29b0b0 /src/import/generic/memory/lib/utils
parent9fb424b8af396bb626c28105b4383fc22aeccd94 (diff)
downloadblackbird-hostboot-1061da0271bf85fe29fedff6f242181b3dc6d5ed.tar.gz
blackbird-hostboot-1061da0271bf85fe29fedff6f242181b3dc6d5ed.zip
Add missing attributes needed to be set for generic mss_kind
Setting DRAM_MFG_ID, RCD_MFG_ID, and MODULE_HEIGHT to eff_config and editing SPD timing values to use common API for calculations that come with values from the EEPROM Change-Id: If33f2f2a49a62f114575ef36d6325fc537d1dc27 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79921 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79936 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: Daniel M Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/generic/memory/lib/utils')
-rw-r--r--src/import/generic/memory/lib/utils/mss_field.H7
-rw-r--r--src/import/generic/memory/lib/utils/shared/mss_generic_consts.H28
2 files changed, 20 insertions, 15 deletions
diff --git a/src/import/generic/memory/lib/utils/mss_field.H b/src/import/generic/memory/lib/utils/mss_field.H
index d6bdeaaed..c69bde04f 100644
--- a/src/import/generic/memory/lib/utils/mss_field.H
+++ b/src/import/generic/memory/lib/utils/mss_field.H
@@ -327,9 +327,10 @@ inline fapi2::ReturnCode get_field( const fapi2::Target<T>& i_target,
typename TT::template COMPARISON_OP<IT>() ),
F.get_byte(i_data),
l_temp,
- i_ffdc_codes),
- "Failed fail_for_invalid_value() for " TARGIDFORMAT, TARGTID );
-
+ i_ffdc_codes,
+ TT::FIELD_STR),
+ "%s failed check::invalid_value() for %s",
+ TT::FIELD_STR, spd::c_str(i_target) );
// Output should only change if data check passes
o_value = static_cast<OT>(l_temp);
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 72970ae63..1d95e82d5 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
@@ -217,6 +217,12 @@ enum generic_ffdc_codes
SET_PRIM_BUS_WIDTH = 0x1069,
SET_PRIM_DIE_COUNT = 0x1070,
SET_DRAM_DENSITY = 0x1071,
+
+ // Power thermal functions
+ POWER_LIMIT = 0x1072,
+ SLOPE = 0x1073,
+ INTERCEPT = 0x1074,
+
SET_SI_RD_VREF_DQ = 0x1075,
SET_CAC_DELAY_A = 0x1076,
SET_CAC_DELAY_B = 0x1077,
@@ -240,24 +246,22 @@ enum generic_ffdc_codes
SET_DRAM_TRRD_L = 0x1091,
SET_DRAM_TRFC = 0x1092,
SET_DRAM_TRFC_DLR = 0x1093,
-
-
- // Power thermal functions
- POWER_LIMIT = 0x1072,
- SLOPE = 0x1073,
- INTERCEPT = 0x1074,
+ SET_DRAM_MFG_ID = 0x1094,
// Used in fw_mark_store.H for MSS_INVALID_RANK_PASSED
- FWMS_READ = 30,
- FWMS_WRITE = 31,
+ FWMS_READ = 0x1095,
+ FWMS_WRITE = 0x1096,
// Used in hw_mark_store.H for MSS_INVALID_RANK_PASSED
- HWMS_READ = 40,
- HWMS_WRITE = 41,
+ HWMS_READ = 0x1097,
+ HWMS_WRITE = 0x1098,
// MSS_INVALID_INDEX_PASSED
- SYMBOL_COUNT_READ = 50,
- SYMBOL_COUNT_WRITE = 51,
+ SYMBOL_COUNT_READ = 0x1099,
+ SYMBOL_COUNT_WRITE = 0x109A,
+
+ SET_RCD_MFG_ID = 0x109B,
+ SET_DRAM_MODULE_HEIGHT = 0x109C,
};
///
OpenPOWER on IntegriCloud