[Mesa-dev] [PATCH 0/4] st/va: indent, leak fix and RGBx/BGRx for vpp

[Julien Isorce]

On 5 November 2015 at 14:22, Christian König <christian.koenig at amd.com>
wrote:

> Patches #1-#3 are Reviewed-by: Christian König <christian.koenig at amd.com>
>
> Please split patch #4 in two with first adding the new vl_video_buffer
> formats and then using them in the state tracker.
>

Makes sense, I'll split it.


>
> Should I commit that for you as well or do you now have an account?
>

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.

Thx
Julien


>
> Regards,
> Christian.
>
>
> On 05.11.2015 09:24, Julien Isorce wrote:
>
>> A few fixes and small improvement from recent changes in st/va:
>> * Indentation was wrong for some functions.
>> * Coverity reported a memory leak. (thx to Ilia for pointing this)
>> * More color formats convertion for Video Post Processing
>>
>> Julien Isorce (4):
>>    st/va: indent vlVaQuerySurfaceAttributes and vlVaCreateSurfaces2
>>    st/va: fix memory leak on error in vlVaCreateSurfaces2
>>    st/va: properly indent buffer.c, config.c, image.c and picture.c
>>    st/va: add support for RGBX and BGRX in VPP
>>
>>   src/gallium/auxiliary/vl/vl_video_buffer.c |  18 +
>>   src/gallium/state_trackers/va/buffer.c     |  23 +-
>>   src/gallium/state_trackers/va/config.c     |  12 +-
>>   src/gallium/state_trackers/va/image.c      |   4 +-
>>   src/gallium/state_trackers/va/picture.c    |  87 ++---
>>   src/gallium/state_trackers/va/surface.c    | 602
>> +++++++++++++++--------------
>>   src/gallium/state_trackers/va/va_private.h |   1 +
>>   7 files changed, 388 insertions(+), 359 deletions(-)
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20151105/e0d353fb/attachment.html>