diff options
author | Brian Silver <bsilver@us.ibm.com> | 2016-08-26 10:55:28 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-08-30 17:18:15 -0400 |
commit | bde3b600402dafbda62cb147134a0ac59fcf5315 (patch) | |
tree | e6b479d35705517fc0c91826debdba37488182fa /src/import/chips/p9/procedures/hwp/memory/lib/freq | |
parent | 738f0a0e2e3e6594029695386672118a2d28b838 (diff) | |
download | talos-hostboot-bde3b600402dafbda62cb147134a0ac59fcf5315.tar.gz talos-hostboot-bde3b600402dafbda62cb147134a0ac59fcf5315.zip |
Change freq system's sync to account for single MCBIST configs
Change-Id: I02df860352d51dcb8ba0cc25bd56b8232fe3628a
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28846
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28847
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
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/lib/freq')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/freq/sync.C | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/freq/sync.C b/src/import/chips/p9/procedures/hwp/memory/lib/freq/sync.C index 4672cebf8..c3acf1070 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/freq/sync.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/freq/sync.C @@ -93,6 +93,9 @@ fapi2::ReturnCode dimm_speed_map(const std::vector< fapi2::Target<TARGET_TYPE_MC // DIMM speed is equal until we deduce otherwise o_is_speed_equal = speed_equality::EQUAL_DIMM_SPEEDS; + // Make sure to stick the first one we found in the freq map. + o_freq_map.emplace( std::make_pair(*l_found_comp, l_comparator) ); + // Loop through all MCSBISTs and store dimm speeds // Starting from known 1st known good freq (non-zero) value // I found above to avoid double looping target vector @@ -120,6 +123,13 @@ fapi2::ReturnCode dimm_speed_map(const std::vector< fapi2::Target<TARGET_TYPE_MC } } + // Idiot check - most certainly a programming error + if (o_freq_map.size() == 0 ) + { + FAPI_ERR("freq system freq map is empty? found mcbist: %s", mss::c_str(*l_found_comp)); + fapi2::Assert(false); + } + fapi_try_exit: return fapi2::current_err; } |