diff options
Diffstat (limited to 'src/import/chips/p9')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/io/p9_io_obus_image_build.C | 8 | ||||
| -rw-r--r-- | src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml | 38 |
2 files changed, 37 insertions, 9 deletions
diff --git a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_image_build.C b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_image_build.C index 149e6ab76..4e8a11bf9 100644 --- a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_image_build.C +++ b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_image_build.C @@ -108,7 +108,7 @@ fapi2::ReturnCode p9_io_obus_image_build(CONST_OBUS& iTgt, void* const iHwImageP uint8_t configMode = 0; int xipSectionId = 0; bool loadImage = false; - fapi2::ATTR_CHIP_EC_FEATURE_HW446279_Type l_ATTR_CHIP_EC_FEATURE_HW446279; + fapi2::ATTR_CHIP_EC_FEATURE_P9C_LOGIC_ONLY_Type l_p9c; FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_OPTICS_CONFIG_MODE, iTgt, configMode), "Error from FAPI_ATTR_GET(ATTR_OPTICS_CONFIG_MODE)"); @@ -128,9 +128,11 @@ fapi2::ReturnCode p9_io_obus_image_build(CONST_OBUS& iTgt, void* const iHwImageP FAPI_IMP("ABUS IMAGE."); xipSectionId = P9_XIP_SECTION_IOPPE_IOO_ABUS; auto l_chip = iTgt.getParent<fapi2::TARGET_TYPE_PROC_CHIP>(); - FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_HW446279, l_chip, l_ATTR_CHIP_EC_FEATURE_HW446279)); + FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_P9C_LOGIC_ONLY, + l_chip, + l_p9c)); - if(l_ATTR_CHIP_EC_FEATURE_HW446279) + if(l_p9c) { FAPI_IMP("ABUS IMAGE LOAD."); loadImage = true; diff --git a/src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml index 0dcabedbb..30c42b408 100644 --- a/src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml +++ b/src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml @@ -771,22 +771,22 @@ </attribute> <!-- ********************************************************************* --> <attribute> - <id>ATTR_CHIP_EC_FEATURE_HW446279</id> + <id>ATTR_CHIP_EC_FEATURE_HW446279_DISABLE_FIX</id> <targetType>TARGET_TYPE_PROC_CHIP</targetType> <description> HW446279 SMP Abus Cable Failure + Disable HW fixes to rely on PPE SW recovery </description> <chipEcFeature> <chip> <name>ENUM_ATTR_NAME_CUMULUS</name> <ec> <value>0x13</value> - <test>LESS_THAN</test> + <test>GREATER_THAN_OR_EQUAL</test> </ec> </chip> </chipEcFeature> </attribute> - <!-- ********************************************************************* --> <attribute> <id>ATTR_CHIP_EC_FEATURE_HW430944</id> @@ -7650,17 +7650,43 @@ </attribute> <!-- ******************************************************************** --> <attribute> - <id>ATTR_CHIP_EC_FEATURE_HW440920</id> + <id>ATTR_CHIP_EC_FEATURE_HW440920_DISABLE_FIX</id> <targetType>TARGET_TYPE_PROC_CHIP</targetType> <description> - Revert to ordered ("p8") TLBIE mode, disable NMMU snoop of TLBIE. + HW fixes for HW440920 are present in chip logic, but + revert to ordered ("p8") TLBIE mode, disable NMMU snoop of TLBIE. + disable nest TLBIE changes </description> <chipEcFeature> <chip> <name>ENUM_ATTR_NAME_CUMULUS</name> <ec> <value>0x13</value> - <test>LESS_THAN</test> + <test>GREATER_THAN_OR_EQUAL</test> + </ec> + </chip> + </chipEcFeature> + </attribute> + <!-- ******************************************************************** --> + <attribute> + <id>ATTR_CHIP_EC_FEATURE_HW446453</id> + <targetType>TARGET_TYPE_PROC_CHIP</targetType> + <description> + NCU HW contains design change to address HW446453 + </description> + <chipEcFeature> + <chip> + <name>ENUM_ATTR_NAME_CUMULUS</name> + <ec> + <value>0x13</value> + <test>GREATER_THAN_OR_EQUAL</test> + </ec> + </chip> + <chip> + <name>ENUM_ATTR_NAME_NIMBUS</name> + <ec> + <value>0x23</value> + <test>GREATER_THAN_OR_EQUAL</test> </ec> </chip> </chipEcFeature> |
