Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread Nicolas George
Le quintidi 25 germinal, an CCXXIV, wm4 a écrit : > Wouldn't it be better to keep it internal then? Sharing code between > the sub-libs is unfortunately messy Once again, I think we should merge the libs. (Without API change, of course; just the linking: -lffmpeg instead of -lavfilter -lavformat -

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread nablet developer
> On 13 Apr 2016, at 14:48, Mark Thompson wrote: > > On 13/04/16 09:14, nablet developer wrote: >> >>> On 13 Apr 2016, at 14:08, wm4 wrote: >>> >>> On Thu, 7 Apr 2016 11:44:20 -0400 >>> nablet developer mailto:s...@nablet.com>> wrote: >>> Signed-off-by: nablet developer ---

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread Mark Thompson
On 13/04/16 09:14, nablet developer wrote: > >> On 13 Apr 2016, at 14:08, wm4 wrote: >> >> On Thu, 7 Apr 2016 11:44:20 -0400 >> nablet developer mailto:s...@nablet.com>> wrote: >> >>> Signed-off-by: nablet developer >>> --- >>> libavcodec/qsv.c | 64 >>> +--

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread nablet developer
> On 13 Apr 2016, at 14:33, wm4 wrote: > > On Wed, 13 Apr 2016 14:14:37 +0600 > nablet developer wrote: > >>> Why would this API need to be exported? >> >> previously QuickSync was used only by libavcodec and its components - e.g. >> there are QSV encoder and decoders for AVC and MPEG-2. s

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread wm4
On Wed, 13 Apr 2016 14:14:37 +0600 nablet developer wrote: > > Why would this API need to be exported? > > previously QuickSync was used only by libavcodec and its components - e.g. > there are QSV encoder and decoders for AVC and MPEG-2. so it was OK that > QuickSync initialisation and clea

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread nablet developer
> On 13 Apr 2016, at 14:08, wm4 wrote: > > On Thu, 7 Apr 2016 11:44:20 -0400 > nablet developer mailto:s...@nablet.com>> wrote: > >> Signed-off-by: nablet developer >> --- >> libavcodec/qsv.c | 64 >> +-- >> libavcodec/qsv.h | 53 +

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread nablet developer
> On 13 Apr 2016, at 14:16, Hendrik Leppkes wrote: > > On Thu, Apr 7, 2016 at 5:44 PM, nablet developer wrote: >> >> +/** >> + * Initialize a MSDK session >> + * >> + * Media SDK is based on sessions, so this is the prerequisite >> + * initialization for HW acceleration. For Windows the sessi

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread Hendrik Leppkes
On Thu, Apr 7, 2016 at 5:44 PM, nablet developer wrote: > > +/** > + * Initialize a MSDK session > + * > + * Media SDK is based on sessions, so this is the prerequisite > + * initialization for HW acceleration. For Windows the session is > + * complete and ready to use, for Linux a display handle

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-13 Thread wm4
On Thu, 7 Apr 2016 11:44:20 -0400 nablet developer wrote: > Signed-off-by: nablet developer > --- > libavcodec/qsv.c | 64 > +-- > libavcodec/qsv.h | 53 +++ > libavcodec/qsv_api.c | 26

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-12 Thread nablet developer
> On 11 Apr 2016, at 18:30, nablet developer wrote: > > >>> >>> diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c >>> index 4c8e6b0..81d1f0c 100644 >>> --- a/libavcodec/qsv.c >>> +++ b/libavcodec/qsv.c >>> @@ -18,14 +18,19 @@ >>> * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 0

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-11 Thread nablet developer
>> >> diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c >> index 4c8e6b0..81d1f0c 100644 >> --- a/libavcodec/qsv.c >> +++ b/libavcodec/qsv.c >> @@ -18,14 +18,19 @@ >> * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 >> USA >> */ >> >> -#include >> -#include >> - >>

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

2016-04-11 Thread Michael Niedermayer
On Thu, Apr 07, 2016 at 11:44:20AM -0400, nablet developer wrote: > Signed-off-by: nablet developer > --- > libavcodec/qsv.c | 64 > +-- > libavcodec/qsv.h | 53 +++ > libavcodec/qsv_api.c | 26