Attribute support of customization of Nimbus DD1 PCI reference clock …

…speed. Change-Id: I5bb1646868fca15aca744b311ab5d2bc5dd64739 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38297 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Reviewed-by: Matt K. Light <mklight@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38306 Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
open-power · Mar 24, 2017 · c6c8d5e · c6c8d5e
1 parent b4f215e
commit c6c8d5e
Show file tree

Hide file tree

Showing 5 changed files with 47 additions and 2 deletions.
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/mss_attribute_accessors.H b/src/import/chips/p9/procedures/hwp/memory/lib/mss_attribute_accessors.H
@@ -27378,6 +27378,28 @@ fapi_try_exit:
     return fapi2::current_err;
 }
 
+///
+/// @brief ATTR_DD1_SLOW_PCI_REF_CLOCK getter
+/// @param[out] uint8_t& reference to store the value
+/// @note Generated by gen_accessors.pl generateParameters (SYSTEM)
+/// @return fapi2::ReturnCode - FAPI2_RC_SUCCESS iff get is OK
+/// @note  MRW control to permit Normal (100 MHz) or Slow (94 MHz) operation of PCIE
+/// reference clock. On Nimbus DD1 HW, Slow operation is required to achieve Gen4
+/// operation. Provided by the
+/// MRW.
+///
+inline fapi2::ReturnCode dd1_slow_pci_ref_clock(uint8_t& o_value)
+{
+
+    FAPI_TRY( FAPI_ATTR_GET(fapi2::ATTR_DD1_SLOW_PCI_REF_CLOCK, fapi2::Target<fapi2::TARGET_TYPE_SYSTEM>(), o_value) );
+    return fapi2::current_err;
+
+fapi_try_exit:
+    FAPI_ERR("failed accessing ATTR_DD1_SLOW_PCI_REF_CLOCK: 0x%lx (system target)",
+             uint64_t(fapi2::current_err));
+    return fapi2::current_err;
+}
+
 ///
 /// @brief ATTR_PROC_FABRIC_ASYNC_SAFE_MODE getter
 /// @param[out] uint8_t& reference to store the value

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
@@ -1844,10 +1844,10 @@
   </attribute>
   <!-- ******************************************************************** -->
   <attribute>
-    <id>ATTR_CHIP_EC_FEATURE_SLOW_PCI_REF_CLOCK</id>
+    <id>ATTR_CHIP_EC_FEATURE_SLOW_PCI_REF_CLOCK_ENABLE</id>
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
-      DD1 only: leverage SS PLL to provide reduced frequency reference clock
+      DD1 only: enable use of SS PLL to provide reduced frequency reference clock
                 (94 MHz, instead of nominal 100 MHz) for PCI PLL
     </description>
     <chipEcFeature>

diff --git a/src/import/chips/p9/procedures/xml/attribute_info/nest_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/nest_attributes.xml
@@ -135,6 +135,24 @@
   <platInit/>
  </attribute>
 <!-- ********************************************************************** -->
+<attribute>
+  <id>ATTR_DD1_SLOW_PCI_REF_CLOCK</id>
+  <targetType>TARGET_TYPE_SYSTEM</targetType>
+  <description>
+    MRW control to permit Normal (100 MHz) or Slow (94 MHz) operation
+    of PCIE reference clock.  On Nimbus DD1 HW, Slow operation is required
+    to achieve Gen4 operation.
+    Provided by the MRW.
+  </description>
+  <valueType>uint8</valueType>
+  <enum>
+    NORMAL = 0x00,
+    SLOW = 0x01
+  </enum>
+  <platInit/>
+  <writeable/>
+ </attribute>
+<!-- ********************************************************************** -->
 <attribute>
   <id>ATTR_PROC_FABRIC_ASYNC_SAFE_MODE</id>
   <targetType>TARGET_TYPE_SYSTEM</targetType>

diff --git a/src/import/hwpf/fapi2/xml/attribute_info/hb_temp_defaults.xml b/src/import/hwpf/fapi2/xml/attribute_info/hb_temp_defaults.xml
@@ -153,6 +153,10 @@
         <id>ATTR_PROC_EFF_FABRIC_CHIP_ID</id>
         <default>0x0</default>
      </attribute>
+     <attribute>
+        <id>ATTR_DD1_SLOW_PCI_REF_CLOCK</id>
+        <default>0x01</default>
+     </attribute>
     <attribute>
         <id>ATTR_SYSTEM_DISABLE_QUEUED_SCAN</id>
         <default>0x00</default>

diff --git a/src/usr/targeting/common/xmltohb/attribute_types.xml b/src/usr/targeting/common/xmltohb/attribute_types.xml
@@ -14792,6 +14792,7 @@ Measured in GB</description>
     </simpleType>
     <persistency>non-volatile</persistency>
     <readable/>
+    <writeable/><!-- SBE requirement only -->
     <hwpfToHbAttrMap>
         <id>ATTR_DD1_SLOW_PCI_REF_CLOCK</id>
         <macro>DIRECT</macro>