Re: [FFmpeg-devel] [PATCH v7 05/10] qsv: build audio related code when MFX_VERSION < 2.0

["Xiang, Haihao" <haihao.xiang-at-intel.com@ffmpeg.org>]

On Sat, 2022-04-30 at 16:51 +0000, Soft Works wrote:
> > -----Original Message-----
> > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> > Xiang, Haihao
> > Sent: Wednesday, April 6, 2022 5:38 AM
> > To: anton@khirnov.net; ffmpeg-devel@ffmpeg.org
> > Subject: Re: [FFmpeg-devel] [PATCH v7 05/10] qsv: build audio related
> > code when MFX_VERSION < 2.0
> > 
> > On Tue, 2022-04-05 at 13:50 +0200, Anton Khirnov wrote:
> > > Quoting Xiang, Haihao (2022-03-11 09:16:25)
> > > > From: Haihao Xiang <haihao.xiang@intel.com>
> > > > 
> > > > Audio isn't supported for MFX_VERSION >= 2.0[1][2]. This is in
> > > > preparation for oneVPL support
> > > > 
> > > > [1]:
> > > > 
> > 
> > https://spec.oneapi.io/versions/latest/elements/oneVPL/source/VPL_inte
> > l_media_sdk.html#msdk-full-name-feature-removals
> > > > [2]: https://github.com/oneapi-src/oneVPL
> > > > ---
> > > >  libavcodec/qsv.c     | 5 +++++
> > > >  libavfilter/qsvvpp.c | 6 ++++++
> > > >  libavfilter/qsvvpp.h | 2 ++
> > > >  3 files changed, 13 insertions(+)
> > > 
> > > Why not just remove this completely?
> > > None of our QSV code  does anything with audio.
> > 
> > It was removed in an older version, however someone objected the
> > removal of
> > this.  See
> > https://patchwork.ffmpeg.org/project/ffmpeg/patch/20200819061023.44330
> > 4-2-haihao.xiang@intel.com/
> 
> I think this was a misunderstanding. I see not objection. One was 
> just asking "why" and the other one had missed the point that audio
> has never been functional.

Please find the comment below in the original thread

"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"

So my understanding was that the reviewer preferred to keep the audio stuff
unchanged for libmfx.

> 
> I agree with Anton that the audio stuff should be removed 
> completely - ideally even beforehand in a separate patch.

Actually I prefer to remove the audio stuff too. I may submit a separate patch
if no more comments.

Thanks
Haihao

_______________________________________________
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".