| Message ID | 1569317341-17873-1-git-send-email-linjie.fu@intel.com |
|---|---|
| State | New |
| Headers | show |
Please ignore this patch, will send a new one. > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Linjie Fu > Sent: Tuesday, September 24, 2019 17:29 > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie <linjie.fu@intel.com> > Subject: [FFmpeg-devel] [PATCH] lavc/vaapi_encode: remove consistency > check of depth for each component > > Previously, consistency check of depth is introduced to make sure > each component of the pixel format has the same depth. > > Since Y410(with a different depth in alpha channel) is supported for > ICL+, this check should be removed to make the encoder work. > > Signed-off-by: Linjie Fu <linjie.fu@intel.com> > --- > libavcodec/vaapi_encode.c | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c > index cb598d5..ba7a432 100644 > --- a/libavcodec/vaapi_encode.c > +++ b/libavcodec/vaapi_encode.c > @@ -1236,14 +1236,6 @@ static av_cold int > vaapi_encode_profile_entrypoint(AVCodecContext *avctx) > ctx->input_frames->sw_format); > return AVERROR(EINVAL); > } > - depth = desc->comp[0].depth; > - for (i = 1; i < desc->nb_components; i++) { > - if (desc->comp[i].depth != depth) { > - av_log(avctx, AV_LOG_ERROR, "Invalid input pixfmt (%s).\n", > - desc->name); > - return AVERROR(EINVAL); > - } > - } > av_log(avctx, AV_LOG_VERBOSE, "Input surface format is %s.\n", > desc->name); > > -- > 2.7.4 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c index cb598d5..ba7a432 100644 --- a/libavcodec/vaapi_encode.c +++ b/libavcodec/vaapi_encode.c @@ -1236,14 +1236,6 @@ static av_cold int vaapi_encode_profile_entrypoint(AVCodecContext *avctx) ctx->input_frames->sw_format); return AVERROR(EINVAL); } - depth = desc->comp[0].depth; - for (i = 1; i < desc->nb_components; i++) { - if (desc->comp[i].depth != depth) { - av_log(avctx, AV_LOG_ERROR, "Invalid input pixfmt (%s).\n", - desc->name); - return AVERROR(EINVAL); - } - } av_log(avctx, AV_LOG_VERBOSE, "Input surface format is %s.\n", desc->name);
Previously, consistency check of depth is introduced to make sure each component of the pixel format has the same depth. Since Y410(with a different depth in alpha channel) is supported for ICL+, this check should be removed to make the encoder work. Signed-off-by: Linjie Fu <linjie.fu@intel.com> --- libavcodec/vaapi_encode.c | 8 -------- 1 file changed, 8 deletions(-)