Skip to content

Commit 3810076

Browse files
anusrangdcrowell77
authored andcommitted
literal definitions
instead of constants in the optimized procs Change-Id: I761ec79dc173465cd502844b6e2da7eff7dd800c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37954 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: Soma Bhanutej <soma.bhanu@in.ibm.com> Reviewed-by: SRINIVAS V. POLISETTY <srinivan@in.ibm.com> Reviewed-by: PARVATHI RACHAKONDA <prachako@in.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38025 Tested-by: Jenkins OP Build CI <op-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>
1 parent 996a0d2 commit 3810076

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/import/chips/p9/procedures/hwp/perv/p9_setup_sbe_config.C

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141

4242
#include <p9_perv_scom_addresses.H>
4343
#include <p9_perv_scom_addresses_fld.H>
44+
#include <p9_const_common.H>
4445

4546
enum P9_SETUP_SBE_CONFIG_Private_Constants
4647
{
@@ -161,28 +162,28 @@ fapi2::ReturnCode p9_setup_sbe_config(const
161162
FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_OPTICS_CONFIG_MODE, targ, l_optics_cfg_mode),
162163
"Error from FAPI_ATTR_GET(ATTR_OPTICS_CONFIG_MODE)");
163164

164-
if (l_chipletID == 9)
165+
if (l_chipletID == OB0_CHIPLET_ID)
165166
{
166167
l_read_scratch_reg.writeBit<ATTR_OPTICS_CONFIG_MODE_OBUS0_BIT>(((l_optics_cfg_mode ==
167168
fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_SMP)
168169
|| (l_optics_cfg_mode == fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_CAPI)) ? (1) : (0));
169170
}
170171

171-
if (l_chipletID == 10)
172+
if (l_chipletID == OB1_CHIPLET_ID)
172173
{
173174
l_read_scratch_reg.writeBit<ATTR_OPTICS_CONFIG_MODE_OBUS1_BIT>(((l_optics_cfg_mode ==
174175
fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_SMP)
175176
|| (l_optics_cfg_mode == fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_CAPI)) ? (1) : (0));
176177
}
177178

178-
if (l_chipletID == 11)
179+
if (l_chipletID == OB2_CHIPLET_ID)
179180
{
180181
l_read_scratch_reg.writeBit<ATTR_OPTICS_CONFIG_MODE_OBUS2_BIT>(((l_optics_cfg_mode ==
181182
fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_SMP)
182183
|| (l_optics_cfg_mode == fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_CAPI)) ? (1) : (0));
183184
}
184185

185-
if (l_chipletID == 12)
186+
if (l_chipletID == OB3_CHIPLET_ID)
186187
{
187188
l_read_scratch_reg.writeBit<ATTR_OPTICS_CONFIG_MODE_OBUS3_BIT>(((l_optics_cfg_mode ==
188189
fapi2::ENUM_ATTR_OPTICS_CONFIG_MODE_SMP)

0 commit comments

Comments
 (0)