diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2016-10-31 22:40:51 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-11-05 22:38:27 -0400 |
commit | 5351daa3654973d841686f861b2296a8cd315439 (patch) | |
tree | 40eec9c6ea665eeb8ab9e52a466925834f609e85 /src/import/chips/p9/procedures/hwp/customize | |
parent | d5640d8a61933d2fea07dc18fb69593378a98efe (diff) | |
download | talos-hostboot-5351daa3654973d841686f861b2296a8cd315439.tar.gz talos-hostboot-5351daa3654973d841686f861b2296a8cd315439.zip |
xip_customize: Updated to make non-configured chiplets not good.
Change-Id: I43788d81370fcdfe5c5986000d0f0abd8f4ba85e
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32053
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Michael S. Floyd <mfloyd@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32067
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/customize')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C b/src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C index fbef441fe..3ec0bb2b2 100644 --- a/src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C +++ b/src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C @@ -79,8 +79,17 @@ fapi2::ReturnCode writePG( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_proc_target, void* i_image) { + const uint8_t IMG_PG_ENTRIES = 64; + const uint16_t DEFAULT_PG_VAL = 0xffff; + FAPI_DBG ("writePG Entering..."); - const uint8_t IMG_PG_ENTRIES = 64; + + // Make all chiplets "not good". + for (auto l_pg_idx = 0; l_pg_idx < IMG_PG_ENTRIES; l_pg_idx++) + { + FAPI_TRY( p9_xip_set_element(i_image, "ATTR_PG", l_pg_idx, DEFAULT_PG_VAL), + "Error (1) from p9_xip_set_element (idx %d)", l_pg_idx ); + } for (auto l_perv_tgt : i_proc_target.getChildren<fapi2::TARGET_TYPE_PERV>()) { @@ -104,7 +113,7 @@ fapi2::ReturnCode writePG( // Update the image FAPI_TRY( p9_xip_set_element(i_image, "ATTR_PG", l_pg_idx, l_pg_data), - "Error from p9_xip_set_element (idx %d)", l_pg_idx ); + "Error (2) from p9_xip_set_element (idx %d)", l_pg_idx ); FAPI_DBG("Write value of pg_data[%d] = %08X", l_pg_idx, l_pg_data); } |