Mesa (master): genxml: Fix 3DSTATE_DEPTH_BUFFER length on gen5.

Kenneth Graunke kwg at kemper.freedesktop.org
Thu May 4 01:59:03 UTC 2017


Module: Mesa
Branch: master
Commit: f321f695d38a28ccec91eafa9a5a83170032b0a4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f321f695d38a28ccec91eafa9a5a83170032b0a4

Author: Rafael Antognolli <rafael.antognolli at intel.com>
Date:   Wed May  3 17:53:35 2017 -0700

genxml: Fix 3DSTATE_DEPTH_BUFFER length on gen5.

The hardware docs are wrong, but the length used in the xml is also
wrong.

Signed-off-by: Rafael Antognolli <rafael.antognolli at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/intel/genxml/gen5.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml
index 0b8465086f..447499ca8c 100644
--- a/src/intel/genxml/gen5.xml
+++ b/src/intel/genxml/gen5.xml
@@ -649,14 +649,14 @@
     <field name="Y Offset" start="180" end="183" type="uint"/>
   </struct>
 
-  <instruction name="3DSTATE_DEPTH_BUFFER" bias="2" length="5">
-    <!-- The hardware docs incorrectly say length is 6. The actual length is 5. -->
+  <instruction name="3DSTATE_DEPTH_BUFFER" bias="2" length="6">
+    <!-- The hardware docs incorrectly say length is 7. The actual length is 6. -->
     <!-- DWord 0 -->
     <field name="Command Type" start="29" end="31" type="uint" default="3"/>
     <field name="Command SubType" start="27" end="28" type="uint" default="3"/>
     <field name="3D Command Opcode" start="24" end="26" type="uint" default="1"/>
     <field name="3D Command Sub Opcode" start="16" end="23" type="uint" default="5"/>
-    <field name="DWord Length" start="0" end="7" type="uint" default="3"/>
+    <field name="DWord Length" start="0" end="7" type="uint" default="4"/>
     <!-- DWord 1 -->
     <field name="Surface Type" start="61" end="63" type="uint">
       <value name="SURFTYPE_1D" value="0"/>




More information about the mesa-commit mailing list