Implementing Michael Floyds improvements.

Change-Id: I98bc95095f71d9d795b37cfb0cfb5c37f959a0b4
Original-Change-Id: I719f5f2ab79755ec09f2420b6279569b0e05f3b2
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40148
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: Anusha Reddy Rangareddygari <anusrang@in.ibm.com>
Reviewed-by: Michael S. Floyd <mfloyd@us.ibm.com>
Reviewed-by: PARVATHI RACHAKONDA <prachako@in.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55507
CI-Ready: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>

src/import/chips/p9/procedures/xml/attribute_info/chip_ec_attributes.xml

@@ -1946,7 +1946,7 @@
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
       Nimbus DD1;  set to 0b000
-      Numbus DD2+; set to 0b110 
+      Numbus DD2+; set to 0b110
     </description>
     <chipEcFeature>
       <chip>
@@ -2045,7 +2045,7 @@
     <id>ATTR_CHIP_EC_FEATURE_HW409069</id>
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
-      Nimbus DD2+: HW409069 load_larx protection not activated because of dtag_data_resp 
+      Nimbus DD2+: HW409069 load_larx protection not activated because of dtag_data_resp
       in hot lock load loop pvp test
     </description>
     <chipEcFeature>
@@ -2497,8 +2497,41 @@
       </chip>
     </chipEcFeature>
   </attribute>
-
-
+  <!-- ********************************************************************* -->
+  <attribute>
+    <id>ATTR_CHIP_EC_FEATURE_SKEWADJ_P9NDD1_INIT</id>
+    <targetType>TARGET_TYPE_PROC_CHIP</targetType>
+    <description>
+      Inits for skewAdj in P9 ndd1. These are different from the default inits used starting in NDD2.
+    </description>
+    <chipEcFeature>
+      <chip>
+        <name>ENUM_ATTR_NAME_NIMBUS</name>
+        <ec>
+          <value>0x20</value>
+          <test>LESS_THAN</test>
+        </ec>
+      </chip>
+    </chipEcFeature>
+  </attribute>
+  <!-- ********************************************************************* -->
+  <attribute>
+    <id>ATTR_CHIP_EC_FEATURE_DCADJ_P9NDD1_INIT</id>
+    <targetType>TARGET_TYPE_PROC_CHIP</targetType>
+    <description>
+      Inits for dutyCycleAdj in P9 ndd1. These are different from the default inits used starting in NDD2.
+    </description>
+    <chipEcFeature>
+      <chip>
+        <name>ENUM_ATTR_NAME_NIMBUS</name>
+        <ec>
+          <value>0x20</value>
+          <test>LESS_THAN</test>
+        </ec>
+      </chip>
+    </chipEcFeature>
+  </attribute>
+  <!-- ********************************************************************* -->
   <attribute>
     <id>ATTR_CHIP_EC_FEATURE_HW397129</id>>
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>