Re: [FFmpeg-devel] [PATCH] mpeg2_hwaccel: always submit twice for a frame with two fileds.

> -Original Message- > From: Jun Zhao [mailto:mypopy...@gmail.com] > Sent: Thursday, September 21, 2017 1:01 PM > To: FFmpeg development discussions and patches > ; Li, Zhong > Cc: s...@jkqxz.net; Zhao, Jun ;

Re: [FFmpeg-devel] [PATCHv12] libavcodec: v4l2: add support for v4l2 mem2mem codecs

On 09/20/2017 08:31 PM, wm4 wrote: On Wed, 20 Sep 2017 19:01:52 -0700 Jorge Ramirez-Ortiz wrote: Btw. I noticed that this apparently never sets chroma_location? I didnt find anything in v4l2 to retrieve the chroma_location from the kernel driver (the API

Re: [FFmpeg-devel] [PATCH] mpeg2_hwaccel: always submit twice for a frame with two fileds.

On 2017/9/21 11:26, Zhong Li wrote: > Though this patch to fix https://trac.ffmpeg.org/ticket/6668, and allows > to set the slice_flags option to decoder, I belive it is unnecessary to set > SLICE_FLAG_ALLOW_FIELD flag to other hwaccels(dxva, vdpau, etc). > > Signed-off-by: Zhong Li

Re: [FFmpeg-devel] [PATCHv12] libavcodec: v4l2: add support for v4l2 mem2mem codecs

On Wed, 20 Sep 2017 19:01:52 -0700 Jorge Ramirez-Ortiz wrote: > > Btw. I noticed that this apparently never sets chroma_location? > > I didnt find anything in v4l2 to retrieve the chroma_location from the kernel > driver (the API doesnt handle this

[FFmpeg-devel] [PATCH] mpeg2_hwaccel: always submit twice for a frame with two fileds.

Though this patch to fix https://trac.ffmpeg.org/ticket/6668, and allows to set the slice_flags option to decoder, I belive it is unnecessary to set SLICE_FLAG_ALLOW_FIELD flag to other hwaccels(dxva, vdpau, etc). Signed-off-by: Zhong Li --- libavcodec/mpeg12dec.c | 3

[FFmpeg-devel] [PATCHv13] libavcodec: v4l2: add support for v4l2 mem2mem codecs

This patchset enhances Alexis Ballier's original patch and validates it using Qualcomm's Venus hardware (driver recently landed upstream [1]). This has been tested on Qualcomm's DragonBoard 410c and 820c Configure/make scripts have been validated on Ubuntu 10.04 and 16.04.

Re: [FFmpeg-devel] [PATCHv12] libavcodec: v4l2: add support for v4l2 mem2mem codecs

On 09/19/2017 12:35 PM, wm4 wrote: On Mon, 11 Sep 2017 16:26:33 +0200 Jorge Ramirez-Ortiz wrote: This patchset enhances Alexis Ballier's original patch and validates it using Qualcomm's Venus hardware (driver recently landed upstream [1]).

[FFmpeg-devel] V4L2 M2M v12

This patchset fixes handling the draining use case (tested on db410c) and simplifies the overall design making it easier to analyze and maintain. All encoders/decoders have been tested on db410c and db820c (Qualcomm's 96Boards) on their latest software releases.

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Reduced conditional cases in avc inter lpf msa functions

On Wed, Sep 20, 2017 at 07:15:26AM +, Manojkumar Bhosale wrote: > LGTM applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Everything should be made as simple as possible, but not simpler. -- Albert Einstein signature.asc Description: Digital

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Unrolled loops and expanded functions in avc put mc 10 & 30 msa functions

On Tue, Sep 19, 2017 at 11:14:56AM +, Manojkumar Bhosale wrote: > LGTM applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If you drop bombs on a foreign country and kill a hundred thousand innocent people, expect your government to call the

Re: [FFmpeg-devel] [PATCH v4] fate: add tests for psnr and ssim filter

On Wed, Sep 20, 2017 at 11:01:59AM +0200, Tobias Rapp wrote: > Metadata filter output is passed through an Awk script comparing floats > against reference values with specified "fuzz" tolerance to account for > architectural differences (e.g. x86-32 vs. x86-64). > > Signed-off-by: Tobias Rapp

Re: [FFmpeg-devel] [PATCH] avformat/hls: support cache protocol in hls

On Wed, Sep 20, 2017 at 11:03:00AM +0800, Steven Liu wrote: > support play hls from cache > > Signed-off-by: Steven Liu > --- > libavformat/hls.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This may be unsafe and allow bypassing the file/http checks [...] --

Re: [FFmpeg-devel] [PATCH]lavf/rtsp: Allow to set SDP timeout.

On Wed, Sep 20, 2017 at 02:12:40AM +0200, Carl Eugen Hoyos wrote: > 2017-09-20 2:01 GMT+02:00 Marton Balint : > > > > > > On Wed, 20 Sep 2017, Carl Eugen Hoyos wrote: > > > >> Hi! > >> > >> Attached patch fixes part of ticket #2415. > >> > >> Please comment, Carl Eugen > >> > > > >

Re: [FFmpeg-devel] [PATCH] configure: quote compiler paths

On Tue, Sep 19, 2017 at 04:54:16PM -0700, Jacob Trimble wrote: > Signed-off-by: Jacob Trimble > --- > configure | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) breaks build with ccache ./configure --cc='ccache gcc' ccache gcc is unable to create an

Re: [FFmpeg-devel] [PATCH v3] fate: add tests for psnr and ssim filter

On Wed, Sep 20, 2017 at 09:29:34AM +0200, Tobias Rapp wrote: > On 20.09.2017 01:38, Michael Niedermayer wrote: > >On Tue, Sep 19, 2017 at 09:48:45AM +0200, Tobias Rapp wrote: > >>Metadata filter output is passed through an Awk script comparing floats > >>against reference values with specified

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

On Wed, Sep 20, 2017 at 02:24:35AM +0200, Carl Eugen Hoyos wrote: > 2017-09-14 23:48 GMT+02:00 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

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

On 20/09/17 17:10, Andy Furniss wrote: > Mark Thompson wrote: >> On 19/09/17 22:21, Andy Furniss wrote: > >>> That point being around 7k frames it will run out of something. >>> >>> [AVHWFramesContext @ 0x31ed880] Failed to create surface from DRM object: 2 >>> (resource allocation failed). >>>

Re: [FFmpeg-devel] [PATCH] GnuTLS: eat PREMATURE_TERMINATION error

On Tue, Sep 19, 2017 at 09:14:58PM +0200, wm4 wrote: > On Fri, 15 Sep 2017 17:04:38 +0900 > Tatsuyuki Ishi wrote: > > > Subject: [PATCH] GnuTLS: eat PREMATURE_TERMINATION error > > > > GnuTLS is too strict on the SSL shutdown alert, and it's neither > > mandatory in the

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

On 9/20/2017 7:30 PM, wm4 wrote: > To be honest, ENOSYS is even more confusing than -1. Think about what > happens if someone converts the error to a string and displays that to > a user. AVERROR(EINVAL)? - Derek ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 22:00:34 +0200 Michael Niedermayer wrote: > On Wed, Sep 20, 2017 at 08:32:59PM +0200, wm4 wrote: > > On Wed, 20 Sep 2017 15:22:42 -0300 > > James Almer wrote: > > > > > On 9/20/2017 3:18 PM, wm4 wrote: > > > > On Wed, 20 Sep

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On Wed, Sep 20, 2017 at 08:32:59PM +0200, wm4 wrote: > On Wed, 20 Sep 2017 15:22:42 -0300 > James Almer wrote: > > > On 9/20/2017 3:18 PM, wm4 wrote: > > > On Wed, 20 Sep 2017 04:00:28 +0100 > > > Rostislav Pehlivanov wrote: > > > > > >> +if

Re: [FFmpeg-devel] [PATCH] x86/exrdsp: optimize ff_reorder_pixels_avx2()

> > I also had a hard time getting to notice a difference with > star/stop_timer, and it's clear why seeing how little difference this > change truly makes. > > You can build the above tool with "make checkasm", and the executable > will be in the tests/checkasm folder. The results tend to be less

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

On Wed, Sep 20, 2017 at 02:03:02PM +0200, Nicolas George wrote: > Le jour de la Raison, an CCXXV, Vittorio Giovara a écrit : > > It's to be consistent with the other APIs that map a string to an enum, > > like in spherical.c and stereo3d.c. > > I think they should be fixed. Returning -1 as an

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 15:22:42 -0300 James Almer wrote: > On 9/20/2017 3:18 PM, wm4 wrote: > > On Wed, 20 Sep 2017 04:00:28 +0100 > > Rostislav Pehlivanov wrote: > > > >> +if (mdm.has_gamma || mdm.has_primaries) { > >> +

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 15:28:26 -0300 James Almer wrote: > On 9/20/2017 3:18 PM, wm4 wrote: > > On Wed, 20 Sep 2017 04:00:27 +0100 > > Rostislav Pehlivanov wrote: > > > >> PNG exposes it and its required in order to correctly display some images, > >>

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

On Wed, 20 Sep 2017 14:23:32 +0200 Vittorio Giovara wrote: > Signed-off-by: Vittorio Giovara > --- > Updated following review. > Vittorio > > libavutil/pixdesc.c | 65 > + >

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 9/20/2017 3:18 PM, wm4 wrote: > On Wed, 20 Sep 2017 04:00:27 +0100 > Rostislav Pehlivanov wrote: > >> PNG exposes it and its required in order to correctly display some images, >> particularly images crafted to contain 2 different images which appear >> differently

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 04:00:28 +0100 Rostislav Pehlivanov wrote: > +if (mdm.has_gamma || mdm.has_primaries) { > +AVMasteringDisplayMetadata *new_mdm = > av_mastering_display_metadata_create_side_data(p); > +memcpy(new_mdm, ,

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 12:58:14 -0300 James Almer wrote: > On 9/20/2017 12:47 PM, Rostislav Pehlivanov wrote: > > On 20 September 2017 at 16:05, Hendrik Leppkes wrote: > > > >> On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov > >>

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On 9/20/2017 3:18 PM, wm4 wrote: > On Wed, 20 Sep 2017 04:00:28 +0100 > Rostislav Pehlivanov wrote: > >> +if (mdm.has_gamma || mdm.has_primaries) { >> +AVMasteringDisplayMetadata *new_mdm = >> av_mastering_display_metadata_create_side_data(p); >> +

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 04:00:27 +0100 Rostislav Pehlivanov wrote: > PNG exposes it and its required in order to correctly display some images, > particularly images crafted to contain 2 different images which appear > differently depending on whether the gamma has been taken

Re: [FFmpeg-devel] [PATCH 2/2] pngdec: expose gAMA and cHRM chunks as AVMasteringDisplayMetadata

On Wed, 20 Sep 2017 04:00:28 +0100 Rostislav Pehlivanov wrote: > +if (mdm.has_gamma || mdm.has_primaries) { > +AVMasteringDisplayMetadata *new_mdm = > av_mastering_display_metadata_create_side_data(p); > +memcpy(new_mdm, ,

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On Wed, Sep 20, 2017 at 5:47 PM, Rostislav Pehlivanov wrote: > On 20 September 2017 at 16:05, Hendrik Leppkes wrote: > >> On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov >> wrote: >> > On 20 September 2017 at 12:55, Vittorio

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 9/20/2017 12:58 PM, Vittorio Giovara wrote: >> * In my opinion we should not add new fields to structures that map 1:1 to > *>* something defined elsewhere (with the exception of booleans). > *>* I think this should be a separate side data and type, ie AVGammaResponse, > *>* that can of course

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

Mark Thompson wrote: On 19/09/17 22:21, Andy Furniss wrote: That point being around 7k frames it will run out of something. [AVHWFramesContext @ 0x31ed880] Failed to create surface from DRM object: 2 (resource allocation failed). [Parsed_hwmap_0 @ 0x3114c40] Failed to map frame: -5. I see

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

>* In my opinion we should not add new fields to structures that map 1:1 to *>* something defined elsewhere (with the exception of booleans). *>* I think this should be a separate side data and type, ie AVGammaResponse, *>* that can of course reside in the same header. *>* -- *>* Vittorio *>*

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 9/20/2017 12:05 PM, Hendrik Leppkes wrote: > On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov > wrote: >> On 20 September 2017 at 12:55, Vittorio Giovara >> wrote: >> >>> diff --git a/libavutil/mastering_display_metadata.h >>>

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 9/20/2017 12:47 PM, Rostislav Pehlivanov wrote: > On 20 September 2017 at 16:05, Hendrik Leppkes wrote: > >> On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov >> wrote: >>> On 20 September 2017 at 12:55, Vittorio Giovara < >>

[FFmpeg-devel] Gif from transparent video

Hey all! I’m working on this: https://trac.ffmpeg.org/ticket/4443, which relates to creating a GIF from an image with transparency support. I have modified paletteuse, so that I can generate a transparent gif just fine — but only a single frame. Multiple frames cause problems with the gif

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

On Mon, Sep 18, 2017 at 6:57 PM, Nicolas George wrote: > I would not mind a patch that expands Mo into mega-octet. Attached with that change. (Sorry, earlier mail didn't go to the entire mailing list.) 0001-doc-faq-replace-Mo-with-mega-octet.patch Description: Binary data

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 20 September 2017 at 16:05, Hendrik Leppkes wrote: > On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov > wrote: > > On 20 September 2017 at 12:55, Vittorio Giovara < > vittorio.giov...@gmail.com> > > wrote: > > > >> diff --git

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On Wed, Sep 20, 2017 at 4:34 PM, Rostislav Pehlivanov wrote: > On 20 September 2017 at 12:55, Vittorio Giovara > wrote: > >> diff --git a/libavutil/mastering_display_metadata.h >> b/libavutil/mastering_display_metadata.h >> index

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

On 20 September 2017 at 12:55, Vittorio Giovara wrote: > diff --git a/libavutil/mastering_display_metadata.h > b/libavutil/mastering_display_metadata.h > index 847b0b62c6..3de58bf468 100644 > --- a/libavutil/mastering_display_metadata.h > +++

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

On Wed, Sep 20, 2017 at 14:23:32 +0200, Vittorio Giovara wrote: > +/** > + * @return the AVChromaLocation value for name or -1 if not found. > + */ You missed modifying the "-1" here. Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avfilter: add starfield video source

On Thu, Sep 07, 2017 at 21:29:33 +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol Works fine for me. > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vsrc_starfield.c | 191 > +++

[FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

Signed-off-by: Vittorio Giovara --- Updated following review. Vittorio libavutil/pixdesc.c | 65 + libavutil/pixdesc.h | 25 + 2 files changed, 90 insertions(+) diff --git a/libavutil/pixdesc.c

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

2017-09-20 14:13 GMT+02:00 Vittorio Giovara : > If this helps, I could modify documentation saying that these functions > will return a "negative value" ... on error. That is what the documentation should say anyway, the functions can still return an error code.

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

Le jour de la Raison, an CCXXV, Vittorio Giovara a écrit : >* It's to be consistent with the other APIs that map a string to an enum, *>* like in spherical.c and stereo3d.c. * I think they should be fixed. Returning -1 as an error code is a path we definitely do not want to go again. Regards, --

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add a gamma field to AVMasteringDisplayMetadata

diff --git a/libavutil/mastering_display_metadata.h b/libavutil/mastering_display_metadata.h index 847b0b62c6..3de58bf468 100644 --- a/libavutil/mastering_display_metadata.h +++ b/libavutil/mastering_display_metadata.h @@ -66,6 +66,16 @@ typedef struct AVMasteringDisplayMetadata { */

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

Le jour de la Raison, an CCXXV, Vittorio Giovara a écrit : > It's to be consistent with the other APIs that map a string to an enum, > like in spherical.c and stereo3d.c. I think they should be fixed. Returning -1 as an error code is a path we definitely do not want to go again. Regards, --

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

+2017-09-20 13:32 GMT+02:00 Vittorio Giovara http://ffmpeg.org/mailman/listinfo/ffmpeg-devel>>: + +>* +int av_color_range_from_name(const char *name) *+>* +{ *+>* +int i; *+>* + *+>* +for (i = 0; i < FF_ARRAY_ELEMS(color_range_names); i++) { *+>* +size_t len =

Re: [FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

2017-09-20 13:32 GMT+02:00 Vittorio Giovara : > +int av_color_range_from_name(const char *name) > +{ > +int i; > + > +for (i = 0; i < FF_ARRAY_ELEMS(color_range_names); i++) { > +size_t len = strlen(color_range_names[i]); > +if

Re: [FFmpeg-devel] [PATCH] add Weston Capture demuxer, parser and decoder

2017-09-10 21:03 GMT+02:00 Paul B Mahol : > +static int wcap_probe(AVProbeData *pd) > +{ > +if (AV_RB32(pd->buf) == MKTAG('W','C','A','P')) > +return AVPROBE_SCORE_MAX; > +else > +return 0; > +} MAX / 2 is what we normally use for 32bit compliance iirc.

[FFmpeg-devel] [PATCH] pixdesc: Add API to map color property name to enum value

Signed-off-by: Vittorio Giovara --- TODO: version bump, APIdoc entry. Vittorio libavutil/pixdesc.c | 65 + libavutil/pixdesc.h | 25 + 2 files changed, 90 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH}lavf/mxfdec: Search all components of material track for source package

2016-11-05 1:50 GMT+01:00 Carl Eugen Hoyos : > Hi! > > Attached patch fixes ticket #5925 here. I'll push this if there are no comments. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH v4] fate: add tests for psnr and ssim filter

Metadata filter output is passed through an Awk script comparing floats against reference values with specified "fuzz" tolerance to account for architectural differences (e.g. x86-32 vs. x86-64). Signed-off-by: Tobias Rapp --- tested on Linux x86-32/64 and Mips (Qemu)

Re: [FFmpeg-devel] [PATCH v3] fate: add tests for psnr and ssim filter

On 20.09.2017 01:38, Michael Niedermayer wrote: On Tue, Sep 19, 2017 at 09:48:45AM +0200, Tobias Rapp wrote: Metadata filter output is passed through an Awk script comparing floats against reference values with specified "fuzz" tolerance to account for architectural differences (e.g. x86-32 vs.

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Reduced conditional cases in avc inter lpf msa functions

LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 18, 2017 2:09 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Reduced conditional cases in

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

>Then please add the relevant copyright line to the file you changed. >(Probably "Rafaël Carré" but please verify yourself) > >Carl Eugen Thanks for the clarification. I have added the copyright line as you have advised. (Yes, I have verified that that “Rafaël Carré” was the original author of