Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-03 Thread Ivan Uskov
Hello Michael, Monday, August 3, 2015, 12:14:39 PM, you wrote: MN> On Mon, Aug 03, 2015 at 11:36:09AM +0300, Ivan Uskov wrote: >> Hello Hendrik, >> >> Monday, August 3, 2015, 12:45:36 AM, you wrote: >> >> >> HL> The decoder should depend on the header in configure directly already, >> HL> so i

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2015 at 11:36:09AM +0300, Ivan Uskov wrote: > Hello Hendrik, > > Monday, August 3, 2015, 12:45:36 AM, you wrote: > > > HL> The decoder should depend on the header in configure directly already, > HL> so its not built at all when the header is not available. > In general I do not

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-03 Thread Hendrik Leppkes
On Mon, Aug 3, 2015 at 10:36 AM, Ivan Uskov wrote: > Hello Hendrik, > > Monday, August 3, 2015, 12:45:36 AM, you wrote: > > > HL> The decoder should depend on the header in configure directly already, > HL> so its not built at all when the header is not available. > In general I do not understandi

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-03 Thread Ivan Uskov
Hello Hendrik, Monday, August 3, 2015, 12:58:28 AM, you wrote: >> Because if it's missing, ff_get_format() refuses to return the QSV >> opaque format. I think. So you need AVHWAccels for every codec/decoder >> combination. HL> But if you use the normal h264 decoder and select the QSV format in H

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-03 Thread Ivan Uskov
Hello Hendrik, Monday, August 3, 2015, 12:45:36 AM, you wrote: HL> The decoder should depend on the header in configure directly already, HL> so its not built at all when the header is not available. In general I do not understanding why it necessary at all. All necessary headers currently avail

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Hendrik Leppkes
On Sun, Aug 2, 2015 at 11:52 PM, wm4 wrote: > On Sun, 2 Aug 2015 23:45:36 +0200 > Hendrik Leppkes wrote: > >> On Sun, Aug 2, 2015 at 11:27 PM, Ivan Uskov wrote: >> > Hello Michael, >> > >> > Sunday, August 2, 2015, 9:46:23 PM, you wrote: >> >>> MN> it appears the file was not in mfx_dispatch pre

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread wm4
On Sun, 2 Aug 2015 23:45:36 +0200 Hendrik Leppkes wrote: > On Sun, Aug 2, 2015 at 11:27 PM, Ivan Uskov wrote: > > Hello Michael, > > > > Sunday, August 2, 2015, 9:46:23 PM, you wrote: > >>> MN> it appears the file was not in mfx_dispatch previously > >>> MN> so a check in confgure might be neede

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Hendrik Leppkes
On Sun, Aug 2, 2015 at 11:27 PM, Ivan Uskov wrote: > Hello Michael, > > Sunday, August 2, 2015, 9:46:23 PM, you wrote: >>> MN> it appears the file was not in mfx_dispatch previously >>> MN> so a check in confgure might be needed >>> As I can see here >>> https://github.com/lu-zero/mfx_dispatch/tre

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Ivan Uskov
Hello Michael, Sunday, August 2, 2015, 9:46:23 PM, you wrote: >> MN> it appears the file was not in mfx_dispatch previously >> MN> so a check in confgure might be needed >> As I can see here >> https://github.com/lu-zero/mfx_dispatch/tree/master/mfx >> The mfxjpeg.h was added 17 days ago and marke

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread compn
On Sun, 2 Aug 2015 20:27:17 +0200 wm4 wrote: > On Sun, 2 Aug 2015 15:31:07 +0300 > Ivan Uskov wrote: > > > Hello All, > > > > The attached patch adds QSV-based mjpeg video decoder. > > Please review. > > > > > > Is mjpeg decoding so important that we need QSV decoding of it? some ip vide

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Ivan Uskov
Hello wm4, Sunday, August 2, 2015, 9:38:33 PM, you wrote: >> w> Is mjpeg decoding so important that we need QSV decoding of it? >> Why not? It is for free. w> Having to maintain additional code has a cost, though. Near about zero, since qsv core common for all formats. -- Best regards, Ivan

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Michael Niedermayer
On Sun, Aug 02, 2015 at 09:23:21PM +0300, Ivan Uskov wrote: > Hello Michael, > > Sunday, August 2, 2015, 8:55:45 PM, you wrote: > > >> +#if QSV_VERSION_ATLEAST(1, 3) > >> +#include > >> +#endif > > MN> this seems not working > > MN> CC libavcodec/qsv.o > MN> libavcodec/qsv.c:33:25: fatal

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread wm4
On Sun, 2 Aug 2015 21:36:15 +0300 Ivan Uskov wrote: > Hello wm4, > > Sunday, August 2, 2015, 9:27:17 PM, you wrote: > > w> Is mjpeg decoding so important that we need QSV decoding of it? > Why not? It is for free. Having to maintain additional code has a cost, though. _

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Ivan Uskov
Hello wm4, Sunday, August 2, 2015, 9:27:17 PM, you wrote: w> Is mjpeg decoding so important that we need QSV decoding of it? Why not? It is for free. -- Best regards, Ivanmailto:ivan.us...@nablet.com ___ ffmpeg-devel mai

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread wm4
On Sun, 2 Aug 2015 15:31:07 +0300 Ivan Uskov wrote: > Hello All, > > The attached patch adds QSV-based mjpeg video decoder. > Please review. > > Is mjpeg decoding so important that we need QSV decoding of it? ___ ffmpeg-devel mailing list ffmpeg-d

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Ivan Uskov
Hello Michael, Sunday, August 2, 2015, 8:55:45 PM, you wrote: >> +#if QSV_VERSION_ATLEAST(1, 3) >> +#include >> +#endif MN> this seems not working MN> CC libavcodec/qsv.o MN> libavcodec/qsv.c:33:25: fatal error: mfx/mfxjpeg.h: No such file or directory MN> #include MN>

Re: [FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Michael Niedermayer
On Sun, Aug 02, 2015 at 03:31:07PM +0300, Ivan Uskov wrote: > Hello All, > > The attached patch adds QSV-based mjpeg video decoder. > Please review. > > > -- > Best regards, > Ivan mailto:ivan.us...@nablet.com > Changelog |1 > configure

[FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

2015-08-02 Thread Ivan Uskov
Hello All, The attached patch adds QSV-based mjpeg video decoder. Please review. -- Best regards, Ivan mailto:ivan.us...@nablet.com 0001-QSV-MJPEG-video-decoder-has-been-added.patch Description: Binary data ___ ffmpeg-deve