[Mesa-dev] [PATCH] intel/genxml: Fix a few invalid field widths

Chris Wilson chris at chris-wilson.co.uk
Sat May 5 18:39:50 UTC 2018


A couple of typos found by inspecting field.end - field.start, revealed
a few wide integers declared as bool and some that ended before they
started.

Cc: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
---
 src/intel/genxml/gen4.xml  | 12 ++++++------
 src/intel/genxml/gen45.xml | 12 ++++++------
 src/intel/genxml/gen5.xml  | 12 ++++++------
 src/intel/genxml/gen6.xml  |  6 +++---
 src/intel/genxml/gen7.xml  |  6 +++---
 src/intel/genxml/gen75.xml |  8 ++++----
 6 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/src/intel/genxml/gen4.xml b/src/intel/genxml/gen4.xml
index 6f513c5833b..cd50a1012bc 100644
--- a/src/intel/genxml/gen4.xml
+++ b/src/intel/genxml/gen4.xml
@@ -961,12 +961,12 @@
     <field name="CLIP Unit URB Reallocation Request" start="10" end="10" type="bool"/>
     <field name="GS Unit URB Reallocation Request" start="9" end="9" type="bool"/>
     <field name="VS Unit URB Reallocation Request" start="8" end="8" type="bool"/>
-    <field name="CLIP Fence" start="52" end="61" type="bool"/>
-    <field name="GS Fence" start="42" end="51" type="bool"/>
-    <field name="VS Fence" start="32" end="41" type="bool"/>
-    <field name="CS Fence" start="84" end="94" type="bool"/>
-    <field name="VFE Fence" start="74" end="83" type="bool"/>
-    <field name="SF Fence" start="64" end="73" type="bool"/>
+    <field name="CLIP Fence" start="52" end="61" type="uint"/>
+    <field name="GS Fence" start="42" end="51" type="uint"/>
+    <field name="VS Fence" start="32" end="41" type="uint"/>
+    <field name="CS Fence" start="84" end="94" type="uint"/>
+    <field name="VFE Fence" start="74" end="83" type="uint"/>
+    <field name="SF Fence" start="64" end="73" type="uint"/>
   </instruction>
 
   <instruction name="XY_COLOR_BLT" bias="2" length="6">
diff --git a/src/intel/genxml/gen45.xml b/src/intel/genxml/gen45.xml
index fbd57a00c50..4d2c1534d3f 100644
--- a/src/intel/genxml/gen45.xml
+++ b/src/intel/genxml/gen45.xml
@@ -994,12 +994,12 @@
     <field name="CLIP Unit URB Reallocation Request" start="10" end="10" type="bool"/>
     <field name="GS Unit URB Reallocation Request" start="9" end="9" type="bool"/>
     <field name="VS Unit URB Reallocation Request" start="8" end="8" type="bool"/>
-    <field name="CLIP Fence" start="52" end="61" type="bool"/>
-    <field name="GS Fence" start="42" end="51" type="bool"/>
-    <field name="VS Fence" start="32" end="41" type="bool"/>
-    <field name="CS Fence" start="84" end="94" type="bool"/>
-    <field name="VFE Fence" start="74" end="83" type="bool"/>
-    <field name="SF Fence" start="64" end="73" type="bool"/>
+    <field name="CLIP Fence" start="52" end="61" type="uint"/>
+    <field name="GS Fence" start="42" end="51" type="uint"/>
+    <field name="VS Fence" start="32" end="41" type="uint"/>
+    <field name="CS Fence" start="84" end="94" type="uint"/>
+    <field name="VFE Fence" start="74" end="83" type="uint"/>
+    <field name="SF Fence" start="64" end="73" type="uint"/>
   </instruction>
 
   <instruction name="XY_COLOR_BLT" bias="2" length="6">
diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml
index 5c93ecdda30..5bb5a2c3312 100644
--- a/src/intel/genxml/gen5.xml
+++ b/src/intel/genxml/gen5.xml
@@ -1086,12 +1086,12 @@ i
     <field name="CLIP Unit URB Reallocation Request" start="10" end="10" type="bool"/>
     <field name="GS Unit URB Reallocation Request" start="9" end="9" type="bool"/>
     <field name="VS Unit URB Reallocation Request" start="8" end="8" type="bool"/>
-    <field name="CLIP Fence" start="52" end="61" type="bool"/>
-    <field name="GS Fence" start="42" end="51" type="bool"/>
-    <field name="VS Fence" start="32" end="41" type="bool"/>
-    <field name="CS Fence" start="84" end="94" type="bool"/>
-    <field name="VFE Fence" start="74" end="83" type="bool"/>
-    <field name="SF Fence" start="64" end="73" type="bool"/>
+    <field name="CLIP Fence" start="52" end="61" type="uint"/>
+    <field name="GS Fence" start="42" end="51" type="uint"/>
+    <field name="VS Fence" start="32" end="41" type="uint"/>
+    <field name="CS Fence" start="84" end="94" type="uint"/>
+    <field name="VFE Fence" start="74" end="83" type="uint"/>
+    <field name="SF Fence" start="64" end="73" type="uint"/>
   </instruction>
 
   <instruction name="XY_COLOR_BLT" bias="2" length="6">
diff --git a/src/intel/genxml/gen6.xml b/src/intel/genxml/gen6.xml
index 0493221bd72..f258065ebae 100644
--- a/src/intel/genxml/gen6.xml
+++ b/src/intel/genxml/gen6.xml
@@ -1888,7 +1888,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -1904,7 +1904,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -1920,7 +1920,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
diff --git a/src/intel/genxml/gen7.xml b/src/intel/genxml/gen7.xml
index baf42a7d32d..895f5d232b5 100644
--- a/src/intel/genxml/gen7.xml
+++ b/src/intel/genxml/gen7.xml
@@ -2537,7 +2537,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -2553,7 +2553,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -2569,7 +2569,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml
index 7b635b22dac..fe59446d83f 100644
--- a/src/intel/genxml/gen75.xml
+++ b/src/intel/genxml/gen75.xml
@@ -3021,7 +3021,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -3037,7 +3037,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -3053,7 +3053,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
@@ -3069,7 +3069,7 @@
       <value name="Invalid and Unloaded PD fault" value="3"/>
     </field>
     <field name= "SRCID of Fault" start="3" end="10" type="uint"/>
-    <field name="GTTSEL" start="11" end="1" type="uint">
+    <field name="GTTSEL" start="11" end="11" type="uint">
       <value name="PPGTT" value="0"/>
       <value name="GGTT" value="1"/>
     </field>
-- 
2.17.0



More information about the mesa-dev mailing list