path: root/src/import/chips/p9a
diff options
authorAndre A. Marin <>2019-04-26 16:41:53 -0500
committerChristian R. Geddes <>2019-04-30 11:55:05 -0500
commit3b13a6483b18a93d67b36c5a28d07415212a6995 (patch)
treeef114a2ef722e7a3b355b5b0738d49b560839614 /src/import/chips/p9a
parent114d10d29c181671b0ec51c1bef37652a2df9b83 (diff)
Add call to setup derived attributes in p9a_mss_eff_config
Previously there was some work done to write the functions that do the calculations for these derived attributes but the code was not being called, and one of the functions had an incorrect parameter. This commit addresses both of those issues and also improves on a trace in eff_grouping that made debugging this issue tricky. Change-Id: I8394f1dcc29374afc765668fce79e55f9f823e7d RTC: 207850 Reviewed-on: Tested-by: FSP CI Jenkins <> Tested-by: Jenkins Server <> Reviewed-by: ANDRE A. MARIN <> Tested-by: Hostboot CI <> Reviewed-by: Louis Stermole <> Reviewed-by: STEPHEN GLANCY <> Reviewed-by: Jennifer A. Stofer <> Reviewed-on: Reviewed-by: Jenkins Server <> Reviewed-by: Christian R. Geddes <> Tested-by: Christian R. Geddes <>
Diffstat (limited to 'src/import/chips/p9a')
1 files changed, 4 insertions, 0 deletions
diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C
index 3bbf0ea9f..8a5529734 100644
--- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C
+++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config.C
@@ -112,6 +112,10 @@ fapi2::ReturnCode p9a_mss_eff_config( const fapi2::Target<fapi2::TARGET_TYPE_MEM
FAPI_TRY( mss::attr_eff_engine<mss::exp::attr_eff_engine_fields>::set(l_spd_decoder) );
+ // Set up derived ATTRS
+ FAPI_TRY( mss::attr_derived_engine<mss::attr_engine_derived_fields>::set(dimm) );
}// dimm
OpenPOWER on IntegriCloud