<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 November 2015 at 12:03, 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"><span class="">On 26 November 2015 at 08:45, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>> wrote:<br>
<br>
> --- a/src/gallium/state_trackers/va/picture.c<br>
> +++ b/src/gallium/state_trackers/va/picture.c<br>
<br>
</span><span class="">> @@ -113,12 +118,37 @@ handlePictureParameterBuffer(vlVaDriver *drv, vlVaContext *context, vlVaBuffer *<br>
>     default:<br>
>        break;<br>
>     }<br>
> +<br>
> +   /* Create the decoder once max_references is known. */<br>
> +   if (!context->decoder) {<br>
> +      if (!context->target)<br>
> +         return VA_STATUS_ERROR_INVALID_CONTEXT;<br>
> +<br>
> +      if (context->templat.max_references == 0)<br>
> +         return VA_STATUS_ERROR_INVALID_BUFFER;<br>
> +<br>
> +      if (u_reduce_video_profile(context->templat.profile) !=<br>
> +          PIPE_VIDEO_FORMAT_MPEG4_AVC)<br>
> +         context->templat.level = u_get_h264_level(context->templat.width,<br>
> +            context->templat.height, &context->templat.max_references);<br>
<br>
</span>Erm shouldn't this one be "if profile == h264 { get_h264_level(...) }" ?<br></blockquote><div> </div><div>Indeed :) Thx Emil.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
With the above fixed<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
-Emil<br>
</font></span></blockquote></div><br></div></div>