Re: [libav-devel] [PATCH] configure: Escape elements being filtered

2017-04-07 Thread Luca Barbato
On 06/04/2017 18:25, Diego Biurrun wrote: > I still haven't seen the problem this is supposed to fix. You can either install ekopath or run the configure as mentioned on the box I pointed out. lu ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] hevc: Add NEON 16x16 IDCT

2017-04-07 Thread Martin Storsjö
On Wed, 5 Apr 2017, Alexandra Hájková wrote: The speedup vs C code is around 8x. On my devboards, the speedup seems to be 6x on Cortex-A9, 7x on A7, 9x on A53, and 13x on A8. So this can probably be amended into "around 6-13x". --- libavcodec/arm/hevc_idct.S| 187

Re: [libav-devel] [PATCH 2/8] libavutil: add mastering display metadata sidedata

2017-04-07 Thread James Almer
On 4/7/2017 3:20 PM, Vittorio Giovara wrote: > On Fri, Apr 7, 2017 at 8:13 PM, James Almer wrote: >> On 4/7/2017 1:48 PM, Vittorio Giovara wrote: >>> On Fri, Apr 7, 2017 at 2:27 PM, Steve Lhomme wrote: + * @note The struct should be allocated with

Re: [libav-devel] [PATCH 2/8] libavutil: add mastering display metadata sidedata

2017-04-07 Thread Vittorio Giovara
On Fri, Apr 7, 2017 at 8:13 PM, James Almer wrote: > On 4/7/2017 1:48 PM, Vittorio Giovara wrote: >> On Fri, Apr 7, 2017 at 2:27 PM, Steve Lhomme wrote: >>> + * @note The struct should be allocated with >>> av_mastering_display_metadata_alloc() >>> + *

Re: [libav-devel] [PATCH 2/8] libavutil: add mastering display metadata sidedata

2017-04-07 Thread James Almer
On 4/7/2017 1:48 PM, Vittorio Giovara wrote: > On Fri, Apr 7, 2017 at 2:27 PM, Steve Lhomme wrote: >> + * @note The struct should be allocated with >> av_mastering_display_metadata_alloc() >> + * and its size is not a part of the public ABI. >> + */ >> +typedef struct

Re: [libav-devel] [PATCH 2/8] libavutil: add mastering display metadata sidedata

2017-04-07 Thread Vittorio Giovara
On Fri, Apr 7, 2017 at 2:27 PM, Steve Lhomme wrote: > + * @note The struct should be allocated with > av_mastering_display_metadata_alloc() > + * and its size is not a part of the public ABI. > + */ > +typedef struct AVMasteringDisplayMetadata { > +}

Re: [libav-devel] [PATCH 1/3 v2] avutil/spherical: add functions to retrieve and request projection names

2017-04-07 Thread Luca Barbato
On 07/04/2017 16:09, James Almer wrote: > +int av_spherical_from_name(const char *name); It can directly output the enum now, it is fine both ways to me. Thank you for amending it :) lu ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH 1/3 v2] avutil/spherical: add functions to retrieve and request projection names

2017-04-07 Thread James Almer
Signed-off-by: James Almer --- doc/APIchanges| 4 libavutil/spherical.c | 28 libavutil/spherical.h | 24 libavutil/version.h | 2 +- 4 files changed, 57 insertions(+), 1 deletion(-) diff --git

Re: [libav-devel] [PATCH] matroska: Read only the data written in the scratch buffer

2017-04-07 Thread James Almer
On 4/7/2017 6:43 AM, Luca Barbato wrote: > The private buffer is 20bytes but depending on the type only 12 bytes > are actually filled. > --- > > Like this, in case somebody does a similar copy+paste. > > libavformat/matroskaenc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >

[libav-devel] [PATCH 7/8] lavc: Add Content Light Level side metadata found in HEVC

2017-04-07 Thread Steve Lhomme
These data are necessary when transmitting HDR over HDMI. Signed-off-by: James Almer --- doc/APIchanges| 3 +++ libavcodec/avcodec.h | 7 +++ libavcodec/decode.c | 1 + libavcodec/hevc_sei.c | 15 +++ libavcodec/hevcdec.c | 18 ++

[libav-devel] [PATCH 8/8] avformat/dump : Display Content Light Level metadata

2017-04-07 Thread Steve Lhomme
Signed-off-by: James Almer --- libavformat/dump.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavformat/dump.c b/libavformat/dump.c index 3645e82d7b..090d5c1a9b 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -325,6 +325,14 @@ static void

[libav-devel] [PATCH 6/8] lavu: add support for Content Light Level side metadata

2017-04-07 Thread Steve Lhomme
As found in HEVC. Signed-off-by: James Almer --- doc/APIchanges | 5 + libavutil/frame.h | 6 ++ libavutil/mastering_display_metadata.c | 23 libavutil/mastering_display_metadata.h | 39

[libav-devel] [PATCH 5/8] lavf/dump.c: Print mastering display metadata

2017-04-07 Thread Steve Lhomme
From: Neil Birkbeck Signed-off-by: Neil Birkbeck Signed-off-by: Michael Niedermayer --- libavformat/dump.c | 21 + 1 file changed, 21 insertions(+) diff --git a/libavformat/dump.c

[libav-devel] [PATCH 4/8] libavcodec:add packet level support for mastering metadata

2017-04-07 Thread Steve Lhomme
From: Neil Birkbeck Some containers, like webm/mkv, will contain this mastering metadata. This is analogous to the way 3D fpa data is handled (in frame and packet side data). Signed-off-by: Neil Birkbeck Signed-off-by: Michael Niedermayer

