diff options
author | Stephen Glancy <sglancy@us.ibm.com> | 2018-04-24 09:40:59 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2018-04-27 11:28:54 -0400 |
commit | f35a5eed8791ebe555bb85e42e73ae430e013cb5 (patch) | |
tree | dca42edec6619d281452bc05424c0dcd49f3eeac | |
parent | 03b0fc84b1bf36b4f8567e862c0335290428dd7e (diff) | |
download | talos-hostboot-f35a5eed8791ebe555bb85e42e73ae430e013cb5.tar.gz talos-hostboot-f35a5eed8791ebe555bb85e42e73ae430e013cb5.zip |
Moves FAPI_INF to FAPI_DBG
Shmoo file sizes are massive for Cumulus. Moving print statements from
FAPI_INF to FAPI_DBG will help reduce the file size. Also, adds in
c_str calls to track the target.
Change-Id: Ie80550eab959c1fa429aac4dfd270bc7f1dd7279
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57743
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57751
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: Christian R. Geddes <crgeddes@us.ibm.com>
-rw-r--r-- | src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C | 4 | ||||
-rwxr-xr-x | src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mrs6_DDR4.C | 26 |
2 files changed, 17 insertions, 13 deletions
diff --git a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C index 425aa18d4..d660d280a 100644 --- a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C +++ b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mcbist_common.C @@ -590,7 +590,7 @@ extern "C" if (*o_mcb_status == 1) { - FAPI_DBG("poll_mcb: MCBIST failed"); + FAPI_DBG("%s poll_mcb: MCBIST failed", mss::c_str(i_target_mba)); return fapi2::FAPI2_RC_FALSE; } @@ -634,7 +634,7 @@ extern "C" uint8_t l_marray0[DIMM_DQ_SPD_DATA_SIZE] = { 0 }; uint8_t l_num, io_num, l_inter, l_num2, l_index2; fapi2::variable_buffer l_data_buffer1_64(64), l_data_buffer3_64(64); - FAPI_INF("Function MCB_ERROR_MAP_PRINT"); + FAPI_DBG("%s Function MCB_ERROR_MAP_PRINT", mss::c_str(i_target_mba)); FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, i_target_mba, l_mba_position)); FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CEN_EFF_STACK_TYPE, i_target_mba, dram_stack)); diff --git a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mrs6_DDR4.C b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mrs6_DDR4.C index 3e78c6282..e773c5a24 100755 --- a/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mrs6_DDR4.C +++ b/src/import/chips/centaur/procedures/hwp/memory/p9c_mss_mrs6_DDR4.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2016,2017 */ +/* Contributors Listed Below - COPYRIGHT 2016,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -44,7 +44,9 @@ #include <cen_gen_scom_addresses.H> #include <p9c_mss_mrs6_DDR4.H> #include <dimmConsts.H> +#include <generic/memory/lib/utils/c_str.H> +using mss::c_str; extern "C" { /// @@ -59,7 +61,7 @@ extern "C" for (uint8_t l_port_number = 0; l_port_number < 2; ++l_port_number) { // Step four: Load MRS Setting - FAPI_INF("Loading MRS6 for port %d", l_port_number); + FAPI_DBG("%s Loading MRS6 for port %d", c_str(i_target), l_port_number); FAPI_TRY(mss_mr6_loader(i_target, l_port_number, l_ccs_inst_cnt), " mrs_load Failed"); } @@ -115,7 +117,7 @@ extern "C" fapi2::variable_buffer l_rank_cal_4(4); fapi2::variable_buffer l_cal_enable_1(1); fapi2::variable_buffer l_ccs_end_1(1); - FAPI_INF("\n Running NO -OP command"); + FAPI_DBG("%s Running NO -OP command", c_str(i_target_mba)); fapi2::buffer<uint8_t> l_data_8; fapi2::buffer<uint16_t> l_data_16; uint8_t l_dram_stack[MAX_PORTS_PER_MBA][MAX_DIMM_PER_PORT] = {0}; @@ -135,7 +137,7 @@ extern "C" if (l_dram_stack[0][0] == fapi2::ENUM_ATTR_CEN_EFF_STACK_TYPE_STACK_3DS) { - FAPI_INF( "============= Got in the 3DS stack loop CKE !!!!=====================\n"); + FAPI_DBG( "%s ============= Got in the 3DS stack loop CKE !!!!=====================", c_str(i_target_mba)); FAPI_TRY(l_csn_8.clearBit(2, 2)); FAPI_TRY(l_csn_8.clearBit(6, 2)); FAPI_TRY(l_cke_4.clearBit(1)); @@ -227,7 +229,8 @@ extern "C" uint8_t l_dram_stack[MAX_PORTS_PER_MBA][MAX_DIMM_PER_PORT] = {0}; uint8_t l_tccd_l = 0; //tccd_l - NEW - FAPI_INF( "+++++++++++++++++++++ LOADING MRS SETTINGS FOR PORT %d +++++++++++++++++++++", i_port_number); + FAPI_DBG( "%s +++++++++++++++++++++ LOADING MRS SETTINGS FOR PORT %d +++++++++++++++++++++", c_str(i_target), + i_port_number); FAPI_TRY(l_activate_1.setBit(0)); FAPI_TRY(l_rasn_1.clearBit(0)); @@ -269,14 +272,14 @@ extern "C" FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CEN_EFF_STACK_TYPE, i_target, l_dram_stack)); - FAPI_INF( "Stack Type: %d\n", l_dram_stack[0][0]); + FAPI_DBG( "%s Stack Type: %d", c_str(i_target), l_dram_stack[0][0]); //MRS6 FAPI_TRY(FAPI_ATTR_GET( fapi2::ATTR_CEN_EFF_VREF_DQ_TRAIN_VALUE, i_target, l_vrefdq_train_value)); FAPI_TRY(FAPI_ATTR_GET( fapi2::ATTR_CEN_EFF_VREF_DQ_TRAIN_RANGE, i_target, l_vrefdq_train_range)); FAPI_TRY(FAPI_ATTR_GET( fapi2::ATTR_CEN_EFF_VREF_DQ_TRAIN_ENABLE, i_target, l_vrefdq_train_enable)); - FAPI_INF("enable attribute %d", l_vrefdq_train_enable[0][0][0]); + FAPI_DBG("%s enable attribute %d", c_str(i_target), l_vrefdq_train_enable[0][0][0]); FAPI_TRY(FAPI_ATTR_GET( fapi2::ATTR_CEN_TCCD_L, i_target, l_tccd_l)); @@ -339,14 +342,15 @@ extern "C" if (l_num_ranks == 0) { - FAPI_INF( "PORT%d DIMM%d not configured. Num_ranks: %d ", i_port_number, l_dimm_number, l_num_ranks); + FAPI_INF( "%s PORT%d DIMM%d not configured. Num_ranks: %d ", c_str(i_target), i_port_number, l_dimm_number, + l_num_ranks); } else { // Rank 0-3 for ( l_rank_number = 0; l_rank_number < l_num_ranks; l_rank_number ++) { - FAPI_INF( "MRS SETTINGS FOR PORT%d DIMM%d RANK%d", i_port_number, l_dimm_number, l_rank_number); + FAPI_DBG( "%s MRS SETTINGS FOR PORT%d DIMM%d RANK%d", c_str(i_target), i_port_number, l_dimm_number, l_rank_number); FAPI_TRY(l_csn_8.setBit(0, 8)); FAPI_TRY(l_address_16.clearBit(0, 16)); @@ -389,7 +393,7 @@ extern "C" FAPI_TRY(l_mrs6.extract(l_MRS6, 0, 16)); - FAPI_INF( "MRS 6: 0x%04X", l_MRS6); + FAPI_DBG( "%s MRS 6: 0x%04X", c_str(i_target), l_MRS6); // Only corresponding CS to rank @@ -398,7 +402,7 @@ extern "C" if (l_dram_stack[0][0] == fapi2::ENUM_ATTR_CEN_EFF_STACK_TYPE_STACK_3DS) { - FAPI_INF( "============= Got in the 3DS stack loop CKE !!!!=====================\n"); + FAPI_DBG( "%s ============= Got in the 3DS stack loop CKE !!!!=====================", c_str(i_target)); FAPI_TRY(l_csn_8.clearBit(2, 2)); FAPI_TRY(l_csn_8.clearBit(6, 2)); FAPI_TRY(l_cke_4.clearBit(1)); |