<div dir="ltr">Thanks!  This has always bothered me.<br><br>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There are is no alternative.<br>
<br>
Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
---<br>
 src/mesa/drivers/dri/i965/<wbr>gen6_depth_state.c | 22 ++++++++--------------<br>
 1 file changed, 8 insertions(+), 14 deletions(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/<wbr>gen6_depth_state.c b/src/mesa/drivers/dri/i965/<wbr>gen6_depth_state.c<br>
index 3f14006..cb0ed25 100644<br>
--- a/src/mesa/drivers/dri/i965/<wbr>gen6_depth_state.c<br>
+++ b/src/mesa/drivers/dri/i965/<wbr>gen6_depth_state.c<br>
@@ -191,20 +191,14 @@ gen6_emit_depth_stencil_hiz(<wbr>struct brw_context *brw,<br>
          uint32_t offset = 0;<br>
<br>
          if (stencil_mt->array_layout == ALL_SLICES_AT_EACH_LOD) {<br>
-            if (stencil_mt->format == MESA_FORMAT_S_UINT8) {<br>
-               /* Note: we can't compute the stencil offset using<br>
-                * intel_region_get_aligned_<wbr>offset(), because stencil_region<br>
-                * claims that the region is untiled even though it's W tiled.<br>
-                */<br>
-               offset =<br>
-                  stencil_mt->level[lod].level_y * stencil_mt->pitch +<br>
-                  stencil_mt->level[lod].level_x * 64;<br>
-            } else {<br>
-               offset = intel_miptree_get_aligned_<wbr>offset(<br>
-                           stencil_mt,<br>
-                           stencil_mt->level[lod].level_<wbr>x,<br>
-                           stencil_mt->level[lod].level_<wbr>y);<br>
-            }<br>
+            assert(stencil_mt->format == MESA_FORMAT_S_UINT8);<br>
+<br>
+            /* Note: we can't compute the stencil offset using<br>
+             * intel_region_get_aligned_<wbr>offset(), because stencil_region<br>
+             * claims that the region is untiled even though it's W tiled.<br>
+             */<br>
+            offset = stencil_mt->level[lod].level_y * stencil_mt->pitch +<br>
+                     stencil_mt->level[lod].level_x * 64;<br>
          }<br>
<br>
         BEGIN_BATCH(3);<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.5.5<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>