<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 29, 2016 at 12:48 PM, Kristian H. Kristensen <span dir="ltr"><<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Kristian H. Kristensen <<a href="mailto:hoegsberg@gmail.com">hoegsberg@gmail.com</a>><br>
---<br>
 src/intel/genxml/gen8.xml |  9 ++++++++-<br>
 src/intel/genxml/gen9.xml | 14 ++++++++------<br>
 2 files changed, 16 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/src/intel/genxml/gen8.xml b/src/intel/genxml/gen8.xml<br>
index 165ff25..17234f6 100644<br>
--- a/src/intel/genxml/gen8.xml<br>
+++ b/src/intel/genxml/gen8.xml<br>
@@ -401,6 +401,13 @@<br>
     <value name="HALF_BORDER" value="6"/><br>
   </enum><br>
<br>
+  <enum name="TILE_MODE"><br>
+    <value name="LINEAR" value="0"/><br>
+    <value name="WMAJOR" value="1"/><br>
+    <value name="XMAJOR" value="2"/><br>
+    <value name="YMAJOR" value="3"/><br>
+  </enum><br>
+<br>
   <struct name="3DSTATE_CONSTANT_BODY" length="10"><br>
     <field name="Constant Buffer 1 Read Length" start="16" end="31" type="uint"/><br>
     <field name="Constant Buffer 0 Read Length" start="0" end="15" type="uint"/><br>
@@ -670,7 +677,7 @@<br>
       <value name="HALIGN 8" value="2"/><br>
       <value name="HALIGN 16" value="3"/><br>
     </field><br>
-    <field name="Tile Mode" start="12" end="13" type="uint"><br>
+    <field name="Tile Mode" start="12" end="13" type="TILE_MODE"><br>
       <value name="LINEAR" value="0"/><br>
       <value name="WMAJOR" value="1"/><br>
       <value name="XMAJOR" value="2"/><br></blockquote><div><br></div><div>I don't think you want these anymore.<br><br></div><div>As a side-note, what's the purpose of pulling one-use things out into enums?  Tile Mode is not very general.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/src/intel/genxml/gen9.xml b/src/intel/genxml/gen9.xml<br>
index ec85494..c9fb770 100644<br>
--- a/src/intel/genxml/gen9.xml<br>
+++ b/src/intel/genxml/gen9.xml<br>
@@ -422,6 +422,13 @@<br>
     <value name="HALF_BORDER" value="6"/><br>
   </enum><br>
<br>
+  <enum name="TILE_MODE"><br>
+    <value name="LINEAR" value="0"/><br>
+    <value name="WMAJOR" value="1"/><br>
+    <value name="XMAJOR" value="2"/><br>
+    <value name="YMAJOR" value="3"/><br>
+  </enum><br>
+<br>
   <struct name="3DSTATE_CONSTANT_BODY" length="10"><br>
     <field name="Constant Buffer 1 Read Length" start="16" end="31" type="uint"/><br>
     <field name="Constant Buffer 0 Read Length" start="0" end="15" type="uint"/><br>
@@ -696,12 +703,7 @@<br>
       <value name="HALIGN 8" value="2"/><br>
       <value name="HALIGN 16" value="3"/><br>
     </field><br>
-    <field name="Tile Mode" start="12" end="13" type="uint"><br>
-      <value name="LINEAR" value="0"/><br>
-      <value name="WMAJOR" value="1"/><br>
-      <value name="XMAJOR" value="2"/><br>
-      <value name="YMAJOR" value="3"/><br>
-    </field><br>
+    <field name="Tile Mode" start="12" end="13" type="TILE_MODE"/><br>
     <field name="Vertical Line Stride" start="11" end="11" type="uint"/><br>
     <field name="Vertical Line Stride Offset" start="10" end="10" type="uint"/><br>
     <field name="Sampler L2 Bypass Mode Disable" start="9" end="9" type="bool"/><br>
<span class="HOEnZb"><font color="#888888">--<br>
2.9.3<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>