diff options
author | Eric Anholt <[email protected]> | 2013-01-29 11:59:29 +1100 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2013-03-05 16:02:38 -0800 |
commit | 88b20d58344ed336b146799c0bf1149a932dc2d7 (patch) | |
tree | 97235799a3de1306d9c3d777bf6df9c3f23bf416 /src | |
parent | 731d474d98ab0cbba5e8baeaf4b2b9d24950c430 (diff) |
intel: Improve the matching (more formats!) for TexImage from PBOs.
Mesa core is the place for encoding what format/type matches a mesa
format, so rely on that.
Reviewed-by: Chad Versace <[email protected]>
Diffstat (limited to 'src')
-rw-r--r-- | src/mesa/drivers/dri/intel/intel_tex_image.c | 28 |
1 files changed, 3 insertions, 25 deletions
diff --git a/src/mesa/drivers/dri/intel/intel_tex_image.c b/src/mesa/drivers/dri/intel/intel_tex_image.c index 63a1be03775..fa51afba724 100644 --- a/src/mesa/drivers/dri/intel/intel_tex_image.c +++ b/src/mesa/drivers/dri/intel/intel_tex_image.c @@ -108,29 +108,6 @@ intel_miptree_create_for_teximage(struct intel_context *intel, false /* force_y_tiling */); } -/* There are actually quite a few combinations this will work for, - * more than what I've listed here. - */ -static bool -check_pbo_format(GLenum format, GLenum type, - gl_format mesa_format) -{ - switch (mesa_format) { - case MESA_FORMAT_ARGB8888: - return (format == GL_BGRA && (type == GL_UNSIGNED_BYTE || - type == GL_UNSIGNED_INT_8_8_8_8_REV)); - case MESA_FORMAT_RGB565: - return (format == GL_RGB && type == GL_UNSIGNED_SHORT_5_6_5); - case MESA_FORMAT_L8: - return (format == GL_LUMINANCE && type == GL_UNSIGNED_BYTE); - case MESA_FORMAT_YCBCR: - return (type == GL_UNSIGNED_SHORT_8_8_MESA || type == GL_UNSIGNED_BYTE); - default: - return false; - } -} - - /* XXX: Do this for TexSubImage also: */ static bool @@ -157,9 +134,10 @@ try_pbo_upload(struct gl_context *ctx, return false; } - if (!check_pbo_format(format, type, intelImage->base.Base.TexFormat)) { + if (!_mesa_format_matches_format_and_type(image->TexFormat, + format, type, false)) { DBG("%s: format mismatch (upload to %s with format 0x%x, type 0x%x)\n", - __FUNCTION__, _mesa_get_format_name(intelImage->base.Base.TexFormat), + __FUNCTION__, _mesa_get_format_name(image->TexFormat), format, type); return false; } |