diff options
| author | Scott D Phillips <[email protected]> | 2017-08-04 22:21:43 +0000 |
|---|---|---|
| committer | Rafael Antognolli <[email protected]> | 2017-08-15 09:06:29 -0700 |
| commit | d6539608a440f0102f88b375d20143f5f2d0c798 (patch) | |
| tree | d28f4d0da44ef37b47de1cdec7b58da4728bfce0 | |
| parent | fe74c8ffbf0658f2b5a9be65192ae1bf72eebe95 (diff) | |
intel/genxml: Fix gen10 BLEND_STATE variable length packing
BLEND_STATE packing was modified to be variable-length in:
9670124e31 genxml: Make BLEND_STATE command support variable length array.
The initial gen10.xml still had the old, fixed-length style
definition for BLEND_STATE. So gen10_upload_blend_state would
overwrite the packed BLEND_STATE_ENTRYs with its own fixed array
of all-zero entries when packing BLEND_STATE. This caused
BLEND_STATE upload to not work at all.
Fixes: aa416f515a ("i965/genxml: Add gen10.xml")
Reviewed-by: Rafael Antognolli <[email protected]>
Reviewed-by: Anuj Phogat <[email protected]>
| -rw-r--r-- | src/intel/genxml/gen10.xml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel/genxml/gen10.xml b/src/intel/genxml/gen10.xml index 23c2adb9951..a7ae49ae659 100644 --- a/src/intel/genxml/gen10.xml +++ b/src/intel/genxml/gen10.xml @@ -554,7 +554,7 @@ <field name="Write Disable Blue" start="0" end="0" type="bool"/> </struct> - <struct name="BLEND_STATE" length="17"> + <struct name="BLEND_STATE" length="1"> <field name="Alpha To Coverage Enable" start="31" end="31" type="bool"/> <field name="Independent Alpha Blend Enable" start="30" end="30" type="bool"/> <field name="Alpha To One Enable" start="29" end="29" type="bool"/> @@ -564,7 +564,7 @@ <field name="Color Dither Enable" start="23" end="23" type="bool"/> <field name="X Dither Offset" start="21" end="22" type="uint"/> <field name="Y Dither Offset" start="19" end="20" type="uint"/> - <group count="8" start="32" size="64"> + <group count="0" start="32" size="64"> <field name="Entry" start="0" end="63" type="BLEND_STATE_ENTRY"/> </group> </struct> |