On Fri, 2020-08-21 at 09:47 +0200, Thilo Borgmann wrote:
> >
> >
> > On Thu, 2020-08-20 at 02:26 +0200, Thilo Borgmann wrote:
> > > Am 20.08.20 um 02:13 schrieb Xiang, Haihao:
> > > > On Wed, 2020-08-19 at 13:53 +0200, Nicolas George wrote:
> > > > > Haihao Xiang (12020-08-19):
> > > > > > ---
> > > > > > libavcodec/qsv.c | 3 ---
> > > > > > 1 file changed, 3 deletions(-)
> > > > > >
> > > > > > diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c
> > > > > > index faa424bb68..6776b745b9 100644
> > > > > > --- a/libavcodec/qsv.c
> > > > > > +++ b/libavcodec/qsv.c
> > > > > > @@ -143,8 +143,6 @@ static const struct {
> > > > > > { MFX_ERR_INVALID_VIDEO_PARAM, AVERROR(EINVAL), "invalid
> > > > > > video
> > > > > > parameters" },
> > > > > > { MFX_ERR_UNDEFINED_BEHAVIOR, AVERROR_BUG, "undefined
> > > > > > behavior" },
> > > > > > { MFX_ERR_DEVICE_FAILED, AVERROR(EIO), "device
> > > > > > failed" },
> > > > > > - { MFX_ERR_INCOMPATIBLE_AUDIO_PARAM, AVERROR(EINVAL),
> > > > > > "incompatible
> > > > > > audio parameters" },
> > > > > > - { MFX_ERR_INVALID_AUDIO_PARAM, AVERROR(EINVAL), "invalid
> > > > > > audio
> > > > > > parameters" },
> > > > > >
> > > > > > { MFX_WRN_IN_EXECUTION, 0, "operation
> > > > > > in
> > > > > > execution" },
> > > > > > { MFX_WRN_DEVICE_BUSY, 0, "device
> > > > > > busy" },
> > > > > > @@ -154,7 +152,6 @@ static const struct {
> > > > > > { MFX_WRN_VALUE_NOT_CHANGED, 0, "value is
> > > > > > saturated" },
> > > > > > { MFX_WRN_OUT_OF_RANGE, 0, "value out
> > > > > > of
> > > > > > range" },
> > > > > > { MFX_WRN_FILTER_SKIPPED, 0, "filter
> > > > > > skipped" },
> > > > > > - { MFX_WRN_INCOMPATIBLE_AUDIO_PARAM,
> > > > > > 0, "incompatible
> > > > > > audio parameters" },
> > > > > > };
> > > > > >
> > > > > > int ff_qsv_map_error(mfxStatus mfx_err, const char **desc)
> > > > >
> > > > > Why?
> > > >
> > > > qsv in FFmpeg is for video only, actually the SDK is for HW accelerated
> > > > video
> > > > decode, encode and filtering now, see
> > > > https://github.com/Intel-Media-SDK/MediaSDK#intel-media-sdk.
> > >
> > > This seems like a generic translation from the library errors to FF error
> > > codes. So even if we'll never touch the audio functionality of it, I'd
> > > prefer
> > > to have that struct complete already.
> > >
> >
> > In general, I agree with you. But the SDK no longer support audio and the
> > corresponding code might be deprecated or removed in the future, so I'd
> > prefer
> > to do a clean up (of course we may add some #ifdef.. #endif)
>
> Hm it's ugly yet used in other places in qsv.c already to do the same. So you
> should also make use of it there IMHO.
>
OK, please ignore this patch, I will submit a new patch once these error codes
are removed or deprecated.
Thanks
Haihao
>
> > BTW did you not reply the mailing list? I didn't see your comment in the
> > patchwork (
> >
https://patchwork.ffmpeg.org/project/ffmpeg/patch/20200819061023.443304-2-haihao.xi...@intel.com/
> > )
>
> Oh sorry I messed that up, so I included the short history now, thx!
>
> -Thilo
>
> _______________________________________________
> 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".
_______________________________________________
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".
