<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 16, 2016 at 10:14 AM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat 11 Jun 2016, Jason Ekstrand wrote:<br>
> They're already zero-initialized and we have no plans of doing anything<br>
> more interesting with them.<br>
> ---<br>
>  src/intel/isl/isl_surface_state.c | 9 ---------<br>
>  1 file changed, 9 deletions(-)<br>
<br>
<br>
<br>
</span><span class="">> @@ -261,12 +258,6 @@ isl_genX(surf_fill_state_s)(const struct isl_device *dev, void *state,<br>
>     s.MultisampledSurfaceStorageFormat =<br>
>        isl_to_gen_multisample_layout[info->surf->msaa_layout];<br>
>     s.NumberofMultisamples = ffs(info->surf->samples) - 1;<br>
> -   s.MultisamplePositionPaletteIndex = 0; /* UNUSED */<br>
> -<br>
> -   s.XOffset = 0;<br>
> -   s.YOffset = 0;<br>
<br>
</span>Wait. Seriously? i965 no longer uses XOffset/YOffset on *any* gen!?!?!?<br>
When did that happen? I've been waiting for years for i965 to stop using<br>
XOffset/YOffset on old hardware, but obviously I wasn't paying enough<br>
attention to the git logs.<br></blockquote><div><br></div><div>Nope.  Further on I add them back in for real. <br></div></div><br></div></div>