Hi Julien, On 5 November 2015 at 08:24, Julien Isorce <julien.iso...@gmail.com> wrote: > Before it was only possible to convert a NV12 surface to > RGBA or BGRA. This patch uses the same post processing > function, "handleVAProcPipelineParameterBufferType", but > add definitions for RGBX and BGRX. > > This patch also makes vlVaQuerySurfaceAttributes more generic. > > Signed-off-by: Julien Isorce <j.iso...@samsung.com> > --- > src/gallium/auxiliary/vl/vl_video_buffer.c | 18 +++++++++++++++ > src/gallium/state_trackers/va/picture.c | 5 +++-- > src/gallium/state_trackers/va/surface.c | 36 > +++++++++++++++++------------- > src/gallium/state_trackers/va/va_private.h | 1 + > 4 files changed, 42 insertions(+), 18 deletions(-) > As the diffstat suggests - two different areas, thus this should be different patches.
> @@ -314,7 +319,9 @@ vlVaQuerySurfaceAttributes(VADriverContextP ctx, > VAConfigID config, > vlVaDriver *drv; > VASurfaceAttrib *attribs; > struct pipe_screen *pscreen; > - int i; > + int i, j; > + > + STATIC_ASSERT(ARRAY_SIZE(vpp_surface_formats) <= VL_VA_MAX_IMAGE_FORMATS); > Nice, thank you ! > --- a/src/gallium/state_trackers/va/va_private.h > +++ b/src/gallium/state_trackers/va/va_private.h > @@ -49,6 +49,7 @@ > #define VL_VA_PSCREEN(ctx) (VL_VA_DRIVER(ctx)->vscreen->pscreen) > > #define VL_VA_MAX_IMAGE_FORMATS 9 > +#define VL_VA_MAX_SURFACE_ATTRIBUTES 24 > Unused define ? Cheers, Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
