From af200c2285fb6096b3657b3e983007205ac706b5 Mon Sep 17 00:00:00 2001
From: Prasad Bg Ranganath <prasadbgr@in.ibm.com>
Date: Thu, 14 Sep 2017 11:39:45 -0500
Subject: SAFE mode:moved writing safe mode value from istep 6 to istep 10

Change-Id: Ic67241ad192f82c43a4e5f2af8cab6f945a4dc35
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46210
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
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: Juan R. Medina <jrmedina@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46320
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
---
 .../xml/attribute_info/pm_plat_attributes.xml      | 23 ++++++++++------------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml
index fb5043c7..fb2ebfdd 100644
--- a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml
+++ b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml
@@ -147,19 +147,16 @@
   <!-- ********************************************************************* -->
   <attribute>
     <id>ATTR_SAFE_MODE_NOVDM_UPLIFT_MV</id>
-
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
-      Safe mode voltage in mv
-      Consumer: p9_setup_evid.C
-
-      Overridden by p9_pstate_parameter_block
-
+    Attribute in place to allow override for boot mode voltage
+    Consumer: p9_setup_evid.C
     </description>
     <valueType>uint32</valueType>
     <platInit/>
     <initToZero/>
     <writeable/>
+    <overrideOnly/>
   </attribute>
 
 
@@ -170,13 +167,13 @@
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
       Safe mode voltage in mv
-      Consumer: p9_setup_evid.C
-
-      Overridden by p9_pstate_parameter_block
+    This attribute is used within the hwp to update the 
+    VDM config register.
+    Consumer: p9_setup_evid.C
 
+    Modified in  p9_pstate_parameter_block
     </description>
     <valueType>uint32</valueType>
-    <platInit/>
     <initToZero/>
     <writeable/>
   </attribute>
@@ -188,13 +185,13 @@
     <targetType>TARGET_TYPE_PROC_CHIP</targetType>
     <description>
       Safe mode frequency in MHZ
+      This attribute is used within the hwp to update QPPM DPLL
+      frequency register
       Consumer: p9_setup_evid.C
 
-      Overridden by p9_pstate_parameter_block
-
+      Modified by p9_pstate_parameter_block
     </description>
     <valueType>uint32</valueType>
-    <platInit/>
     <initToZero/>
     <writeable/>
   </attribute>
-- 
cgit v1.2.1