Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Zhao, Halley
Thanks for your valued comments, I followed them in updated patch set. >> +config_buffer.profile = VAProfileNone; >> +status = s->decoder->start(&config_buffer); >> +if (status != DECODE_SUCCESS) { >> +av_log(avctx, AV_LOG_ERROR, "yami h264 decoder fail to start\n"); >> +

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: fix dts>pts when b frame >= 1

2015-01-13 Thread Agatha Hu
On 2015/1/11 0:44, Michael Niedermayer wrote: * PGP Signed by an unknown key On Sat, Jan 10, 2015 at 05:30:03PM +0100, Timo Rothenpieler wrote: Looks good to merge for me. i wanted to apply it but the patch is corrupted Applying: avcodec/nvenc: fix dts>pts when b frame >= 1 fatal: corrupt pa

[FFmpeg-devel] [PATCH] x86/swr: make int32_to_int32 un/pack_2ch functions SSE

2015-01-13 Thread James Almer
unpack_2ch is already using sse float ops only, and pack_2ch is a trivial change. Rename both to float_to_float for consistency. Signed-off-by: James Almer --- libswresample/x86/audio_convert.asm| 14 -- libswresample/x86/audio_convert_init.c | 11 +++ 2 files changed, 1

[FFmpeg-devel] [PATCH 2/2 v2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Zhao, Halley
From: "Zhao, Halley" - do not support multi-thread decoding, it is unnecessary for hw - create a decode thread to interface with yami decoding, decouple frame in and out - the output frame type (raw data | drm handle | dmabuf) are specified in avctx->coder during init - yami frame is assigned

[FFmpeg-devel] [PATCH 1/2 v2] update configure etc to use libyami for h264 decode

2015-01-13 Thread Zhao, Halley
From: "Zhao, Halley" add possible C++ based library dependency add multiple packages dependency for one component add h264dec basing on libyami --- configure | 78 -- libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 1 + 3 file

[FFmpeg-devel] [PATCH 0/2 v2] Add h264 decoder (hw acceleration) through libyami

2015-01-13 Thread Zhao, Halley
From: "Zhao, Halley" libyami is a core codec library to support hw acceleration basing on vaapi: https://github.com/01org/libyami this patch add h264dec basing on libyami, similar to the solution basing on libstagefright. moreover, it supports dma_buf and other frame mode, enables texture video

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Jean-Baptiste Kempf
On 12 Jan, Zhao, Halley wrote : > to be honest, libyami is core codec library for vaapi (especially on Intel > platform). FFmpeg already accesses vaapi directly. Adding a library in the middle is just making everything slower... With my kindest regards, -- Jean-Baptiste Kempf http://www.jbkempf

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Jean-Baptiste Kempf
On 12 Jan, Zhao, Halley wrote : > > Maintaining decoders is the point of this project. > Yes, I agree the core of ffmpeg is codec; > But, from the user, ffmpeg is usually treated as a light-weight media > framework. Being a media framework, it is good to leverage hw codec in many > cases. FFmpeg

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Jean-Baptiste Kempf
On 12 Jan, Ronald S. Bultje wrote : > - how long will it take us to implement these features ourselves? What features? -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic Device ___ ffmpeg-devel mailing list ffmpeg-

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Jean-Baptiste Kempf
On 14 Jan, Zhao, Halley wrote : > 1. I know our team/Intel plan to maintain this lib for the following years. > I'm a developer, I can't promise more. Many people have made such claims and backtracked in the end. > 2. as to libxyz, it is some way like investment: Nobody is sure of the > future,

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Michael Niedermayer
On Wed, Jan 14, 2015 at 01:35:14AM +, Zhao, Halley wrote: > Thanks your explanation. > 1. I know our team/Intel plan to maintain this lib for the following years. > I'm a developer, I can't promise more. will someone also maintain the libyami wraper code from this patch ? If so the patch shou

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-13 Thread Zhao, Halley
Thanks your explanation. 1. I know our team/Intel plan to maintain this lib for the following years. I'm a developer, I can't promise more. 2. as to libxyz, it is some way like investment: Nobody is sure of the future, but make decision basing on investment and profit. A small yami wrapper enab

Re: [FFmpeg-devel] [PATCH 2/4] lavd: add device capabilities API

2015-01-13 Thread Roger Pack
On 2/2/14, Lukasz Marek wrote: > Signed-off-by: Lukasz Marek > --- > libavdevice/avdevice.c | 191 +++ > libavdevice/avdevice.h | 238 > + > libavdevice/version.h | 2 +- > libavformat/avformat.h | 12 +++ >

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread Marc Giger
Hi, On Tue, 13 Jan 2015 14:54:33 +0100 wm4 wrote: > On Mon, 12 Jan 2015 18:59:33 +0100 > Marc Giger wrote: > > > Hi, > > > > Attached is a preliminary patch that enables runtime loading of > > external libraries via dlopen and friends. Dynamic loading is a > > build time option (--enable-dyna

Re: [FFmpeg-devel] [PATCH] doc: fix option setting for encoding example

2015-01-13 Thread Nicolas George
Le quartidi 24 nivôse, an CCXXIII, Michael Bradshaw a écrit : > In the encoding/decoding example[1], in the video_encode_example() > function, av_opt_set() is used to set the option "preset" to "slow" (here, > c is a AVCodecContext*): > > av_opt_set(c->priv_data, "preset", "slow", 0); > > I p

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread Marc Giger
Hi Stephen, On Tue, 13 Jan 2015 08:26:10 -0500 Stephen Hutchinson wrote: > On general principle, the idea would be nice. I'll leave broader > critiques on the code to others, but I do have one thing to say: > > >diff --git a/libavcodec/utils.c b/libavcodec/utils.c > >index 1ec5cae..2520e69 100

[FFmpeg-devel] [PATCH] doc: fix option setting for encoding example

2015-01-13 Thread Michael Bradshaw
In the encoding/decoding example[1], in the video_encode_example() function, av_opt_set() is used to set the option "preset" to "slow" (here, c is a AVCodecContext*): av_opt_set(c->priv_data, "preset", "slow", 0); I presume this is wrong to use priv_data, and instead the codec context should

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread Marc Giger
Hi Michael, On Mon, 12 Jan 2015 22:57:35 +0100 Michael Niedermayer wrote: > On Mon, Jan 12, 2015 at 06:59:33PM +0100, Marc Giger wrote: > > Hi, > > > > Attached is a preliminary patch that enables runtime loading of > > external libraries via dlopen and friends. Dynamic loading is a > > build

Re: [FFmpeg-devel] [PATCH] qpeg: avoid pointless invalid memcpy()

2015-01-13 Thread Michael Niedermayer
On Tue, Jan 13, 2015 at 02:47:47PM +0100, wm4 wrote: > If refdata was NULL, the memcpy() ended up copying the same memory > block onto itself, which is not only pointless, but also undefined > behavior. > --- > Someone spotted the issue in valgrind output. > --- > libavcodec/qpeg.c | 13 +++---

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread Lukasz Marek
On 13 January 2015 at 14:54, wm4 wrote: > On Mon, 12 Jan 2015 18:59:33 +0100 > Marc Giger wrote: > > > Hi, > > > > Attached is a preliminary patch that enables runtime loading of external > > libraries via dlopen and friends. Dynamic loading is a build time option > > (--enable-dynamic-loading)

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread wm4
On Mon, 12 Jan 2015 18:59:33 +0100 Marc Giger wrote: > Hi, > > Attached is a preliminary patch that enables runtime loading of external > libraries via dlopen and friends. Dynamic loading is a build time option > (--enable-dynamic-loading) and if it is not activated the libs are > linked as usua

[FFmpeg-devel] [PATCH] qpeg: avoid pointless invalid memcpy()

2015-01-13 Thread wm4
If refdata was NULL, the memcpy() ended up copying the same memory block onto itself, which is not only pointless, but also undefined behavior. --- Someone spotted the issue in valgrind output. --- libavcodec/qpeg.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-13 Thread Stephen Hutchinson
On general principle, the idea would be nice. I'll leave broader critiques on the code to others, but I do have one thing to say: >diff --git a/libavcodec/utils.c b/libavcodec/utils.c >index 1ec5cae..2520e69 100644 >--- a/libavcodec/utils.c >+++ b/libavcodec/utils.c >@@ -69,6 +69,25 @@ > #include

Re: [FFmpeg-devel] Adding Closed caption in nut muxer

2015-01-13 Thread Michael Niedermayer
On Tue, Jan 13, 2015 at 12:00:36PM +0100, Nicolas George wrote: > Le quartidi 24 nivôse, an CCXXIII, Anshul a écrit : > > Please find attachment. > > Using this people can also mux cc608 stream in nut muxer > > IIRC, you need to submit a patch to libnut before, because it holds the > official list

Re: [FFmpeg-devel] [PATCH] lavfi: Port mp=pp7 to libavfilter

2015-01-13 Thread Stefano Sabatini
On date Saturday 2015-01-10 01:46:38 +0530, Arwa Arif encoded: > On Fri, Jan 9, 2015 at 10:07 PM, Stefano Sabatini [...] > From 2367adb44a62e67427f0de5af8047a0008f3b0f8 Mon Sep 17 00:00:00 2001 > From: Arwa Arif > Date: Sat, 10 Jan 2015 01:21:14 +0530 > Subject: [PATCH] Remove mp=pp7 > > --- >

[FFmpeg-devel] Need Help Regarding Playing Live RTSP URL using FFMPEG Library

2015-01-13 Thread Bilal Ahmed
> > Hi Everyone, >> > > Hope you all will be fine and doing great. >> >> I am currently working on manipulating RTSP Streaming URL >> Programatically, but I am not getting able to do at all. I actually want to >> capture Streams from RTSP URL at run time and want to convert to some >> format (.wav

Re: [FFmpeg-devel] Adding Closed caption in nut muxer

2015-01-13 Thread Anshul
On 01/13/2015 04:30 PM, Nicolas George wrote: > Le quartidi 24 nivôse, an CCXXIII, Anshul a écrit : >> Please find attachment. >> Using this people can also mux cc608 stream in nut muxer > IIRC, you need to submit a patch to libnut before, because it holds the > official list of codes. > > Who main

Re: [FFmpeg-devel] Adding Closed caption in nut muxer

2015-01-13 Thread Nicolas George
Le quartidi 24 nivôse, an CCXXIII, Anshul a écrit : > Please find attachment. > Using this people can also mux cc608 stream in nut muxer IIRC, you need to submit a patch to libnut before, because it holds the official list of codes. Regards, -- Nicolas George signature.asc Description: Digi

[FFmpeg-devel] Adding Webvtt in hls muxer

2015-01-13 Thread Anshul
Hi I have attached the patch. Thanks Anshul Maheshwari >From eacd2356796ae927c500afdca15c040a2318415b Mon Sep 17 00:00:00 2001 From: Anshul Maheshwari Date: Tue, 13 Jan 2015 16:18:16 +0530 Subject: [PATCH] webvtt in hls muxer Signed-off-by: Anshul Maheshwari --- libavformat/hlsenc.c | 174 +++

[FFmpeg-devel] Adding Closed caption in nut muxer

2015-01-13 Thread Anshul
Hi Please find attachment. Using this people can also mux cc608 stream in nut muxer -Anshul >From 996a1d20061e9c966b02c07d98096b1a87fdff10 Mon Sep 17 00:00:00 2001 From: Anshul Maheshwari Date: Fri, 2 Jan 2015 13:20:13 +0530 Subject: [PATCH 2/2] Added codec tag in nut muxer Signed-off-by: Ansh