diff options
| author | Chris Steffen <cwsteffen@us.ibm.com> | 2019-02-14 10:22:18 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2019-02-22 12:56:52 -0600 |
| commit | 688a9733e614a8cb04224f5c7e33da4170f87f51 (patch) | |
| tree | c745ccafc2ed502bdc3cf0defe77996f8aa16c27 /src/import/chips/p9/procedures/hwp | |
| parent | ab41d473f8b631ba0f9fbdb1d48ffb42421610a3 (diff) | |
| download | talos-hostboot-688a9733e614a8cb04224f5c7e33da4170f87f51.tar.gz talos-hostboot-688a9733e614a8cb04224f5c7e33da4170f87f51.zip | |
P9 Obus MNFG Errors Attribute Fix
- Changed MNFG thresholds coniditional statement to bitwise AND
Change-Id: I71cdad3384f2ba942a5bd13754ce200ed1d76da3
CQ: SW457203
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71887
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: Megan P. Nguyen <pmegan@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71901
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/io/p9_io_obus_post_trainadv.C | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_post_trainadv.C b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_post_trainadv.C index 86492ce39..1b1e0c276 100644 --- a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_post_trainadv.C +++ b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_post_trainadv.C @@ -152,12 +152,12 @@ fapi2::ReturnCode p9_io_obus_post_trainadv( uint64_t l_mfg_flags = 0x0; FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_MNFG_FLAGS, l_sys_tgt, l_mfg_flags)); - if(l_mfg_flags == fapi2::ENUM_ATTR_MNFG_FLAGS_MNFG_THRESHOLDS) + if(l_mfg_flags & fapi2::ENUM_ATTR_MNFG_FLAGS_MNFG_THRESHOLDS) { uint8_t l_mfg_error_threshold = 0x0; FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_IO_O_MNFG_ERROR_THRESHOLD, l_sys_tgt, l_mfg_error_threshold)); - if(l_mfg_error_threshold != fapi2:: ENUM_ATTR_IO_O_MNFG_ERROR_THRESHOLD_CORNER_MODE) + if(l_mfg_error_threshold == fapi2::ENUM_ATTR_IO_O_MNFG_ERROR_THRESHOLD_CORNER_MODE) { // Setup ECC & CRC Masks FAPI_TRY(p9_io_obus_mnfg_setup_ecc_crc_masks(i_tgt), @@ -175,7 +175,7 @@ fapi2::ReturnCode p9_io_obus_post_trainadv( "error from p9_io_obus_mnfg_adj_tally_logic()"); } - else if(l_mfg_error_threshold != fapi2:: ENUM_ATTR_IO_O_MNFG_ERROR_THRESHOLD_RELIABILITY_MODE) + else if(l_mfg_error_threshold == fapi2::ENUM_ATTR_IO_O_MNFG_ERROR_THRESHOLD_RELIABILITY_MODE) { // Setup ECC & CRC Masks FAPI_TRY(p9_io_obus_mnfg_setup_ecc_crc_masks(i_tgt), |
