Re: [FFmpeg-devel] [PATCH] libavfilter: Add OpenCL convolution filter v0.2: add name

2018-03-23 Thread Josh de Kock
On 2018/03/23 13:22, Danil Iashchenko wrote: Thanks, fixed! --- libavfilter/opencl/convolution.cl | 2 ++ libavfilter/vf_convolution_opencl.c | 2 ++ 2 files changed, 4 insertions(+) When you update patches you should send the same patch again (yes the whole patch) with the required

Re: [FFmpeg-devel] [PATCH] libavfilter: Add OpenCL convolution filter v0.2

2018-03-23 Thread Mark Thompson
On 22/03/18 22:52, Danil Iashchenko wrote: > Hi there! > Thank you for advices, I have fixed the problem when per plane matrices > application was incorrect. Now it works as expected and behaves like the > existing vf_convolution filter. > > Tested for yuv and nv12 formats. > The following

[FFmpeg-devel] [PATCH] libavfilter: Add OpenCL convolution filter v0.2: add name

2018-03-23 Thread Danil Iashchenko
Thanks, fixed! --- libavfilter/opencl/convolution.cl | 2 ++ libavfilter/vf_convolution_opencl.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/libavfilter/opencl/convolution.cl b/libavfilter/opencl/convolution.cl index aa1db97..c0748cc 100644 --- a/libavfilter/opencl/convolution.cl

Re: [FFmpeg-devel] [PATCH] libavfilter: Add OpenCL convolution filter v0.2

2018-03-23 Thread Carl Eugen Hoyos
2018-03-22 23:52 GMT+01:00, Danil Iashchenko : > --- /dev/null > +++ b/libavfilter/opencl/convolution.cl > @@ -0,0 +1,40 @@ > +/* > + * This file is part of FFmpeg. Any reason why you don't put your name here? To state the obvious: There are now two probably qualified

[FFmpeg-devel] [PATCH] libavfilter: Add OpenCL convolution filter v0.2

2018-03-22 Thread Danil Iashchenko
Hi there! Thank you for advices, I have fixed the problem when per plane matrices application was incorrect. Now it works as expected and behaves like the existing vf_convolution filter. Tested for yuv and nv12 formats. The following filters from ffmpeg documentation