diff options
author | Jacob Harvey <jlharvey@us.ibm.com> | 2016-12-12 14:33:41 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-03-03 16:06:59 -0500 |
commit | 6dea39bab63bf54a977f7e35adb9ac71bb638c98 (patch) | |
tree | 5825afa2e772584cedee7e4e9147ac7bade51827 | |
parent | eb369b6353dbb3cb414706f680c724ef5c4b165e (diff) | |
download | talos-hostboot-6dea39bab63bf54a977f7e35adb9ac71bb638c98.tar.gz talos-hostboot-6dea39bab63bf54a977f7e35adb9ac71bb638c98.zip |
Move MRS attributes to eff_config to calc LRDIMMs
Change-Id: I63c44ec9d40abd7f2729d9316164ddabf987f74f
Original-Change-Id: Ie2b6d187d67f8bc7ed975e7627fd31ff343e8969
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33774
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: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37413
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/spd/spd_factory.C | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/spd/spd_factory.C b/src/import/chips/p9/procedures/hwp/memory/lib/spd/spd_factory.C index 73cfb8114..9741df37b 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/spd/spd_factory.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/spd/spd_factory.C @@ -836,7 +836,13 @@ fapi2::ReturnCode populate_decoder_caches( const fapi2::Target<TARGET_TYPE_DIMM> // Populate custom spd caches maps one dimm at a time o_factory_caches.emplace( std::make_pair( pos(i_target), i_pDecoder ) ); - return fapi2::FAPI2_RC_SUCCESS; + // Populate some of the DIMM attributes early. This allows the following code to make + // decisions based on DIMM information. Expressly done after the factory has decided on the SPD version + FAPI_TRY( master_ranks_per_dimm_setter(i_target, i_pDecoder), + "%s. Failed master_ranks_per_dimm_setter()", mss::c_str(i_target) ); + +fapi_try_exit: + return fapi2::current_err; } }// spd |