<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 5 November 2015 at 16:27, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Julien,<br>
<span class=""><br>
On 5 November 2015 at 08:24, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>> wrote:<br>
> Before it was only possible to convert a NV12 surface to<br>
> RGBA or BGRA. This patch uses the same post processing<br>
> function, "handleVAProcPipelineParameterBufferType", but<br>
> add definitions for RGBX and BGRX.<br>
><br>
> This patch also makes vlVaQuerySurfaceAttributes more generic.<br>
><br>
> Signed-off-by: Julien Isorce <<a href="mailto:j.isorce@samsung.com">j.isorce@samsung.com</a>><br>
> ---<br>
>  src/gallium/auxiliary/vl/vl_video_buffer.c | 18 +++++++++++++++<br>
>  src/gallium/state_trackers/va/picture.c    |  5 +++--<br>
>  src/gallium/state_trackers/va/surface.c    | 36 +++++++++++++++++-------------<br>
>  src/gallium/state_trackers/va/va_private.h |  1 +<br>
>  4 files changed, 42 insertions(+), 18 deletions(-)<br>
><br>
</span>As the diffstat suggests - two different areas, thus this should be<br>
different patches.<br>
<span class=""><br></span></blockquote><div>Ok I'll split it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
> @@ -314,7 +319,9 @@ vlVaQuerySurfaceAttributes(VADriverContextP ctx, VAConfigID config,<br>
>     vlVaDriver *drv;<br>
>     VASurfaceAttrib *attribs;<br>
>     struct pipe_screen *pscreen;<br>
> -   int i;<br>
> +   int i, j;<br>
> +<br>
> +   STATIC_ASSERT(ARRAY_SIZE(vpp_surface_formats) <= VL_VA_MAX_IMAGE_FORMATS);<br>
><br>
</span>Nice, thank you !<br></blockquote><div><br></div><div>no pb :)<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
<br>
> --- a/src/gallium/state_trackers/va/va_private.h<br>
> +++ b/src/gallium/state_trackers/va/va_private.h<br>
> @@ -49,6 +49,7 @@<br>
>  #define VL_VA_PSCREEN(ctx) (VL_VA_DRIVER(ctx)->vscreen->pscreen)<br>
><br>
>  #define VL_VA_MAX_IMAGE_FORMATS 9<br>
> +#define VL_VA_MAX_SURFACE_ATTRIBUTES 24<br>
><br>
</span>Unused define ?<br></blockquote><div><br></div><div>Thx I'll remove this residual from a previous attempt.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Emil<br>
</blockquote></div><br></div></div>