diff options
| author | Greg Still <stillgs@us.ibm.com> | 2017-07-06 13:41:16 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-08-04 10:44:58 -0400 |
| commit | 1b9910cea325b42cf0dadd071b30d648dc7092da (patch) | |
| tree | 55bc32919c6b49a5ba64fbfff6fa0132f76dece9 /src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C | |
| parent | 390fd421b3e8b679d5582824a0fac9a93e37f476 (diff) | |
| download | talos-hostboot-1b9910cea325b42cf0dadd071b30d648dc7092da.tar.gz talos-hostboot-1b9910cea325b42cf0dadd071b30d648dc7092da.zip | |
PM: no EQ (eg no core) support
- disables Pstates due to lack of #V VPD
- Doesn't start Pstates in AUTO mode
- Fixed in setup evid procedure related to same issue
- rebased
Change-Id: I6f52b5d968e3f66da8c9d33067424bebd333d587
CQ:SW390516
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43083
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: BRIAN D. VICTOR <brian.d.victor1@ibm.com>
Reviewed-by: RAHUL BATRA <rbatra@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43085
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
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/pm/p9_pstate_parameter_block.C')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C index d766231cd..d351fce85 100644 --- a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C +++ b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C @@ -296,13 +296,22 @@ p9_pstate_parameter_block( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_ if (!present_chiplets) { - FAPI_ERR("**** ERROR : There are eq chiplets present"); + FAPI_IMP("**** WARNING : There are no EQ chiplets present which means there is no valid #V VPD"); + FAPI_IMP("**** WARNING : Pstates and all related functions will NOT be enabled."); + l_state.iv_pstates_enabled = false; - FAPI_ASSERT(false, - fapi2::PSTATE_PB_NO_PRESENT_CHIPLETS_ERROR() - .set_CHIP_TARGET(i_target) - .set_PRESENT_CHIPLETS(present_chiplets), - "No eq chiplets are present for a give proc target"); +// FAPI_ASSERT(false, +// fapi2::PSTATE_PB_NO_PRESENT_CHIPLETS_ERROR() +// .set_CHIP_TARGET(i_target) +// .set_PRESENT_CHIPLETS(present_chiplets), +// "No eq chiplets are present for a give proc target"); + + // Set the io_size to 0 so that memory allocation issues won't be + // detected by the caller. + + io_size = 0; + + break; } // --------------------------------------------- @@ -346,7 +355,7 @@ p9_pstate_parameter_block( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_ fapi2::PSTATE_PB_FUNCTION_FAIL(fapi2::FAPI2_ERRL_SEV_RECOVERED) .set_CHIP_TARGET(i_target) .set_FAPI_RC(l_rc), - "Pstate Parameter Block proc_get_mvpd_iddq funciton failed"); + "Pstate Parameter Block proc_get_mvpd_iddq function failed"); fapi2::current_err = fapi2::FAPI2_RC_SUCCESS; } } |
