diff options
| author | Joe McGill <jmcgill@us.ibm.com> | 2017-01-28 12:01:39 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-02-01 12:18:39 -0500 |
| commit | 2dd60aa28799eece57e303617e231e926abcbd79 (patch) | |
| tree | e4a931aadd909c1a2e0ead3399a2c940dc6db588 /src | |
| parent | 885c60521f8626b855988e7687bc6e437c2355f7 (diff) | |
| download | talos-hostboot-2dd60aa28799eece57e303617e231e926abcbd79.tar.gz talos-hostboot-2dd60aa28799eece57e303617e231e926abcbd79.zip | |
p9_xip_customize -- remove customization of slave status and FBC IDs
skip customization of slave status and fabric IDs stored in image
explicitly set as master chip, with groupID = 0, chip ID = 0
this will ensure that a part placed into the master socket for
OpenPower/BMC based systems will always boot correctly with
no reliance on the SBE mailbox being written before SBE start
a part placed into a slave socket will always have these fields
customized via the SBE mailbox
Change-Id: I7027671518b5144f47cd34f27a4d83e992797c08
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35532
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35533
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/customize/p9_xip_customize.C | 23 |
1 files changed, 20 insertions, 3 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 c0ffea65d..7bbb5720a 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 @@ -43,6 +43,23 @@ using namespace fapi2; #ID); \ } +#define MBOX_ATTR_SET(ID,TARGET,IMAGE) \ + { \ + fapi2::ID##_Type ID##_attrVal = 1; \ + FAPI_TRY(p9_xip_set_scalar(IMAGE,#ID,ID##_attrVal),\ + "MBOX_ATTR_SET: Error writing attr %s to seeprom image",\ + #ID); \ + } + +#define MBOX_ATTR_CLEAR(ID,TARGET,IMAGE) \ + { \ + fapi2::ID##_Type ID##_attrVal = 0; \ + FAPI_TRY(p9_xip_set_scalar(IMAGE,#ID,ID##_attrVal),\ + "MBOX_ATTR_CLEAR: Error writing attr %s to seeprom image",\ + #ID); \ + } + + fapi2::ReturnCode writeMboxRegs ( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_proc_target, void* i_image) @@ -66,9 +83,9 @@ fapi2::ReturnCode writeMboxRegs ( MBOX_ATTR_WRITE (ATTR_RISK_LEVEL, FAPI_SYSTEM, i_image); MBOX_ATTR_WRITE (ATTR_DISABLE_HBBL_VECTORS, FAPI_SYSTEM, i_image); MBOX_ATTR_WRITE (ATTR_MC_SYNC_MODE, i_proc_target, i_image); - MBOX_ATTR_WRITE (ATTR_PROC_SBE_MASTER_CHIP, i_proc_target, i_image); - MBOX_ATTR_WRITE (ATTR_PROC_FABRIC_GROUP_ID, i_proc_target, i_image); - MBOX_ATTR_WRITE (ATTR_PROC_FABRIC_CHIP_ID, i_proc_target, i_image); + MBOX_ATTR_SET (ATTR_PROC_SBE_MASTER_CHIP, i_proc_target, i_image); + MBOX_ATTR_CLEAR (ATTR_PROC_FABRIC_GROUP_ID, i_proc_target, i_image); + MBOX_ATTR_CLEAR (ATTR_PROC_FABRIC_CHIP_ID, i_proc_target, i_image); fapi_try_exit: FAPI_DBG("writeMboxRegs Exiting..."); |
