Skip to content

Commit

Permalink
Initf procedure updates for Axone OMI rings
Browse files Browse the repository at this point in the history
using SBE_AXONE_CONFIG compile flag for axone specific changes

Change-Id: Ibbbb69d6f8d87b4cbbd011fa5af4f496e5106335
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64915
Tested-by: Jenkins Server <pfd-jenkins+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: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64917
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
  • Loading branch information
anusrang authored and sgupta2m committed Sep 8, 2018
1 parent 97b4146 commit 957a647
Show file tree
Hide file tree
Showing 3 changed files with 83 additions and 2 deletions.
33 changes: 32 additions & 1 deletion src/import/chips/p9/procedures/hwp/perv/p9_sbe_gptr_time_initf.C
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,12 @@ fapi2::ReturnCode p9_sbe_gptr_time_initf(const

FAPI_INF("p9_sbe_gptr_time_initf: Entering ...");

#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)
uint8_t l_read_attr_axone_only;
FAPI_DBG("Reading ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY\n");
FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY, i_target_chip, l_read_attr_axone_only));
#endif

for( auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_PERV>
( fapi2::TARGET_STATE_FUNCTIONAL))
{
Expand Down Expand Up @@ -92,6 +98,7 @@ fapi2::ReturnCode p9_sbe_gptr_time_initf(const
FAPI_DBG("Scan mc_pll_gptr ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_pll_gptr),
"Error from putRing (mc_pll_gptr)");

FAPI_DBG("Scan mc_time ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_time),
"Error from putRing (mc_time)");
Expand Down Expand Up @@ -135,7 +142,7 @@ fapi2::ReturnCode p9_sbe_gptr_time_initf(const
}
}

// mc - Cumulus
// mc - Cumulus, Axone
for (auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_MC>
(fapi2::TARGET_STATE_FUNCTIONAL))
{
Expand All @@ -149,6 +156,30 @@ fapi2::ReturnCode p9_sbe_gptr_time_initf(const
FAPI_DBG("Scan mc_time ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_time),
"Error from putRing (mc_time)");

#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)

// Axone only
if (l_read_attr_axone_only)
{
FAPI_DBG("Scan mc_omi0_gptr, mc_omi1_gptr, mc_omi2_gptr ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi0_gptr),
"Error from putRing (mc_omi0_gptr)");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi1_gptr),
"Error from putRing (mc_omi1_gptr)");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi2_gptr),
"Error from putRing (mc_omi2_gptr)");

FAPI_DBG("Scan mc_omippe_gptr ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omippe_gptr),
"Error from putRing (mc_omippe_gptr)");

FAPI_DBG("Scan mc_omippe_time ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omippe_time),
"Error from putRing (mc_omippe_time)");
}

#endif
}

// mc - Cumulus
Expand Down
33 changes: 33 additions & 0 deletions src/import/chips/p9/procedures/hwp/perv/p9_sbe_nest_initf.C
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ fapi2::ReturnCode p9_sbe_nest_initf(const
bool mc23_iom01 = false;
bool mc23_iom23 = false;
#endif
#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)
uint8_t l_read_attr_axone_only;
FAPI_DBG("Reading ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY\n");
FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY, i_target_chip, l_read_attr_axone_only));
#endif

for (auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_PERV>(fapi2::TARGET_STATE_FUNCTIONAL))
{
Expand Down Expand Up @@ -192,11 +197,39 @@ fapi2::ReturnCode p9_sbe_nest_initf(const
}
}

// mc - Nimbus
for (auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_MCBIST>(fapi2::TARGET_STATE_FUNCTIONAL))
{
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_fure));
}

// mc - Cumulus, Axone
for (auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_MC>(fapi2::TARGET_STATE_FUNCTIONAL))
{
FAPI_DBG("Scan mc_fure ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_fure));

#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)

// Axone only
if (l_read_attr_axone_only)
{
FAPI_DBG("Scan mc_omi0_fure ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi0_fure));

FAPI_DBG("Scan mc_omi1_fure ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi1_fure));

FAPI_DBG("Scan mc_omi2_fure ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omi2_fure));

FAPI_DBG("Scan mc_omippe_fure ring");
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omippe_fure));
}

#endif
}

#if 0

// mcbist0
Expand Down
19 changes: 18 additions & 1 deletion src/import/chips/p9/procedures/hwp/perv/p9_sbe_repr_initf.C
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,12 @@ fapi2::ReturnCode p9_sbe_repr_initf(const
FAPI_DBG("Reading ATTR_CHIP_EC_FEATURE_P9C_LOGIC_ONLY\n");
FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_P9C_LOGIC_ONLY, i_target_chip, l_read_attr_cumulus_only));

#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)
uint8_t l_read_attr_axone_only;
FAPI_DBG("Reading ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY\n");
FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_P9A_LOGIC_ONLY, i_target_chip, l_read_attr_axone_only));
#endif

for( auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_PERV>
( fapi2::TARGET_STATE_FUNCTIONAL))
{
Expand All @@ -79,10 +85,21 @@ fapi2::ReturnCode p9_sbe_repr_initf(const
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_repr));
}

// mc - Cumulus
// mc - Cumulus, Axone
for (auto& l_chplt_trgt : i_target_chip.getChildren<fapi2::TARGET_TYPE_MC>(fapi2::TARGET_STATE_FUNCTIONAL))
{
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_repr));

#if defined(SBE_AXONE_CONFIG) || !defined(__PPE__)

// Axone only
if (l_read_attr_axone_only)
{
FAPI_TRY(fapi2::putRing(l_chplt_trgt, mc_omippe_repr),
"Error from putRing (mc_omippe_repr)");
}

#endif
}

// mc - Cumulus
Expand Down

0 comments on commit 957a647

Please sign in to comment.