[FFmpeg-devel] [PATCH] avcodec/mips: Improve avc mc copy msa functions

2017-09-14 Thread kaustubh.raste
From: Kaustubh Raste Remove loops and unroll as block sizes are known. Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264qpel_msa.c | 81 +--- 1 file changed, 75 insertions(+), 6 deletions(-) diff --git a/libavcodec/mips/h264qpel_msa.c b/libavcodec/mi

[FFmpeg-devel] [PATCH] avcodec/mips: Improve avc lpf msa functions

2017-09-14 Thread kaustubh.raste
From: Kaustubh Raste Optimize luma intra case by reducing conditional cases. Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264dsp_msa.c | 428 + 1 file changed, 138 insertions(+), 290 deletions(-) diff --git a/libavcodec/mips/h264dsp_msa.c b/libav

[FFmpeg-devel] [PATCH] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-14 Thread Brian Matherly
Correctly set frame.interlaced and frame.top_field_first when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c index cd55d50..d0f9966 100644 --- a/libavcodec/hevc_sei.

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-14 Thread Michael Niedermayer
On Tue, Sep 12, 2017 at 06:45:57PM -0700, Thierry Foucu wrote: > Fix ticket #2674 > Tested with examples from ticket 2674. > --- > Sorry Michael, I forgot to configure using --enable-gpl. > Please find new patch. > > > libavfilter/vf_fps.c | 44 > +++-

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Michael Niedermayer
On Fri, Sep 15, 2017 at 12:13:56AM +0200, Carl Eugen Hoyos wrote: > 2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > > > also please add a fate test, if the file is not too large > > Are 10 bytes ok? with mpeg-ts, i think its ok > The alternative is to simply mux any latm sample into mpe

Re: [FFmpeg-devel] [PATCH] avcodec/frame_thread_encoder: Fix AV_OPT_TYPE_STRING handling in avctx

2017-09-14 Thread Michael Niedermayer
On Wed, Sep 13, 2017 at 08:11:38PM +0200, Reimar Döffinger wrote: > On Wed, Sep 13, 2017 at 07:12:48PM +0200, Reimar Döffinger wrote: > > This is the equivalent to what 7d317d4706b49d572a1eb5269438753be18362c7 > > did for the codec-specific options. > > av_opt_copy has specific handling so it's fin

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Mark Thompson
On 15/09/17 00:15, Andy Furniss wrote: > Andy Furniss wrote: >> Mark Thompson wrote: >>> --- >>> Now sets the trusted packet flag; otherwise unchanged. >>> >>> >>>   configure    |   1 + >>>   libavdevice/Makefile |   1 + >>>   libavdevice/alldevices.c |   1 + >>>   libavdevice/kmsg

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc lpf msa functions

2017-09-14 Thread Michael Niedermayer
On Wed, Sep 13, 2017 at 09:30:45AM +, Manojkumar Bhosale wrote: > LGTM applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democracies degenerate into despotisms. -- Aristotle signature.asc Description: Di

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc idct msa functions

2017-09-14 Thread Michael Niedermayer
On Wed, Sep 13, 2017 at 09:31:00AM +, Manojkumar Bhosale wrote: > LGTM applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I do not agree with what you have to say, but I'll defend to the death your right to say it. -- Voltaire signature.asc De

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Andy Furniss
Andy Furniss wrote: Mark Thompson wrote: --- Now sets the trusted packet flag; otherwise unchanged. configure| 1 + libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/kmsgrab.c| 455 +++ 4 fi

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Andy Furniss
Mark Thompson wrote: --- Now sets the trusted packet flag; otherwise unchanged. configure| 1 + libavdevice/Makefile | 1 + libavdevice/alldevices.c | 1 + libavdevice/kmsgrab.c| 455 +++ 4 files changed, 458 insert

Re: [FFmpeg-devel] [PATCH] lavc: Mark functions where ignoring returned error code is always wrong

