i965/gen7: Fix the /* ignored */ comment on constant surface setup.
authorEric Anholt <eric@anholt.net>
Tue, 27 Mar 2012 21:03:26 +0000 (14:03 -0700)
committerEric Anholt <eric@anholt.net>
Mon, 9 Apr 2012 21:35:12 +0000 (14:35 -0700)
It turns out this field *is* used, and it's the stride between samples
from the buffer.  Discovered during TBO debugging.

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

index cd044dffd67d6d3dd3fe47b47afaa96a7b5919f6..439d8b082cacb9998a742d2eb28cb930247fde73 100644 (file)
@@ -164,7 +164,7 @@ gen7_create_constant_surface(struct brw_context *brw,
    surf->ss2.width = w & 0x7f;            /* bits 6:0 of size or width */
    surf->ss2.height = (w >> 7) & 0x1fff;  /* bits 19:7 of size or width */
    surf->ss3.depth = (w >> 20) & 0x7f;    /* bits 26:20 of size or width */
-   surf->ss3.pitch = (16 - 1); /* ignored */
+   surf->ss3.pitch = (16 - 1); /* stride between samples */
    gen7_set_surface_tiling(surf, I915_TILING_NONE); /* tiling now allowed */
 
    if (brw->intel.is_haswell) {