[Mesa-dev] [PATCH 2/2] vl/buffers: fixes vl_video_buffer_formats for RGBX

Emil Velikov emil.l.velikov at gmail.com
Sun Nov 29 08:56:34 PST 2015


On 27 November 2015 at 08:57, Julien Isorce <j.isorce at samsung.com> wrote:
> Signed-off-by: Julien Isorce <j.isorce at samsung.com>
> ---
>  src/gallium/auxiliary/vl/vl_video_buffer.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/vl/vl_video_buffer.c b/src/gallium/auxiliary/vl/vl_video_buffer.c
> index 62f4aa9..b3f6ff5 100644
> --- a/src/gallium/auxiliary/vl/vl_video_buffer.c
> +++ b/src/gallium/auxiliary/vl/vl_video_buffer.c
> @@ -115,7 +115,7 @@ vl_video_buffer_formats(struct pipe_screen *screen, enum pipe_format format)
>        return const_resource_formats_VUYA;
>
>     case PIPE_FORMAT_R8G8B8X8_UNORM:
> -      return const_resource_formats_VUYX;
> +      return const_resource_formats_YUVX;
>
>     case PIPE_FORMAT_B8G8R8X8_UNORM:
>        return const_resource_formats_VUYX;
Oops copy/pasta mistake :-)

Please add the following before pushing

Fixes: 42a5e143a8d "vl/buffers: add RGBX and BGRX to the supported formats"
Cc: mesa-stable at lists.freedesktop.org
Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>

Next time when you're around please make the const_resource_formats_*
arrays static.

Thanks
Emil


More information about the mesa-dev mailing list