summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael Antognolli <rafael.antognolli@intel.com>2017-08-07 12:14:04 -0700
committerRafael Antognolli <rafael.antognolli@intel.com>2018-04-05 07:42:45 -0700
commit7e616ae2014d78e8173fd4ade7a001b85ffd0a08 (patch)
treeeb473382680a3812737d2a4d0ebacbc4c92e9145
parent8e1f2e1d2dee09cf61c13c0aadf99712f59fafb9 (diff)
intel/genxml: Use a single field for clear color address on gen10.
genxml does not support having two address fields with different names but same position in the state struct. Both "Clear Color Address" and "Clear Depth Address Low" mean the same thing, only for different surface types. To workaround this genxml limitation, rename "Clear Color Address" to "Clear Value Address" and use it for both color and depth. Do the same for the high bits. TODO: add support for multiple addresses at the same position in the xml. v2: Combine high and low order bits into a single address field. Signed-off-by: Rafael Antognolli <rafael.antognolli@intel.com> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net> Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
-rw-r--r--src/intel/genxml/gen10.xml7
-rw-r--r--src/intel/genxml/gen11.xml7
2 files changed, 6 insertions, 8 deletions
diff --git a/src/intel/genxml/gen10.xml b/src/intel/genxml/gen10.xml
index 8c35d70e9d2..5aa518fb533 100644
--- a/src/intel/genxml/gen10.xml
+++ b/src/intel/genxml/gen10.xml
@@ -575,12 +575,11 @@
<field name="Clear Value Address Enable" start="330" end="330" type="bool"/>
<field name="Quilt Height" start="325" end="329" type="uint"/>
<field name="Quilt Width" start="320" end="324" type="uint"/>
+ <!-- This address points to the address of a Clear Color in the case of
+ CCS_D and CCS_E surfaces, and Depth Color for HiZ surfaces -->
+ <field name="Clear Value Address" start="390" end="431" type="address"/>
<field name="Red Clear Color" start="384" end="415" type="int"/>
- <field name="Clear Color Address" start="390" end="415" type="address"/>
- <field name="Clear Depth Address Low" start="390" end="415" type="address"/>
<field name="Green Clear Color" start="416" end="447" type="int"/>
- <field name="Clear Color Address High" start="416" end="431" type="address"/>
- <field name="Clear Depth Address High" start="416" end="431" type="address"/>
<field name="Blue Clear Color" start="448" end="479" type="int"/>
<field name="Alpha Clear Color" start="480" end="511" type="int"/>
</struct>
diff --git a/src/intel/genxml/gen11.xml b/src/intel/genxml/gen11.xml
index 517f0beb93b..0cdd6dce0d6 100644
--- a/src/intel/genxml/gen11.xml
+++ b/src/intel/genxml/gen11.xml
@@ -576,13 +576,12 @@
<field name="Clear Value Address Enable" start="330" end="330" type="bool"/>
<field name="Quilt Height" start="325" end="329" type="uint"/>
<field name="Quilt Width" start="320" end="324" type="uint"/>
+ <!-- This address points to the address of a Clear Color in the case of
+ CCS_D and CCS_E surfaces, and Depth Color for HiZ surfaces -->
+ <field name="Clear Value Address" start="390" end="431" type="address"/>
<field name="Red Clear Color" start="384" end="415" type="int"/>
- <field name="Clear Color Address" start="390" end="415" type="address"/>
- <field name="Clear Depth Address Low" start="390" end="415" type="address"/>
<field name="Clear Color Conversion Enable" start="389" end="389" type="uint"/>
<field name="Green Clear Color" start="416" end="447" type="int"/>
- <field name="Clear Color Address High" start="416" end="431" type="address"/>
- <field name="Clear Depth Address High" start="416" end="431" type="address"/>
<field name="Blue Clear Color" start="448" end="479" type="int"/>
<field name="Alpha Clear Color" start="480" end="511" type="int"/>
</struct>