2017-09-14 Thread Mark Thompson
On 14/09/17 23:35, Marton Balint wrote: > > On Thu, 14 Sep 2017, Mark Thompson wrote: > >> On 14/09/17 22:54, Marton Balint wrote: >>> >>> On Thu, 14 Sep 2017, Mark Thompson wrote: >>> --- There are more around, but this marks all of the obvious ones in avcodec.h. >>> >>> [...] >>>

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Mark Thompson
On 14/09/17 22:30, Carl Eugen Hoyos wrote: > 2017-09-07 23:56 GMT+02:00 Mark Thompson : > >> +static const struct { >> +enum AVPixelFormat pixfmt; >> +uint32_t drm_format; >> +} kmsgrab_formats[] = { >> +{ AV_PIX_FMT_GRAY8,DRM_FORMAT_R8 }, >> +{ AV_PIX_FMT_GRAY16LE, DRM_F

Re: [FFmpeg-devel] [PATCH] lavc: Mark functions where ignoring returned error code is always wrong

2017-09-14 Thread Marton Balint
On Thu, 14 Sep 2017, Mark Thompson wrote: On 14/09/17 22:54, Marton Balint wrote: On Thu, 14 Sep 2017, Mark Thompson wrote: --- There are more around, but this marks all of the obvious ones in avcodec.h. [...] +av_warn_unused_result int avcodec_send_packet(AVCodecContext *avctx, const

Re: [FFmpeg-devel] [PATCH 3/4] avdevice/decklink_dec: Added Closed caption decode from VANC

2017-09-14 Thread Marton Balint
On Thu, 14 Sep 2017, Jeyapal, Karthick wrote: Hi Marton, Thanks a lot for your comments. I have modified the patch as per all your comments, except the below one. +tgt = teletext_data_unit_from_ancillary_packet(buf, buf + 1920, tgt, cctx->teletext_lines, 0); this should be

Re: [FFmpeg-devel] [PATCH] lavc: Mark functions where ignoring returned error code is always wrong

2017-09-14 Thread Mark Thompson
On 14/09/17 22:54, Marton Balint wrote: > > On Thu, 14 Sep 2017, Mark Thompson wrote: > >> --- >> There are more around, but this marks all of the obvious ones in avcodec.h. >> > > [...] > >> +av_warn_unused_result >> int avcodec_send_packet(AVCodecContext *avctx, const AVPacket *avpkt); > >>

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Carl Eugen Hoyos
2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > also please add a fate test, if the file is not too large Are 10 bytes ok? The alternative is to simply mux any latm sample into mpegts with patched FFmpeg to force id 0x0f. Carl Eugen ___ ffmpeg-d

Re: [FFmpeg-devel] [PATCH] kmsgrab: fix build error when use old libdrm

2017-09-14 Thread Mark Thompson
On 14/09/17 11:02, Mark Thompson wrote: > On 14/09/17 01:34, Jun Zhao wrote: >> From 8c7ddfcabf686f213aa59544d90055d20bdac0f7 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Wed, 13 Sep 2017 20:21:38 -0400 >> Subject: [PATCH] kmsgrab: fix build error when use old libdrm >> >> DRM_FORMAT_R16 ad

Re: [FFmpeg-devel] [PATCH] lavc: Mark functions where ignoring returned error code is always wrong

2017-09-14 Thread Marton Balint
On Thu, 14 Sep 2017, Mark Thompson wrote: --- There are more around, but this marks all of the obvious ones in avcodec.h. [...] +av_warn_unused_result int avcodec_send_packet(AVCodecContext *avctx, const AVPacket *avpkt); */ +av_warn_unused_result int avcodec_send_frame(AVCodecContext

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-09-14 Thread Michael Niedermayer
On Thu, Sep 14, 2017 at 09:51:31PM +0100, Mark Thompson wrote: > On 14/09/17 21:28, Michael Niedermayer wrote: > > On Thu, Sep 14, 2017 at 08:31:28AM +0100, Mark Thompson wrote: > >> On 14/09/17 01:42, Michael Niedermayer wrote: > >>> On Wed, Sep 13, 2017 at 12:43:53AM +0100, Mark Thompson wrote: >

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Michael Niedermayer
On Wed, Sep 13, 2017 at 07:12:17PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6657. > > Please comment, Carl Eugen > mpegts.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > 370a433251f563e16b654eeca8bb3463e787fd61 > 0001-lavf-mpegts-Consider-stream_typ

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Carl Eugen Hoyos
2017-09-07 23:56 GMT+02:00 Mark Thompson : > +static const struct { > +enum AVPixelFormat pixfmt; > +uint32_t drm_format; > +} kmsgrab_formats[] = { > +{ AV_PIX_FMT_GRAY8,DRM_FORMAT_R8 }, > +{ AV_PIX_FMT_GRAY16LE, DRM_FORMAT_R16 }, > +{ AV_PIX_FMT_RGB24,DRM_F

[FFmpeg-devel] [PATCH]lavf/http: Reset encoding on redirect

2017-09-14 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #. Please comment, Carl Eugen From 479e3b2097a73a918f0c62cb6e3d661f9e71c5d0 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 14 Sep 2017 23:21:53 +0200 Subject: [PATCH] lavf/http: Reset encoding on redirect. Fixes ticket #. --- libavformat/http

[FFmpeg-devel] [PATCH] vaapi: Disable deprecation warnings around use of old struct vaapi_context

2017-09-14 Thread Mark Thompson
--- libavcodec/vaapi_decode.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/vaapi_decode.h b/libavcodec/vaapi_decode.h index 4fe414c504..550ee05432 100644 --- a/libavcodec/vaapi_decode.h +++ b/libavcodec/vaapi_decode.h @@ -59,9 +59,11 @@ typedef struct VAAPIDecodeContext {

[FFmpeg-devel] [PATCH] lavc: Mark functions where ignoring returned error code is always wrong

2017-09-14 Thread Mark Thompson
--- There are more around, but this marks all of the obvious ones in avcodec.h. (Prompted by my own stupid error in <01ec4b44-1857-fdf4-ba1d-84fcac474...@jkqxz.net>.) libavcodec/avcodec.h | 13 + 1 file changed, 13 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcod

[FFmpeg-devel] [PATCH 1/3] avfilter/interlace: simplify code

2017-09-14 Thread Thomas Mundt
Patch attached 0001-avfilter-interlace-simplify-code.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 3/3] avfilter/interlace: add support for 10 and 12 bit

2017-09-14 Thread Thomas Mundt
Patch attached 0003-avfilter-interlace-add-support-for-10-and-12-bit.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 2/3] avfilter/tinterlace: use drawutils for pad mode

2017-09-14 Thread Thomas Mundt
Patch attached 0002-avfilter-tinterlace-use-drawutils-for-pad-mode.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-09-14 Thread Mark Thompson
On 14/09/17 21:28, Michael Niedermayer wrote: > On Thu, Sep 14, 2017 at 08:31:28AM +0100, Mark Thompson wrote: >> On 14/09/17 01:42, Michael Niedermayer wrote: >>> On Wed, Sep 13, 2017 at 12:43:53AM +0100, Mark Thompson wrote: > [...] > +int ff_cbs_write_packet(CodedBitstreamContext *ctx, >>>

[FFmpeg-devel] [PATCH 0/3] avfilter/interlace: add support for 10 and 12 bit

2017-09-14 Thread Thomas Mundt
Hi, this patch set adds support for 10 and 12 bit to the interlace filters. This is useful for broadcasters transcoding high bit depth sources to AVC Intra or similar 10 bit video codecs. These patches need to be applied on top of the following patch, which is not pushed yet: http://ffmpeg.org/pip

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-09-14 Thread Michael Niedermayer
On Thu, Sep 14, 2017 at 08:31:28AM +0100, Mark Thompson wrote: > On 14/09/17 01:42, Michael Niedermayer wrote: > > On Wed, Sep 13, 2017 at 12:43:53AM +0100, Mark Thompson wrote: [...] > >> +int ff_cbs_write_packet(CodedBitstreamContext *ctx, > >> +AVPacket *pkt, > >> +

Re: [FFmpeg-devel] img2dec: jpeg_probe logic

2017-09-14 Thread Vadim Kalinsky
Can we change mjpeg_probe() then? It has the same problem. "Everything was valid, and there's more than two frames, but it still has lower score than extension-based guess" libavformat/rawdec.c:184 if (nb_invalid == 0 && nb_frames > 2) return AVPROBE_SCORE_EXTENSION / 2; Vadim

Re: [FFmpeg-devel] [PATCH] vf_fps: when reading EOF, using current_pts to duplicate the last frame if needed.

2017-09-14 Thread Thierry Foucu
On Tue, Sep 12, 2017 at 6:45 PM, Thierry Foucu wrote: > Fix ticket #2674 > Tested with examples from ticket 2674. > --- > Sorry Michael, I forgot to configure using --enable-gpl. > Please find new patch. > > > ping? > libavfilter/vf_fps.c | 44 ++ > +

Re: [FFmpeg-devel] [PATCHv3] avformat/mpegts: opus muxing & demuxing for mapping family 255

2017-09-14 Thread pkv.stream
Thanks for your comments Moritz. Corrected patch in attachment. Regards Le 14/09/2017 à 5:46 PM, Moritz Barsnick a écrit : On Fri, Sep 08, 2017 at 01:46:38 +0200, pkv.stream wrote: -avpriv_request_sample(fc, "Opus in MPEG-TS - channel_config_code > 0x8"); [...] +

Re: [FFmpeg-devel] [PATCHv3] avformat/mpegts: opus muxing & demuxing for mapping family 255

2017-09-14 Thread Moritz Barsnick
On Fri, Sep 08, 2017 at 01:46:38 +0200, pkv.stream wrote: > -avpriv_request_sample(fc, "Opus in MPEG-TS - > channel_config_code > 0x8"); [...] > +avpriv_request_sample(fc, "Opus in MPEG-TS - > channel_config_code"); You probably need to mention the

Re: [FFmpeg-devel] [PATCH] avfilter: add despill filter

2017-09-14 Thread Moritz Barsnick
On Mon, Aug 28, 2017 at 15:25:54 +0200, Paul B Mahol wrote: > +@table @option > +@item type > +Set what type of despill to use. Please add: Accepted values are @code{green} and @code{blue} for greenscreen and bluescreen spills. (Or make this a list.) > +@item mix > +Set how spillmap will

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

2017-09-14 Thread Tobias Rapp
On 14.09.2017 15:55, Tomas Härdin wrote: On 2017-09-14 15:44, Tobias Rapp wrote: Signed-off-by: Tobias Rapp ---   libavformat/mxfenc.c | 2 +-   1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 7289e0b..da4d7b4 100644 --- a/libavformat

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

2017-09-14 Thread Tomas Härdin
On 2017-09-14 15:44, Tobias Rapp wrote: Signed-off-by: Tobias Rapp --- libavformat/mxfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 7289e0b..da4d7b4 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -1

[FFmpeg-devel] [PATCH 2/2] fate: add mxf_dv25/dvcpro50 regression tests

2017-09-14 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- tests/fate/avformat.mak | 2 ++ tests/fate/seek.mak | 4 +++ tests/lavf-regression.sh | 8 ++ tests/ref/lavf/mxf_dv25 | 3 +++ tests/ref/lavf/mxf_dvcpro50 | 3 +++ tests/ref/seek/lavf-mxf_dv25 | 53 +++

[FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

2017-09-14 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- libavformat/mxfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 7289e0b..da4d7b4 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -1810,7 +1810,7 @@ static int mxf_parse_dv_fra

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mxfdec: use the common packet pts setter function for opatom files

2017-09-14 Thread Tomas Härdin
On 2017-09-13 21:31, Marton Balint wrote: On Fri, 8 Sep 2017, Michael Niedermayer wrote: On Thu, Sep 07, 2017 at 05:11:40PM +0200, Marton Balint wrote: Fixes ticket #6631. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(

Re: [FFmpeg-devel] [PATCH] kmsgrab: fix build error when use old libdrm

2017-09-14 Thread Mark Thompson
On 14/09/17 01:34, Jun Zhao wrote: > From 8c7ddfcabf686f213aa59544d90055d20bdac0f7 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Wed, 13 Sep 2017 20:21:38 -0400 > Subject: [PATCH] kmsgrab: fix build error when use old libdrm > > DRM_FORMAT_R16 adding from libdrm 2.4.82, fix the build error >

Re: [FFmpeg-devel] [PATCH] avfilter: add normalize filter

2017-09-14 Thread Nicolas George
L'octidi 28 fructidor, an CCXXV, Richard Ling a écrit : > This patch adds a filter to normalize (contrast stretch) RGB video. > Comments welcome. Thanks for the patch. Unfortunately, your mail software mangled it with line breaks, it cannot be applied as is. Still, see a few comments below. > >

Re: [FFmpeg-devel] [PATCH 5/5] lavd: Add KMS frame grabber

2017-09-14 Thread Mark Thompson
On 14/09/17 05:35, James Almer wrote: > On 9/7/2017 6:56 PM, Mark Thompson wrote: >> +pkt->data = (uint8_t*)frame; >> +pkt->size = sizeof(*frame); >> +pkt->pts = now; >> +pkt->flags = AV_PKT_FLAG_TRUSTED; > > pkt->flags |= AV_PKT_FLAG_TRUSTED; > > I know pkt->flags is zeroed b

Re: [FFmpeg-devel] [PATCH] kmsgrab: fix build error when use old libdrm

2017-09-14 Thread Mark Thompson
On 14/09/17 01:34, Jun Zhao wrote: > From 8c7ddfcabf686f213aa59544d90055d20bdac0f7 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Wed, 13 Sep 2017 20:21:38 -0400 > Subject: [PATCH] kmsgrab: fix build error when use old libdrm > > DRM_FORMAT_R16 adding from libdrm 2.4.82, fix the build error >

Re: [FFmpeg-devel] [PATCH 18/20] mpeg12: Move finding the best frame rate to common code

2017-09-14 Thread Mark Thompson
On 14/09/17 01:48, Michael Niedermayer wrote: > On Wed, Sep 13, 2017 at 12:44:08AM +0100, Mark Thompson wrote: >> Previously in the mpeg2_metadata filter. Also adds a test. > [...] >> +for (c = 1; c <= max_code; c++) { >> +for (n = 1; n <= (mpeg2 ? 4 : 1); n++) { >> +for (d

Re: [FFmpeg-devel] [PATCH 03/20] lavc: Add coded bitstream read/write API

2017-09-14 Thread Mark Thompson
On 14/09/17 01:42, Michael Niedermayer wrote: > On Wed, Sep 13, 2017 at 12:43:53AM +0100, Mark Thompson wrote: >> (cherry picked from commit 18f1706f331bf5dd565774eae680508c8d3a97ad) >> (cherry picked from commit 44cde38c8acbef7d5250e6d1b52b1020871e093b) >> --- >> configure | 1 +