diff options
author | Dan Crowell <dcrowell@us.ibm.com> | 2016-11-17 15:57:19 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-12-20 11:58:40 -0500 |
commit | 17003aae2a9797e990fb2a6869d77967db9c1633 (patch) | |
tree | 5e5efdbf8b62e2536bd5748e3d33dcbb90347e61 /src/usr/targeting/common | |
parent | cec9b06e89d027d084bac7876604efbcb7d297eb (diff) | |
download | talos-hostboot-17003aae2a9797e990fb2a6869d77967db9c1633.tar.gz talos-hostboot-17003aae2a9797e990fb2a6869d77967db9c1633.zip |
Improve detection and description for MEMVPD_POS issues
Updated attribute description based on changes to requirements
Added explicit check for invalid data to improve debug
Change-Id: Ie22287166855be1e0d99a52cc4a3ad89d354acfa
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32793
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33833
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
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>
Diffstat (limited to 'src/usr/targeting/common')
-rw-r--r-- | src/usr/targeting/common/xmltohb/attribute_types.xml | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/src/usr/targeting/common/xmltohb/attribute_types.xml b/src/usr/targeting/common/xmltohb/attribute_types.xml index a8fd917af..3ef5626c1 100644 --- a/src/usr/targeting/common/xmltohb/attribute_types.xml +++ b/src/usr/targeting/common/xmltohb/attribute_types.xml @@ -10640,15 +10640,16 @@ firmware notes: Used as override attribute for pstate procedure </hwpfToHbAttrMap> </attribute> -<!--TOOD RTC: 151938 Make sure that these are set up by parseMRW script--> <attribute> <id>MEMVPD_POS</id> <description> - The position of the MCA target's VPD selector data, relative to the EEPROM - that contains its data. For systems with a single EEPROM for all chips, - this should be equivalent to ATTR_FAPI_POS (the logical MCA position). - For systems with an EEPROM per chip, this value should be equivalent to - ATTR_CHIP_UNIT_POS. + The position of the MCS target's VPD selector data, relative to the EEPROM + that contains its data. The VPD defition supports up to 16 values per + EEPROM. + For systems with an EEPROM per chip, this value should be equivalent to + ATTR_CHIP_UNIT_POS. + For systems with a single EEPROM for all chips, the value should follow + the physical position in such a way to fit within the 16 available slots. </description> <simpleType> <uint8_t> |