> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of Linjie Fu > Sent: Wednesday, September 11, 2019 12:09 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie <linjie...@intel.com> > Subject: [FFmpeg-devel] [PATCH] qsv: get FrameInfo.Shift by > desc->comp[0].shift > > Since Y410 is a pixel format with depth > 8 but shift = 0, get Shift info by > depth is > not quite accurate. > > Signed-off-by: Linjie Fu <linjie...@intel.com> > --- > libavcodec/qsvenc.c | 4 ++-- > libavutil/hwcontext_qsv.c | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 9bf8574..912f2a8 > 100644 > --- a/libavcodec/qsvenc.c > +++ b/libavcodec/qsvenc.c > @@ -437,7 +437,7 @@ static int init_video_param_jpeg(AVCodecContext > *avctx, QSVEncContext *q) > q->param.mfx.FrameInfo.ChromaFormat = MFX_CHROMAFORMAT_YUV420; > q->param.mfx.FrameInfo.BitDepthLuma = desc->comp[0].depth; > q->param.mfx.FrameInfo.BitDepthChroma = desc->comp[0].depth; > - q->param.mfx.FrameInfo.Shift = desc->comp[0].depth > 8; > + q->param.mfx.FrameInfo.Shift = desc->comp[0].shift > 0;
Is it safe enough? As MSDK docs: "Not all codecs and SDK implementations support this value. Use Query function to check if this feature is supported." _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
