<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Nov 29, 2016 at 4:03 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg">On Tue, Nov 29, 2016 at 12:48 PM, Kristian H. Kristensen <span dir="ltr" class="gmail_msg"><<a href="mailto:hoegsberg@gmail.com" class="gmail_msg" target="_blank">hoegsberg@gmail.com</a>></span> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" 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" class="gmail_msg" target="_blank">hoegsberg@gmail.com</a>><br class="gmail_msg">
---<br class="gmail_msg">
 src/intel/genxml/gen8.xml |  9 ++++++++-<br class="gmail_msg">
 src/intel/genxml/gen9.xml | 14 ++++++++------<br class="gmail_msg">
 2 files changed, 16 insertions(+), 7 deletions(-)<br class="gmail_msg">
<br class="gmail_msg">
diff --git a/src/intel/genxml/gen8.xml b/src/intel/genxml/gen8.xml<br class="gmail_msg">
index 165ff25..17234f6 100644<br class="gmail_msg">
--- a/src/intel/genxml/gen8.xml<br class="gmail_msg">
+++ b/src/intel/genxml/gen8.xml<br class="gmail_msg">
@@ -401,6 +401,13 @@<br class="gmail_msg">
     <value name="HALF_BORDER" value="6"/><br class="gmail_msg">
   </enum><br class="gmail_msg">
<br class="gmail_msg">
+  <enum name="TILE_MODE"><br class="gmail_msg">
+    <value name="LINEAR" value="0"/><br class="gmail_msg">
+    <value name="WMAJOR" value="1"/><br class="gmail_msg">
+    <value name="XMAJOR" value="2"/><br class="gmail_msg">
+    <value name="YMAJOR" value="3"/><br class="gmail_msg">
+  </enum><br class="gmail_msg">
+<br class="gmail_msg">
   <struct name="3DSTATE_CONSTANT_BODY" length="10"><br class="gmail_msg">
     <field name="Constant Buffer 1 Read Length" start="16" end="31" type="uint"/><br class="gmail_msg">
     <field name="Constant Buffer 0 Read Length" start="0" end="15" type="uint"/><br class="gmail_msg">
@@ -670,7 +677,7 @@<br class="gmail_msg">
       <value name="HALIGN 8" value="2"/><br class="gmail_msg">
       <value name="HALIGN 16" value="3"/><br class="gmail_msg">
     </field><br class="gmail_msg">
-    <field name="Tile Mode" start="12" end="13" type="uint"><br class="gmail_msg">
+    <field name="Tile Mode" start="12" end="13" type="TILE_MODE"><br class="gmail_msg">
       <value name="LINEAR" value="0"/><br class="gmail_msg">
       <value name="WMAJOR" value="1"/><br class="gmail_msg">
       <value name="XMAJOR" value="2"/><br class="gmail_msg"></blockquote><div class="gmail_msg"><br class="gmail_msg"></div></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">I don't think you want these anymore.<br class="gmail_msg"></div></div></div></div></blockquote><div> </div><div>Right, thanks</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg"></div><div class="gmail_msg">As a side-note, what's the purpose of pulling one-use things out into enums?  Tile Mode is not very general.</div></div></div></div></blockquote><div><br></div><div>To let gen_pack_header.py use an enum in the packing struct. Maybe we should just generate C enums for inline values instead, though that breaks for the cases where inline values are duplicated. That works today because the C preprocessor ignores duplicate but identical #defines.</div><div><br></div><div>Kristian</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg"> </div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" 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 class="gmail_msg">
index ec85494..c9fb770 100644<br class="gmail_msg">
--- a/src/intel/genxml/gen9.xml<br class="gmail_msg">
+++ b/src/intel/genxml/gen9.xml<br class="gmail_msg">
@@ -422,6 +422,13 @@<br class="gmail_msg">
     <value name="HALF_BORDER" value="6"/><br class="gmail_msg">
   </enum><br class="gmail_msg">
<br class="gmail_msg">
+  <enum name="TILE_MODE"><br class="gmail_msg">
+    <value name="LINEAR" value="0"/><br class="gmail_msg">
+    <value name="WMAJOR" value="1"/><br class="gmail_msg">
+    <value name="XMAJOR" value="2"/><br class="gmail_msg">
+    <value name="YMAJOR" value="3"/><br class="gmail_msg">
+  </enum><br class="gmail_msg">
+<br class="gmail_msg">
   <struct name="3DSTATE_CONSTANT_BODY" length="10"><br class="gmail_msg">
     <field name="Constant Buffer 1 Read Length" start="16" end="31" type="uint"/><br class="gmail_msg">
     <field name="Constant Buffer 0 Read Length" start="0" end="15" type="uint"/><br class="gmail_msg">
@@ -696,12 +703,7 @@<br class="gmail_msg">
       <value name="HALIGN 8" value="2"/><br class="gmail_msg">
       <value name="HALIGN 16" value="3"/><br class="gmail_msg">
     </field><br class="gmail_msg">
-    <field name="Tile Mode" start="12" end="13" type="uint"><br class="gmail_msg">
-      <value name="LINEAR" value="0"/><br class="gmail_msg">
-      <value name="WMAJOR" value="1"/><br class="gmail_msg">
-      <value name="XMAJOR" value="2"/><br class="gmail_msg">
-      <value name="YMAJOR" value="3"/><br class="gmail_msg">
-    </field><br class="gmail_msg">
+    <field name="Tile Mode" start="12" end="13" type="TILE_MODE"/><br class="gmail_msg">
     <field name="Vertical Line Stride" start="11" end="11" type="uint"/><br class="gmail_msg">
     <field name="Vertical Line Stride Offset" start="10" end="10" type="uint"/><br class="gmail_msg">
     <field name="Sampler L2 Bypass Mode Disable" start="9" end="9" type="bool"/><br class="gmail_msg">
</blockquote></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="m_3476637870557025169HOEnZb gmail_msg"><font color="#888888" class="gmail_msg">--<br class="gmail_msg">
2.9.3<br class="gmail_msg">
<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
mesa-dev mailing list<br class="gmail_msg">
<a href="mailto:mesa-dev@lists.freedesktop.org" class="gmail_msg" target="_blank">mesa-dev@lists.freedesktop.org</a><br class="gmail_msg">
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br class="gmail_msg">
</font></span></blockquote></div></div></div></blockquote></div></div>