i965/gen10: Remove unnecessary workaround.
authorRafael Antognolli <rafael.antognolli@intel.com>
Fri, 19 Jul 2019 22:41:35 +0000 (15:41 -0700)
committerRafael Antognolli <rafael.antognolli@intel.com>
Mon, 29 Jul 2019 23:54:17 +0000 (16:54 -0700)
commitf27908152b504444d6195b91825a74b11e896780
tree8fe02af6efeeaba5a7f53da62122987dcb979807
parent44e713eddb44a1cbd0861c216ba5921da8ee3570
i965/gen10: Remove unnecessary workaround.

In fact, the description of the workaround states that the mask field
doesn't work correctly on gen10, and we need to set it to 0xffff even we
we only want to update a single field:

 "The mask bits are not implemented properly on 3DSTATE_3D_MODE.  Driver
 must always program bits 31:16 of DW1 a value of 0xFFFF.   This means
 if it is only updating 1 field, it must update all the fields to the
 correct value."

So unless we want to change any of the fields of 3DSTATE_3D_MODE,
there's not need to emit. Additionally, it seems this workaround is not
required on gen11. And last but not least, this workaround is not
implemented on iris or anv, and it doesn't seem to be missed there.

So let's just remove the whole thing.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_state_upload.c