On Wed, 2016-11-09 at 14:39 -0500, Zhao Yakui wrote: > Signed-off-by: Zhao Yakui <yakui.z...@intel.com> > --- > src/gen75_picture_process.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/src/gen75_picture_process.c > b/src/gen75_picture_process.c > index 0b681f1..95bbcd6 100644 > --- a/src/gen75_picture_process.c > +++ b/src/gen75_picture_process.c > @@ -148,6 +148,12 @@ gen75_proc_picture(VADriverContextP ctx, > goto error; > } > > + if (pipeline_param->num_filters == 0 || pipeline_param->filters > == NULL ){ > + /* explicitly initialize the VPP based on Render ring */ > + if (proc_ctx->vpp_fmt_cvt_ctx == NULL) > + proc_ctx->vpp_fmt_cvt_ctx = i965_proc_context_init(ctx, > NULL); > + } > +
If so, the code to set proc_ctx->vpp_fmt_cvt_ctx in gen75_vpp_fmt_cvt() is unnecessary now, could you remove it? > if (!obj_dst_surf->bo) { > unsigned int is_tiled = 1; > unsigned int fourcc = VA_FOURCC_NV12; _______________________________________________ Libva mailing list Libva@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libva