On 04/12/2011 03:33 PM, c...@chad-versace.us wrote:
From: Chad Versace<c...@chad-versace.us>

... because grokking explicit assertions requires fewer neurons.

In brw_misc_state.c:emit_depthbuffer, change
     assert(tiling != I915_TILING_X&&  tiling != I915_TILING_NONE)
to
     assert(tiling == I915_TILING_Y)

Signed-off-by: Chad Versace<c...@chad-versace.us>
---
  src/mesa/drivers/dri/i965/brw_misc_state.c |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c 
b/src/mesa/drivers/dri/i965/brw_misc_state.c
index 74e911b..617712e 100644
--- a/src/mesa/drivers/dri/i965/brw_misc_state.c
+++ b/src/mesa/drivers/dri/i965/brw_misc_state.c
@@ -249,7 +249,7 @@ static void emit_depthbuffer(struct brw_context *brw)

        assert(region->tiling != I915_TILING_X);
        if (intel->gen>= 6)
-        assert(region->tiling != I915_TILING_NONE);
+        assert(region->tiling != I915_TILING_Y);

        BEGIN_BATCH(len);
        OUT_BATCH(_3DSTATE_DEPTH_BUFFER<<  16 | (len - 2));

Clearly you mean:
assert(region->tiling == I915_TILING_Y);

With that change: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to