[libav-devel] [PATCH 3/8] lavc/hevc Parse SEI_TYPE_MASTERING_DISPLAY_INFO and propagate content into the AVMasteringDisplayMetadata side data.

2017-04-07 Thread Steve Lhomme
From: Neil Birkbeck Add support for parsing SEI_TYPE_MASTERING_DISPLAY_INFO and propagate contents into the AVMasteringDisplayMetadata side data. Primaries are ordered in RGB order and the values are converted to rationals ([0,1] for CEI 1931 Chroma coords, and cd/m^2

[libav-devel] [PATCH 1/8] hevc: properly handle no_rasl_output_flag when removing pictures from the DPB

2017-04-07 Thread Steve Lhomme
From: Hendrik Leppkes Fixes ticket #4185. Reviewed-By: Mickael Raulet Signed-off-by: Hendrik Leppkes --- libavcodec/hevcdec.c | 3 +++ libavcodec/hevcdec.h | 1 + 2 files changed, 4 insertions(+) diff --git

[libav-devel] [PATCH 2/8] libavutil: add mastering display metadata sidedata

2017-04-07 Thread Steve Lhomme
From: Neil Birkbeck Adding mastering display metadata struct to avutil. The mastering display metadata contains information about the mastering display color volume (SMPTE 2086:2014). This info comes from HEVC in the SEI_TYPE_MASTERING_DISPLAY_INFO and is soon to be

Re: [libav-devel] [PATCH] matroskaenc: fix writing ProjectionPrivate element on Cubemap projections

2017-04-07 Thread Vittorio Giovara
On Fri, Apr 7, 2017 at 11:54 AM, Hendrik Leppkes wrote: > On Fri, Apr 7, 2017 at 11:32 AM, Vittorio Giovara > wrote: >> On Thu, Apr 6, 2017 at 10:45 PM, James Almer wrote: >>> Calling put_ebml_binary() with sizeof(private) as

Re: [libav-devel] [PATCH] matroskaenc: fix writing ProjectionPrivate element on Cubemap projections

2017-04-07 Thread Hendrik Leppkes
On Fri, Apr 7, 2017 at 11:32 AM, Vittorio Giovara wrote: > On Thu, Apr 6, 2017 at 10:45 PM, James Almer wrote: >> Calling put_ebml_binary() with sizeof(private) as argument is currently >> only valid for Equirectangular projections. >> >>

[libav-devel] [PATCH] matroska: Read only the data written in the scratch buffer

2017-04-07 Thread Luca Barbato
The private buffer is 20bytes but depending on the type only 12 bytes are actually filled. --- Like this, in case somebody does a similar copy+paste. libavformat/matroskaenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/matroskaenc.c

Re: [libav-devel] [PATCH] matroskaenc: fix writing ProjectionPrivate element on Cubemap projections

2017-04-07 Thread Vittorio Giovara
On Thu, Apr 6, 2017 at 10:45 PM, James Almer wrote: > Calling put_ebml_binary() with sizeof(private) as argument is currently > only valid for Equirectangular projections. > > Signed-off-by: James Almer > --- > Please, be more careful when making cosmetic

Re: [libav-devel] [PATCH 13/25] hwcontext: Improve allocation in derived contexts

2017-04-07 Thread Anton Khirnov
Quoting Mark Thompson (2017-04-02 20:37:13) > On 31/03/17 20:14, Anton Khirnov wrote: > > Quoting Mark Thompson (2017-03-05 00:57:40) > >> Use the flags argument of av_hwframe_ctx_create_derived() to pass the > >> mapping flags which will be used on allocation. Also, set the format > >> and

Re: [libav-devel] [PATCH 12/25] hwcontext: Add frame context mapping for nontrivial contexts

2017-04-07 Thread Anton Khirnov
Quoting Mark Thompson (2017-04-02 20:43:54) > On 02/04/17 11:13, Anton Khirnov wrote: > > Quoting Mark Thompson (2017-03-05 00:57:39) > >> Some frames contexts are not usable without additional format-specific > >> state in hwctx. This adds new functions map_frames_from and > >> map_frames_to to

Re: [libav-devel] [PATCH] configure: Escape elements being filtered

2017-04-07 Thread Luca Barbato
On 06/04/2017 18:25, Diego Biurrun wrote: > On Thu, Apr 06, 2017 at 06:14:29PM +0200, Luca Barbato wrote: >> The function is used to filter out ldflags arguments that may contain $. >> Update filter() as well to be consistent. >> --- >> >> Ping in form of updated patch. >> >> configure | 4 ++--

Re: [libav-devel] [PATCH] matroskaenc: fix writing ProjectionPrivate element on Cubemap projections

2017-04-07 Thread Luca Barbato
On 06/04/2017 22:45, James Almer wrote: > Calling put_ebml_binary() with sizeof(private) as argument is currently > only valid for Equirectangular projections. > > Signed-off-by: James Almer > --- > Please, be more careful when making cosmetic changes to make sure they > don't

Re: [libav-devel] [PATCH 1/3] avutil/spherical: add functions to retrieve and request projection names

2017-04-07 Thread Luca Barbato
On 04/04/2017 22:24, James Almer wrote: > +const char *av_spherical_projection_name(enum AVSphericalProjection > projection) > +{ > +if ((unsigned)projection >= FF_ARRAY_ELEMS(spherical_projection_names)) > +return "unknown"; > + > +return spherical_projection_names[projection]; >