Re: [FFmpeg-devel] [PATCH 1/4] avcodec/intrax8: Remove unused IDCTDSPContext

2022-08-02 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > configure| 2 +- > libavcodec/intrax8.c | 4 +--- > libavcodec/intrax8.h | 3 +-- > libavcodec/vc1dec.c | 2 +- > libavcodec/wmv2dec.c | 2 +- > 5 files changed, 5 insertions(+), 8 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH v12 00/13] make QSV works with the Intel's oneVPL

2022-08-02 Thread Xiang, Haihao
On Mon, 2022-07-25 at 12:11 +0800, Xiang, Haihao wrote: > From: Haihao Xiang > > The oneAPI Video Processing Library (oneVPL) is a single interface for > encode, decode and video processing[1][2]. oneVPL is a successor to Intel(R) > Media > SDK, but removed obsolete features. Intel(R) Media SDK

Re: [FFmpeg-devel] [PATCH v4 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder

2022-08-02 Thread Xiang, Haihao
On Tue, 2022-08-02 at 13:29 +0200, Anton Khirnov wrote: > Quoting Xiang, Haihao (2022-08-02 09:56:01) > > From: Haihao Xiang > > > > Usually a HW decoder is expected when user specifies a HW acceleration > > method via -hwaccel option, however the current implementation doesn't > > take HW

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 20:16, Thilo Borgmann wrote: On 2 Aug 2022, at 19:34, Nicolas George wrote: Thilo Borgman (12022-08-02): Of course, thought it would already be there together with CC. There’s a typo in the mail address, otherwise LGTM. Indeed, locally fixed: +The TC can be contacted

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:34, Nicolas George wrote: > Thilo Borgman (12022-08-02): >> Of course, thought it would already be there together with CC. >> >> There’s a typo in the mail address, otherwise LGTM. > > Indeed, locally fixed: > > +The TC can be contacted at . > +The CC can be contacted at .

Re: [FFmpeg-devel] [PATCH] avutil/opt: Combine multiple av_log statements

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:28, Andreas Rheinhardt wrote: > Signed-off-by: Andreas Rheinhardt > --- > libavutil/opt.c | 23 --- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/libavutil/opt.c b/libavutil/opt.c > index 8ffb10449b..da3a65c041 100644 > ---

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Jean-Baptiste Kempf
On Tue, 2 Aug 2022, at 19:10, Nicolas George wrote: > Jean-Baptiste Kempf (12022-07-24): >> Have you tried tc@ffmpeg.o.. ? > > Thanks. > > Would this patch be a good idea? Great idea > * 0001-doc-dev_community-add-the-addresses-of-the-committee.patch > * signature.asc -- Jean-Baptiste

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present

2022-08-02 Thread James Almer
On 7/22/2022 8:10 PM, James Almer wrote: On 7/22/2022 8:00 PM, Alex Converse wrote: On Fri, Jul 22, 2022 at 8:37 AM James Almer wrote: On 7/22/2022 12:14 PM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:56 AM, Andreas Rheinhardt wrote: James Almer: On 7/22/2022 11:23 AM, Andreas

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Nicolas George
Thilo Borgman (12022-08-02): > Of course, thought it would already be there together with CC. > > There’s a typo in the mail address, otherwise LGTM. Indeed, locally fixed: +The TC can be contacted at . +The CC can be contacted at . I will wait some time and push if nobody objects. Thanks,

[FFmpeg-devel] [PATCH] avutil/opt: Combine multiple av_log statements

2022-08-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavutil/opt.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index 8ffb10449b..da3a65c041 100644 --- a/libavutil/opt.c +++ b/libavutil/opt.c @@ -1329,17 +1329,18 @@

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:10, Andreas Rheinhardt wrote: Thilo Borgmann: Hi, On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on either the input or output side. Since rotation, flip and

Re: [FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Thilo Borgmann
On 2 Aug 2022, at 19:10, Nicolas George wrote: > Jean-Baptiste Kempf (12022-07-24): >> Have you tried tc@ffmpeg.o.. ? > > Thanks. > > Would this patch be a good idea? Of course, thought it would already be there together with CC. There’s a typo in the mail address, otherwise LGTM. -Thilo

[FFmpeg-devel] [PATCH] Document the addresses of the committees (was: Contact of the Technical Committee)

2022-08-02 Thread Nicolas George
Jean-Baptiste Kempf (12022-07-24): > Have you tried tc@ffmpeg.o.. ? Thanks. Would this patch be a good idea? Regards, -- Nicolas George From e1912c02efa25c7e150e7b4984144ef50c6a Mon Sep 17 00:00:00 2001 From: Nicolas George Date: Tue, 2 Aug 2022 19:07:05 +0200 Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Andreas Rheinhardt
Thilo Borgmann: > Hi, > >> On 2022-05-18 11:34 pm, Jan Ekström wrote: >>> This enables overriding the rotation as well as horizontal/vertical >>> flip state of a specific video stream on either the input or output >>> side. >> >> Since rotation, flip and scale are stored

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: Rework the AVIF parser to handle multiple items

2022-08-02 Thread James Zern
Andreas, On Thu, Jul 28, 2022 at 11:25 AM Vignesh Venkatasubramanian wrote: > > Stores the item ids of all the items found in the file and > processes the primary item at the end of the meta box. This patch > does not change any behavior. It sets up the code for parsing > alpha channel (and

[FFmpeg-devel] [PATCH 7/7] tests: add coverage for libavfilter's format negotiation

2022-08-02 Thread Nicolas George
Added coverage for the video part of pick_format(). Signed-off-by: Nicolas George --- Only video for now, I would like to push before working on audio. Audio is easier because there are fewer formats. But audio is also harder because I need to test cases where format incompatibilities detected

[FFmpeg-devel] [PATCH 6/7] lavi/pixdesc: add comments about pixel format scoring

2022-08-02 Thread Nicolas George
Signed-off-by: Nicolas George --- libavutil/pixdesc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 6e57a82cb6..923a61b0ab 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -2866,6 +2866,7 @@ static int

[FFmpeg-devel] [PATCH 5/7] lavfi/(a)format: support slash as a delimiter

2022-08-02 Thread Nicolas George
Unlike pipe, slash does not require escaping. Signed-off-by: Nicolas George --- doc/filters.texi | 8 libavfilter/internal.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) Identical to the one posted months ago. diff --git a/doc/filters.texi b/doc/filters.texi index

[FFmpeg-devel] [PATCH 4/7] lavfi/(a)format: factor finding the delimiter

2022-08-02 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/af_aformat.c | 8 +--- libavfilter/internal.h | 9 + libavfilter/vf_format.c | 9 - 3 files changed, 18 insertions(+), 8 deletions(-) Identical to the one posted months ago. diff --git a/libavfilter/af_aformat.c

[FFmpeg-devel] [PATCH 1/7] lavu/pixfmt: summarize yuv naming conventions

2022-08-02 Thread Nicolas George
Signed-off-by: Nicolas George --- libavutil/pixfmt.h | 17 + 1 file changed, 17 insertions(+) Identical to the one I posted last week. diff --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h index 2d3927cc3f..3d0f92ed4d 100644 --- a/libavutil/pixfmt.h +++ b/libavutil/pixfmt.h @@

[FFmpeg-devel] [PATCH 2/7] lavfi/graphdump: add plain listing output

2022-08-02 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/avfilter.h | 5 ++- libavfilter/graphdump.c | 79 +++-- 2 files changed, 80 insertions(+), 4 deletions(-) Identical to the one posted months ago. diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h

[FFmpeg-devel] [PATCH 3/7] fftools: add -lavfi_dump option

2022-08-02 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/ffmpeg.texi | 4 fftools/ffmpeg.h| 1 + fftools/ffmpeg_filter.c | 14 ++ fftools/ffmpeg_opt.c| 3 +++ 4 files changed, 22 insertions(+) Identical to the one posted months ago. diff --git a/doc/ffmpeg.texi

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
Am 02.08.22 um 18:48 schrieb Thilo Borgmann: Hi, On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on either the input or output side. Since rotation, flip and scale are stored in a single

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add display_{rotation, hflip, vflip} options

2022-08-02 Thread Thilo Borgmann
Hi, On 2022-05-18 11:34 pm, Jan Ekström wrote: This enables overriding the rotation as well as horizontal/vertical flip state of a specific video stream on either the input or output side. Since rotation, flip and scale are stored in a single data structure, how about a single option i.e.

Re: [FFmpeg-devel] [PATCH] avcodec/pthread_frame: update the main avctx from the current, ThreadContext

2022-08-02 Thread Anton Khirnov
Why are you not resubmitting your original patch that stops copying hwaccel_priv_data to the user-facing context? It seemed more correct to me, since the user-facing context should never see any hwaccel data. Or does it not fix the issue fully? For a more proper fix, we probably want to bundle

Re: [FFmpeg-devel] [PATCH] avcodec/libaomenc: support AV_CODEC_CAP_ENCODER_RECON_FRAME

2022-08-02 Thread James Almer
On 8/2/2022 8:58 AM, Anton Khirnov wrote: Quoting James Almer (2022-08-02 13:35:19) Signed-off-by: James Almer --- libavcodec/Makefile| 4 +- libavcodec/libaom.c| 49 + libavcodec/libaom.h| 33 ++ libavcodec/libaomdec.c | 27 +---

Re: [FFmpeg-devel] [PATCH] swscale: add NV16 input/output

2022-08-02 Thread Matthieu Bouron
On Thu, Jul 14, 2022 at 8:10 PM Michael Niedermayer wrote: > On Fri, Jul 01, 2022 at 10:08:14AM +0200, Matthieu Bouron wrote: > > On Sat, Jun 11, 2022 at 5:35 PM Michael Niedermayer < > mich...@niedermayer.cc> > > wrote: > > > > > On Fri, Jun 10, 2022 at 04:11:10PM +0200, Matthieu Bouron wrote:

Re: [FFmpeg-devel] [PATCH v3 1/2] ffmpeg: refactor post-decoding steps for subtitles into a function

2022-08-02 Thread Anton Khirnov
Quoting Jan Ekström (2022-07-29 15:35:36) > From: Jan Ekström > > This enables us to later call this when generating additional > subtitles for splitting purposes. > > Co-authored-by: Andrzej Nadachowski > > Signed-off-by: Jan Ekström > --- > fftools/ffmpeg.c | 50

Re: [FFmpeg-devel] [PATCH] avcodec/libaomenc: support AV_CODEC_CAP_ENCODER_RECON_FRAME

2022-08-02 Thread Anton Khirnov
Quoting James Almer (2022-08-02 13:35:19) > Signed-off-by: James Almer > --- > libavcodec/Makefile| 4 +- > libavcodec/libaom.c| 49 + > libavcodec/libaom.h| 33 ++ > libavcodec/libaomdec.c | 27 +--- > libavcodec/libaomenc.c | 97

[FFmpeg-devel] [PATCH] avcodec/libaomenc: support AV_CODEC_CAP_ENCODER_RECON_FRAME

2022-08-02 Thread James Almer
Signed-off-by: James Almer --- libavcodec/Makefile| 4 +- libavcodec/libaom.c| 49 + libavcodec/libaom.h| 33 ++ libavcodec/libaomdec.c | 27 +--- libavcodec/libaomenc.c | 97 ++ 5 files changed, 183

Re: [FFmpeg-devel] [PATCH v4 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder

2022-08-02 Thread Anton Khirnov
Quoting Xiang, Haihao (2022-08-02 09:56:01) > From: Haihao Xiang > > Usually a HW decoder is expected when user specifies a HW acceleration > method via -hwaccel option, however the current implementation doesn't > take HW acceleration method into account, it is possible to select a SW >

Re: [FFmpeg-devel] [PATCH v12 11/13] lavc/qsv: create mfx session using oneVPL for decoding/encoding

2022-08-02 Thread Anton Khirnov
Quoting Xiang, Haihao (2022-08-02 10:20:16) > Thanks for the comment. I planed to cleanup qsv code, including duplicated > code > in the current qsv filters. Does it make sense to enable/merge oneVPL firstly > then cleanup qsv in new patches? Sure, I do not insist it's done right now, given that

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: print a log message when treating mono HE-AAC as stereo

2022-08-02 Thread Anton Khirnov
Quoting James Almer (2022-07-28 19:22:47) > Since this behavior is intentional, use the VERBOSE level instead of WARNING > as > it's nothing the user should worry about. > > Signed-off-by: James Almer > --- > libavcodec/aac.h | 1 + > libavcodec/aacdec_template.c | 4 >

Re: [FFmpeg-devel] [PATCH] avdevice/v4l2: fix leak of timefilter

2022-08-02 Thread Anton Khirnov
Quoting Zhao Zhili (2022-07-26 18:32:51) > From: Zhao Zhili > > Fixes ticket #9844. > --- > I don't have a camera for test. So I'm not sure if it's the only case > of memleak in ticket #9844. > > libavdevice/v4l2.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavdevice/v4l2.c

Re: [FFmpeg-devel] [PATCH v3 1/2] ffmpeg: refactor post-decoding steps for subtitles into a function

2022-08-02 Thread Anton Khirnov
Quoting Jan Ekström (2022-07-29 15:35:36) > From: Jan Ekström > > This enables us to later call this when generating additional > subtitles for splitting purposes. > > Co-authored-by: Andrzej Nadachowski > > Signed-off-by: Jan Ekström > --- > fftools/ffmpeg.c | 50

Re: [FFmpeg-devel] [PATCH 34/39] avcodec/mpegvideoencdsp: Allow pointers to const where possible

2022-08-02 Thread Anton Khirnov
Quoting Michael Niedermayer (2022-07-30 19:12:39) > On Sat, Jul 30, 2022 at 11:55:45AM +0200, Andreas Rheinhardt wrote: > > Michael Niedermayer: > > > On Wed, Jul 27, 2022 at 12:08:09AM +0200, Andreas Rheinhardt wrote: > > >> Signed-off-by: Andreas Rheinhardt > > >> --- > > >>

Re: [FFmpeg-devel] [PATCH v4 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder

2022-08-02 Thread Anton Khirnov
Both patches LGTM -- Anton Khirnov ___ 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] Encoding image

2022-08-02 Thread Marco Vianini
I have a PNG image converted to YUV 420P and I need to encode it to a specific RTMPS URI, with fps 10. The encoding process is quite heavy for the CPU.  In this specific case the frame is always the same, re-encoded at a rate basing on fps. Is it necessary to call "avcodec_send_frame /

Re: [FFmpeg-devel] [PATCH v12 11/13] lavc/qsv: create mfx session using oneVPL for decoding/encoding

2022-08-02 Thread Xiang, Haihao
On Mon, 2022-08-01 at 15:33 +0200, Anton Khirnov wrote: > Quoting Xiang, Haihao (2022-07-25 06:11:49) > > From: Haihao Xiang > > > > If qsv hwdevice is available, use the mfxLoader handle in qsv hwdevice > > to create mfx session. Otherwise create mfx session with a new mfxLoader > > handle. > >

Re: [FFmpeg-devel] [PATCH v3 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder

2022-08-02 Thread Xiang, Haihao
On Mon, 2022-08-01 at 11:12 +0200, Anton Khirnov wrote: > Hi, > the concept is generally ok, but I have a few comments on the > implementation. > > Quoting Xiang, Haihao (2022-07-25 06:30:51) > > +static const AVCodec *choose_decoder2(InputStream *ist, OptionsContext *o, > > AVFormatContext *s,

[FFmpeg-devel] [PATCH v4 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder

2022-08-02 Thread Xiang, Haihao
From: Haihao Xiang Usually a HW decoder is expected when user specifies a HW acceleration method via -hwaccel option, however the current implementation doesn't take HW acceleration method into account, it is possible to select a SW decoder. For example: $ ffmpeg -hwaccel vaapi -i av1.mp4 -f

[FFmpeg-devel] [PATCH v4 1/2] ffmpeg_opt: select a decoder after getting values for per-stream hwdec options

2022-08-02 Thread Xiang, Haihao
From: Haihao Xiang After applying this patch, the desired HW acceleration method is known before selecting decoder, so we may take HW acceleration method into account when selecting decoder for input stream in the next commit There should be no functional changes in this patch Signed-off-by:

[FFmpeg-devel] [PATCH] avcodec/acelp_*: Remove unnecessary headers

2022-08-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/acelp_filters.c | 2 +- libavcodec/acelp_pitch_delay.c | 2 -- libavcodec/acelp_vectors.c | 4 ++-- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/libavcodec/acelp_filters.c b/libavcodec/acelp_filters.c index