From f1186fdef28f6e23a9bda17bbef1d0e217c0430d Mon Sep 17 00:00:00 2001 From: Andrew Geissler Date: Fri, 18 May 2018 14:01:16 -0500 Subject: Update getChildTargetsForCDG to use CEN PORT/DIMM The port and dimm sockets come off of the centaur on cumulus. Update this function to look at the correct attributes. These attribute do not exist on Cumulus so the default will be to just return all dimms under the input target. This code change affected the test case so an update was required there as well (i.e. the fix for this defect). Change-Id: I102c40acc124100e90dd832aeed94716c0f18811 CQ:SW428864 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59073 Reviewed-by: Martin Gloff Tested-by: Jenkins Server Reviewed-by: Christian R. Geddes Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell --- src/usr/fapi2/plat_utils.C | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/usr/fapi2/plat_utils.C') diff --git a/src/usr/fapi2/plat_utils.C b/src/usr/fapi2/plat_utils.C index e6445e219..e340b2d60 100644 --- a/src/usr/fapi2/plat_utils.C +++ b/src/usr/fapi2/plat_utils.C @@ -997,6 +997,8 @@ void processEICDGs(const ErrorInfo & i_errInfo, /// For DIMMs: DIMM Socket Number /// For Chips: Chip Position /// For Chiplets: Chiplet Position +/// @param[o] o_childTargets List of child targets matching input +/// criteria. /// void getChildTargetsForCDG( const fapi2::Target& i_parentTarget, @@ -1081,11 +1083,11 @@ void getChildTargetsForCDG( // Match i_childPort and i_childNum if ( ((i_childPort == ErrorInfoChildrenCDG::ALL_CHILD_PORTS) || (i_childPort == - (*l_itr)->getAttr())) + (*l_itr)->getAttr())) && ((i_childNum == ErrorInfoChildrenCDG::ALL_CHILD_NUMBERS) || (i_childNum == - (*l_itr)->getAttr())) ) + (*l_itr)->getAttr())) ) { o_childTargets.push_back(*l_itr); } -- cgit v1.2.1