From 4dc25b761553215a686650c6bcb8831ef0c9120d Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Mon, 7 Oct 2013 11:08:27 -0700 Subject: [PATCH] i965/blorp: Use R16_UNORM for Z16 surfaces. Currently, all that matters is that we copy the correct number of bits, so any format that has 16-bits of data will work fine. Once BLORP begins handling format conversions, the sampler will need to correctly interpret the data. We don't need a depth format, but we do need the right number of components and data type (UNORM). For Z16, this means using R16_UNORM. Signed-off-by: Kenneth Graunke Reviewed-by: Ian Romanick Reviewed-by: Chad Versace Reviewed-by: Eric Anholt Reviewed-by: Daniel Vetter --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.cpp b/src/mesa/drivers/dri/i965/brw_blorp.cpp index f5731b6c0f9..46bb244b80c 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp.cpp +++ b/src/mesa/drivers/dri/i965/brw_blorp.cpp @@ -96,12 +96,7 @@ brw_blorp_surface_info::set(struct brw_context *brw, this->brw_surfaceformat = BRW_SURFACEFORMAT_B8G8R8A8_UNORM; break; case MESA_FORMAT_Z16: - /* The miptree consists of 16 bits per pixel of depth data. Since depth - * values don't require any blending, it doesn't matter how we interpret - * the bit pattern as long as we copy the right amount of data, so just - * map is as 8-bit RG. - */ - this->brw_surfaceformat = BRW_SURFACEFORMAT_R8G8_UNORM; + this->brw_surfaceformat = BRW_SURFACEFORMAT_R16_UNORM; break; default: if (is_render_target) { -- 2.30.2