On 01/17/2012 04:57 PM, Eric Anholt wrote:
This field is actually set up above. --- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-)diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c index aed0b5d..25d4a3e 100644 --- a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c @@ -94,7 +94,6 @@ gen7_update_texture_surface(struct gl_context *ctx, GLuint unit) gen7_set_surface_tiling(surf, intelObj->mt->region->tiling); /* ss0 remaining fields: - * - vertical_alignment * - horizontal_alignment * - vert_line_stride (exists on gen6 but we ignore it) * - vert_line_stride_ofs (exists on gen6 but we ignore it)
Series looks good to me, aside from the issue with patch 2. For the series: Reviewed-by: Kenneth Graunke <[email protected]> _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
