diff options
| author | crgeddes <crgeddes@us.ibm.com> | 2016-05-25 10:35:02 -0500 |
|---|---|---|
| committer | Matthew A. Ploetz <maploetz@us.ibm.com> | 2016-06-06 18:19:45 -0400 |
| commit | db018773a22a5222cf5845235c18c5efbfcf18e1 (patch) | |
| tree | 485585fc66a7134371b8e25b30a39f493b03ff44 /src/usr/fapi2 | |
| parent | c8dfb4d6781abd10c63fc5531daf4432ba3767be (diff) | |
| download | talos-hostboot-db018773a22a5222cf5845235c18c5efbfcf18e1.tar.gz talos-hostboot-db018773a22a5222cf5845235c18c5efbfcf18e1.zip | |
Update ATTR_POSITION_GETMACRO to lookup FAPI_POS instead of POSTION
POSITION is not a reliable way for systems outside Hostboot to track
targets. We will temporarily put this change in to fix an issue we
were having where we were looking up DIMMS based on this attribute
A better solution is to add a MCA_PORT attribute on the DIMM targets
that is defined by the platform.
Change-Id: Ifbb64a9f326de3d08aef488e7a4b0899024a8cbc
RTC: 154950
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25012
Tested-by: Jenkins Server
Tested-by: FSP CI Jenkins
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Matt Derksen <v2cibmd@us.ibm.com>
Reviewed-by: Andres A. Lugo-Reyes <aalugore@us.ibm.com>
Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
Diffstat (limited to 'src/usr/fapi2')
| -rw-r--r-- | src/usr/fapi2/attribute_service.C | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/usr/fapi2/attribute_service.C b/src/usr/fapi2/attribute_service.C index 831f95c62..8e8daf558 100644 --- a/src/usr/fapi2/attribute_service.C +++ b/src/usr/fapi2/attribute_service.C @@ -375,7 +375,8 @@ ReturnCode platGetTargetPos(const Target<TARGET_TYPE_ALL>& i_pFapiTarget, } else { - uint16_t l_pos = l_pTarget->getAttr<TARGETING::ATTR_POSITION>(); + //RTC: 154950 Add MCA_PORT attribute to DIMM target + uint16_t l_pos = l_pTarget->getAttr<TARGETING::ATTR_FAPI_POS>(); o_pos = l_pos; } |
