diff options
| author | crgeddes <crgeddes@us.ibm.com> | 2017-03-23 14:33:37 -0500 |
|---|---|---|
| committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2017-03-31 01:11:24 -0400 |
| commit | 35921aa763864d1005143d008a1119436affc08a (patch) | |
| tree | bbb08b0cd487a0b6b2f3d5114502c831433fddf4 /src | |
| parent | 41771befe68b8042ff6e1ec5a852a5e29b447166 (diff) | |
| download | talos-sbe-35921aa763864d1005143d008a1119436affc08a.tar.gz talos-sbe-35921aa763864d1005143d008a1119436affc08a.zip | |
Add workaround so p9_suspend_io can be called systems with no PHB5
Change-Id: I3f50f2f2ef0bb3ae562177a2a12f3c8f474257fc
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38378
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Ricardo Mata <ricmata@us.ibm.com>
Reviewed-by: Jesse P. Arroyo <arroyoj@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38383
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_suspend_io.C | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_suspend_io.C b/src/import/chips/p9/procedures/hwp/nest/p9_suspend_io.C index 11fba633..13afe3b1 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_suspend_io.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_suspend_io.C @@ -75,6 +75,9 @@ fapi2::ReturnCode p9_suspend_io(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP auto l_phb_chiplets_vec = i_target.getChildren<fapi2::TARGET_TYPE_PHB>(); FAPI_DBG("PHB target vec size: %#x\n", l_phb_chiplets_vec.size()); uint8_t l_phb_id = 0; + uint8_t l_fabric_group_id = 0; + + FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_PROC_FABRIC_GROUP_ID, i_target, l_fabric_group_id)) //Loop through all PHBs configured @@ -83,6 +86,25 @@ fapi2::ReturnCode p9_suspend_io(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP //Get the PHB id FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, l_phb_chiplets, l_phb_id)); + //TODO SW384392 ZZ workaroud + //This HWP is ran on the SBE and the SBE does not know what PHBs are functional or not + //This causes this HWP to do scoms to targets that are not functional. This workaround + // needs to be removed when the SBE can reliably tell us what PHBs are functional + if(l_fabric_group_id == 0) + { + if(l_phb_id == 0x5) + { + continue; + } + } + else if(l_fabric_group_id == 1) + { + if(l_phb_id == 0x5 || l_phb_id == 0x4) + { + continue; + } + } + //**************************************************************************************************************** //STEP #1: nop //**************************************************************************************************************** |
