Patches #1-#3 are Reviewed-by: Christian König <christian.koe...@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.

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

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(-)


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to