<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 5 November 2015 at 14:22, Christian König <span dir="ltr"><<a href="mailto:christian.koenig@amd.com" target="_blank">christian.koenig@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Patches #1-#3 are Reviewed-by: Christian König <<a href="mailto:christian.koenig@amd.com" target="_blank">christian.koenig@amd.com</a>><br>
<br>
Please split patch #4 in two with first adding the new vl_video_buffer formats and then using them in the state tracker.<br></blockquote><div><br>Makes sense, I'll split it.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Should I commit that for you as well or do you now have an account?<br></blockquote><div><br></div><div>I requested the account few minutes ago but please push #1 and #2. Tomorrow I'll send a v2 about #3 and #4 after addressing remarks + split.<br></div><div><br></div><div>Thx<br></div><div>Julien<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
Christian.<div class=""><div class="h5"><br>
<br>
On 05.11.2015 09:24, Julien Isorce wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
A few fixes and small improvement from recent changes in st/va:<br>
* Indentation was wrong for some functions.<br>
* Coverity reported a memory leak. (thx to Ilia for pointing this)<br>
* More color formats convertion for Video Post Processing<br>
<br>
Julien Isorce (4):<br>
st/va: indent vlVaQuerySurfaceAttributes and vlVaCreateSurfaces2<br>
st/va: fix memory leak on error in vlVaCreateSurfaces2<br>
st/va: properly indent buffer.c, config.c, image.c and picture.c<br>
st/va: add support for RGBX and BGRX in VPP<br>
<br>
src/gallium/auxiliary/vl/vl_video_buffer.c | 18 +<br>
src/gallium/state_trackers/va/buffer.c | 23 +-<br>
src/gallium/state_trackers/va/config.c | 12 +-<br>
src/gallium/state_trackers/va/image.c | 4 +-<br>
src/gallium/state_trackers/va/picture.c | 87 ++---<br>
src/gallium/state_trackers/va/surface.c | 602 +++++++++++++++--------------<br>
src/gallium/state_trackers/va/va_private.h | 1 +<br>
7 files changed, 388 insertions(+), 359 deletions(-)<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>