From 5804a683b1e24789247b270a2b2140642b3bde20 Mon Sep 17 00:00:00 2001
From: Dan Crowell <dcrowell@us.ibm.com>
Date: Fri, 13 May 2016 15:06:18 -0500
Subject: [PATCH] Define attributes for synchronous mode

Changed ATTR_MC_SYNC_MODE to be writeable instead of platInit
ATTR_REQUIRED_SYNCH_MODE holds the rules on what mode to use

Change-Id: I006656e4e41701d2ed2175dd532057c217eff196
Original-Change-Id: Ibb8b01635db885490e4c4d287e0316bb2e9abffc
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/24547
Tested-by: Hostboot CI
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Tested-by: Jenkins Server
Tested-by: PPE CI
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42147
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
---
 .../xml/attribute_info/freq_attributes.xml    | 26 ++++++++++++-------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/src/import/chips/p9/procedures/xml/attribute_info/freq_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/freq_attributes.xml
index b85e22f4301..4b5ef665af9 100644
--- a/src/import/chips/p9/procedures/xml/attribute_info/freq_attributes.xml
+++ b/src/import/chips/p9/procedures/xml/attribute_info/freq_attributes.xml
@@ -50,16 +50,6 @@
     <platInit/>
   </attribute>
   <!-- ********************************************************************* -->
-  <attribute>
-    <id>ATTR_MEM_IN_SYNCH_MODE</id>
-    <targetType>TARGET_TYPE_SYSTEM</targetType>
-    <description>
-      Run the memory clocks in synchronous mode with the nest clocks.
-    </description>
-    <valueType>uint8</valueType>
-    <platInit/>
-  </attribute>
-  <!-- ********************************************************************* -->
   <attribute>
     <id>ATTR_MAX_ALLOWED_DIMM_FREQ</id>
     <targetType>TARGET_TYPE_SYSTEM</targetType>
@@ -91,4 +81,20 @@
     <platInit/>
   </attribute>
   <!-- ********************************************************************* -->
+  <attribute>
+    <id>ATTR_REQUIRED_SYNCH_MODE</id>
+    <targetType>TARGET_TYPE_SYSTEM</targetType>
+    <description>
+      Specify the system policy to enforce synchronous mode between memory and
+      nest. This drives the value of ATTR_MEM_IN_SYNCH_MODE.
+         UNDETERMINED : Run synchronously if the dimm and nest freq matches
+         ALWAYS : Require matching frequencies and deconfigure memory that
+                  does not match the nest
+         NEVER : Do not run synchronously, even if the frequencies match
+    </description>
+    <valueType>uint8</valueType>
+    <enum>UNDETERMINED=0, ALWAYS=1, NEVER=2</enum>
+    <platInit/>
+  </attribute>
+  <!-- ********************************************************************* -->
 </attributes>