diff options
author | Mark Pizzutillo <Mark.Pizzutillo@ibm.com> | 2019-11-18 16:04:42 -0500 |
---|---|---|
committer | Daniel M Crowell <dcrowell@us.ibm.com> | 2019-12-05 13:14:40 -0600 |
commit | ecac8199d6605bc9a2453ae47890881f06ab887e (patch) | |
tree | 1c18b33ede53ace35a11ad5e2907f600e3f8301b /src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C | |
parent | ef6600cc1ac4e47393ea838f64bf37874919798f (diff) | |
download | talos-hostboot-ecac8199d6605bc9a2453ae47890881f06ab887e.tar.gz talos-hostboot-ecac8199d6605bc9a2453ae47890881f06ab887e.zip |
Add asserts for invalid states of enterprise & half dimm mode
Change-Id: I995c333fc7f891ea7fa055097d4377a861a1e391
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87345
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Dev-Ready: STEPHEN GLANCY <sglancy@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://rchgit01.rchland.ibm.com/gerrit1/87600
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/ocmb/explorer/procedures/hwp/memory/exp_getecid.C')
-rw-r--r-- | src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C index 7bad2df39..ac8d7e7cb 100644 --- a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C +++ b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_getecid.C @@ -40,6 +40,7 @@ #include <explorer_scom_addresses_fld.H> #include <mss_explorer_attribute_setters.H> #include <generic/memory/mss_git_data_helper.H> +#include <lib/plug_rules/exp_plug_rules.H> extern "C" { @@ -53,24 +54,20 @@ extern "C" { mss::display_git_commit_info("exp_getecid"); - // Using FUSE enterprise_dis bit, determine whether enterprise is disabled, otherwise - // we will enable it. Override to disable it is done in omi_setup. Half_dimm_mode we - // will also disable by default, as it is not a feature of P systems { - uint8_t l_enterprise_mode = fapi2::ENUM_ATTR_MSS_OCMB_ENTERPRISE_MODE_NON_ENTERPRISE; // 0 - uint8_t l_half_dimm_mode = fapi2::ENUM_ATTR_MSS_OCMB_HALF_DIMM_MODE_FULL_DIMM; // 0 + bool l_enterprise_fuse = false; + bool l_enterprise_final = false; - FAPI_TRY(mss::exp::ecid::get_enterprise_and_half_dimm_from_fuse( - i_target, l_enterprise_mode, l_half_dimm_mode), - "exp_getecid: getting enterprise and half_dimm from fuse failed on %s", + FAPI_TRY(mss::exp::ecid::get_enterprise_from_fuse(i_target, l_enterprise_fuse), + "exp_getecid: getting enterprise from fuse failed on %s", mss::c_str(i_target)); - // Set attributes - FAPI_TRY(mss::attr::set_ocmb_enterprise_mode(i_target, l_enterprise_mode), - "exp_getecid: Could not set ATTR_MSS_OCMB_ENTERPRISE_MODE"); + // Calculate the global enterprise mode state while verifying plug rules with policy and override attributes + FAPI_TRY(mss::exp::plug_rule::enterprise_mode(i_target, l_enterprise_fuse, l_enterprise_final)); - FAPI_TRY(mss::attr::set_ocmb_half_dimm_mode(i_target, l_half_dimm_mode), - "exp_getecid: Could not set ATTR_MSS_OCMB_HALF_DIMM_MODE"); + // Set global enterprise mode attribute + FAPI_TRY(mss::attr::set_ocmb_enterprise_mode(i_target, l_enterprise_final), + "exp_getecid: Could not set ATTR_MSS_OCMB_ENTERPRISE_MODE"); } // |