<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Yeah, agree reviewing them on the mesa
list is probably more appropriate.<br>
<br>
Anyway patch lgtm as well and is Reviewed-by: Christian König
<a class="moz-txt-link-rfc2396E" href="mailto:christian.koenig@amd.com"><christian.koenig@amd.com></a><br>
<br>
Regards,<br>
Christian.<br>
<br>
On 15.12.2015 18:53, Julien Isorce wrote:<br>
</div>
<blockquote
cite="mid:CAHWPjbWg0S74CnHOAAYVypnFM=Vh0KGfWdn3s3SCHckQJEgQug@mail.gmail.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<div dir="ltr">
<div>+ mesa-dev of course :)<br>
<br>
</div>
Thx Sean<br>
<div>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On 15 December 2015 at 17:46,
Sean V Kelley <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:seanvk@posteo.de" target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:seanvk@posteo.de">seanvk@posteo.de</a></a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex"><span
class=""><br>
> On Dec 15, 2015, at 1:01 AM, Julien Isorce <<a
moz-do-not-send="true"
href="mailto:julien.isorce@gmail.com"><a class="moz-txt-link-abbreviated" href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a></a>>
wrote:<br>
><br>
> "image" is not ready yet since it will be set
at<br>
> the end of the function by: *image = *img;<br>
><br>
> Signed-off-by: Julien Isorce <<a
moz-do-not-send="true"
href="mailto:j.isorce@samsung.com"><a class="moz-txt-link-abbreviated" href="mailto:j.isorce@samsung.com">j.isorce@samsung.com</a></a>><br>
> ---<br>
> src/gallium/state_trackers/va/image.c | 2 +-<br>
> 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
<br>
</span>lgtm but you should also add mesa list for this
particular patch…<br>
<br>
Sean<br>
<span class=""><br>
><br>
> diff --git
a/src/gallium/state_trackers/va/image.c
b/src/gallium/state_trackers/va/image.c<br>
> index 58c9ff7..ccc263f 100644<br>
> --- a/src/gallium/state_trackers/va/image.c<br>
> +++ b/src/gallium/state_trackers/va/image.c<br>
> @@ -264,7 +264,7 @@
vlVaDeriveImage(VADriverContextP ctx, VASurfaceID
surface, VAImage *image)<br>
> img->image_id =
handle_table_add(drv->htab, img);<br>
><br>
> img_buf->type = VAImageBufferType;<br>
> - img_buf->size = image->data_size;<br>
> + img_buf->size = img->data_size;<br>
> img_buf->num_elements = 1;<br>
><br>
>
pipe_resource_reference(&img_buf->derived_surface.resource,
surfaces[0]->texture);<br>
> --<br>
> 1.9.1<br>
><br>
</span>>
_______________________________________________<br>
> Libva mailing list<br>
> <a moz-do-not-send="true"
href="mailto:Libva@lists.freedesktop.org">Libva@lists.freedesktop.org</a><br>
> <a moz-do-not-send="true"
href="http://lists.freedesktop.org/mailman/listinfo/libva"
rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libva</a><br>
<br>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
</blockquote>
<br>
</body>
</html>