st/mesa: Prefer 5551 formats for GL_UNSIGNED_SHORT_5_5_5_1.

Previously, internalformat GL_RGBA and type GL_UNSIGNED_SHORT_5_5_5_1
was promoted to RGBA8888 as the table entry with the 5551 formats
is listed below the 8888 entry, and it also doesn't have GL_RGBA as
a possible internalformat.

Using actual 5551 fixes the following dEQP-EGL test:
- dEQP-EGL.functional.image.modify.tex_rgb5_a1_tex_subimage_rgba8

Reviewed-by: Eric Anholt <eric@anholt.net>
This commit is contained in:
Kenneth Graunke 2019-09-12 17:30:28 -07:00
parent ffabcbba60
commit fbad42bbb9
1 changed files with 7 additions and 0 deletions

View File

@ -2266,6 +2266,13 @@ st_choose_format(struct st_context *st, GLenum internalFormat,
internalFormat = GL_RGB10_A2;
}
if (type == GL_UNSIGNED_SHORT_5_5_5_1) {
if (internalFormat == GL_RGB)
internalFormat = GL_RGB5;
else if (internalFormat == GL_RGBA)
internalFormat = GL_RGB5_A1;
}
/* search table for internalFormat */
for (i = 0; i < ARRAY_SIZE(format_map); i++) {
const struct format_mapping *mapping = &format_map[i];