diff options
| author | Chad Versace <[email protected]> | 2016-12-22 14:39:44 -0800 |
|---|---|---|
| committer | Chad Versace <[email protected]> | 2016-12-27 09:14:00 -0800 |
| commit | f3739810e337d0f03514b9b967594c8dc010dfa4 (patch) | |
| tree | 2ed33b5ecfb60ac0515447fa1cc26af2fde859e0 /src | |
| parent | 9aa6ab074876dfa853863a74edad0b5b017fb8c7 (diff) | |
mesa/texformat: Handle GL_RGBA + GL_UNSIGNED_SHORT_5_5_5_1
_mesa_choose_tex_format() already handles GL_RGBA + GL_UNSIGNED_SHORT_1_5_5_5_REV
by converting it to MESA_FORMAT_B5G5R5A1_UNORM. Teach it do the same for
the non-reversed type. Otherwise, the switch's fallthrough converts it
to an 8888 format, which has incompatible precision in the alpha
channel.
Patch 2/2 to fix dEQP-EGL.functional.image.modify.tex_rgb5_a1_tex_subimage_rgba8
on Intel.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99185
Cc: Haixia Shi <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
Cc: "13.0" <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mesa/main/texformat.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c index be2581b004f..baa3988f0a0 100644 --- a/src/mesa/main/texformat.c +++ b/src/mesa/main/texformat.c @@ -74,6 +74,8 @@ _mesa_choose_tex_format(struct gl_context *ctx, GLenum target, RETURN_IF_SUPPORTED(MESA_FORMAT_B4G4R4A4_UNORM); } else if (type == GL_UNSIGNED_SHORT_1_5_5_5_REV) { RETURN_IF_SUPPORTED(MESA_FORMAT_B5G5R5A1_UNORM); + } else if (type == GL_UNSIGNED_SHORT_5_5_5_1) { + RETURN_IF_SUPPORTED(MESA_FORMAT_B5G5R5A1_UNORM); } else if (type == GL_UNSIGNED_INT_2_10_10_10_REV) { RETURN_IF_SUPPORTED(MESA_FORMAT_B10G10R10A2_UNORM); } |