On 17/01/2019 03:33, Zachary Zhou wrote: > Swap width and height when do clock/cclock rotation > Add reversal/hflip/vflip options > > ffmpeg -hwaccel vaapi -vaapi_device /dev/dri/renderD128 > -hwaccel_output_format vaapi -i input.264 -vf "transpose_vaapi=clock_flip" > -c:v h264_vaapi output.h264 > > Signed-off-by: Zachary Zhou <zachary.z...@intel.com> > --- > configure | 2 + > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/transpose.h | 3 + > libavfilter/vf_transpose_vaapi.c | 316 +++++++++++++++++++++++++++++++ > 5 files changed, 323 insertions(+) > create mode 100644 libavfilter/vf_transpose_vaapi.c > > ... > +static int transpose_vaapi_vpp_config_output(AVFilterLink *outlink) > +{ > + AVFilterContext *avctx = outlink->src; > + VAAPIVPPContext *vpp_ctx = avctx->priv; > + TransposeVAAPIContext *ctx = avctx->priv; > + AVFilterLink *inlink = avctx->inputs[0]; > + > + if ((inlink->w >= inlink->h && ctx->passthrough == > TRANSPOSE_PT_TYPE_LANDSCAPE) || > + (inlink->w <= inlink->h && ctx->passthrough == > TRANSPOSE_PT_TYPE_PORTRAIT)) { > + av_log(avctx, AV_LOG_VERBOSE, > + "w:%d h:%d -> w:%d h:%d (passthrough mode)\n", > + inlink->w, inlink->h, inlink->w, inlink->h); > + return 0; > + }
This case was missing the propagation of the hw_frames_ctx to the next link, which is required for all following processing - I added it. With that change - tested, LGTM, applied. Thanks, - Mark _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel