Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Philip Langdale
On Tue, 14 Nov 2017 21:14:58 +0100 Timo Rothenpieler wrote: > looks straight forward enough to me, struct naming matches up as well. > > add a Changelog and bump lavc version, and it's good to go. > Thanks. Pushed with Changelog and version bump. --phil __

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Philip Langdale
On Tue, 14 Nov 2017 21:07:45 +0100 Moritz Barsnick wrote: > On Tue, Nov 14, 2017 at 07:24:14 -0800, Philip Langdale wrote: > > This hwaccel is interesting because it also works for wmv3/9 > > content, which is not supported by the nvidia parser used by > > cuviddec. > > Nits: > Changelog entry

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Timo Rothenpieler
looks straight forward enough to me, struct naming matches up as well. add a Changelog and bump lavc version, and it's good to go. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmp

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Moritz Barsnick
On Tue, Nov 14, 2017 at 07:24:14 -0800, Philip Langdale wrote: > This hwaccel is interesting because it also works for wmv3/9 content, > which is not supported by the nvidia parser used by cuviddec. Nits: Changelog entry? Version bump? > +case AV_CODEC_ID_VC1: return cudaVideoCodec_VC1; >

[FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Philip Langdale
This hwaccel is interesting because it also works for wmv3/9 content, which is not supported by the nvidia parser used by cuviddec. Signed-off-by: Philip Langdale --- configure | 3 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 2 + libavcodec/nvdec.c | 2 + l