[Mesa-dev] [PATCH 09/22] mesa: Add GL_RGBA + GL_UNSIGNED_INT_2_10_10_10_REV for OES read type.

[Marek Olšák]

Do we need to handle B10G10R10X2 as well?

Marek

On Wed, Nov 29, 2017 at 5:20 AM, Mario Kleiner
<mario.kleiner.de at gmail.com> wrote:
> This format + type combo is good for BGRA1010102 framebuffers
> for use with glReadPixels() under GLES, so add it for the
> GL_IMPLEMENTATION_COLOR_READ_TYPE_OES query.
>
> Allows successful testing of 10 bpc / depth 30 rendering with dEQP test
> case dEQP-EGL.functional.wide_color.window_1010102_colorspace_default.
>
> Signed-off-by: Mario Kleiner <mario.kleiner.de at gmail.com>
> ---
>  src/mesa/main/framebuffer.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/main/framebuffer.c b/src/mesa/main/framebuffer.c
> index b17d7cb..a0de669 100644
> --- a/src/mesa/main/framebuffer.c
> +++ b/src/mesa/main/framebuffer.c
> @@ -889,6 +889,9 @@ _mesa_get_color_read_type(struct gl_context *ctx,
>        if (format == MESA_FORMAT_B5G6R5_UNORM)
>           return GL_UNSIGNED_SHORT_5_6_5;
>
> +      if (format == MESA_FORMAT_B10G10R10A2_UNORM)
> +         return GL_UNSIGNED_INT_2_10_10_10_REV;
> +
>        switch (data_type) {
>        case GL_SIGNED_NORMALIZED:
>           return GL_BYTE;
> --
> 2.7.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev