Re: [libav-devel] [PATCH 5/5] [RFC] configure: more fine-grained link-time dependency settings

2016-09-09 Thread Diego Biurrun
On Fri, Sep 09, 2016 at 08:08:55PM +0200, Janne Grunau wrote: > On 2016-09-05 20:31:39 +0200, Diego Biurrun wrote: > > Previously all external library dependencies were added as link-time > > dependencies to all components. This adds bogus dependencies to some > > components, so only add

Re: [libav-devel] [PATCH 3/5] configure: Drop require() helper function in favor of require2()

2016-09-09 Thread Diego Biurrun
On Fri, Sep 09, 2016 at 07:29:42PM +0200, Janne Grunau wrote: > On 2016-09-05 20:31:37 +0200, Diego Biurrun wrote: > > The latter does everything the former can do, but also handles > > multiple header #includes. > > --- > > configure | 10 +- > > 1 file changed, 1 insertion(+), 9

[libav-devel] [PATCH] configure: more fine-grained link-time dependency settings

2016-09-09 Thread Diego Biurrun
Previously all external library dependencies were added as link-time dependencies to all components. This adds bogus dependencies to some components, so only add dependencies to components if said components actually make use of a dependency. --- Updated the log msg according to Janne's

Re: [libav-devel] [PATCH 5/5] [RFC] configure: more fine-grained link-time dependency settings

2016-09-09 Thread Janne Grunau
On 2016-09-05 20:31:39 +0200, Diego Biurrun wrote: > Previously all external library dependencies were added as link-time > dependencies to all components. This adds bogus dependencies to some > components, so only add dependencies to components if said components > actually make use of a

Re: [libav-devel] [PATCH 3/5] configure: Drop require() helper function in favor of require2()

2016-09-09 Thread Janne Grunau
On 2016-09-05 20:31:37 +0200, Diego Biurrun wrote: > The latter does everything the former can do, but also handles > multiple header #includes. > --- > configure | 10 +- > 1 file changed, 1 insertion(+), 9 deletions(-) ok but I would fold it into the check_lib patch Janne

Re: [libav-devel] [PATCH] vaapi_vp8: Explicitly include libva vp8 decode header

2016-09-09 Thread Luca Barbato
On 09/09/16 16:59, Mark Thompson wrote: > With some old libva versions does not automatically include > the per-codec subsidiary headers, so we need to include the right one > explicitly ourselves. > --- > FATE has one of these versions. Apparently testing the oldest (fails the > configure test)

Re: [libav-devel] [PATCH 2/5] configure: Drop check_lib() helper function in favor of check_lib2()

2016-09-09 Thread Janne Grunau
On 2016-09-05 20:31:36 +0200, Diego Biurrun wrote: > The latter does everything the former can do, but also handles > multiple header #includes. > --- > configure | 42 +- > 1 file changed, 17 insertions(+), 25 deletions(-) ok but please mention that

[libav-devel] [PATCH] vaapi_vp8: Explicitly include libva vp8 decode header

2016-09-09 Thread Mark Thompson
With some old libva versions does not automatically include the per-codec subsidiary headers, so we need to include the right one explicitly ourselves. --- FATE has one of these versions. Apparently testing the oldest (fails the configure test) and the newest (all works) libva versions is not

Re: [libav-devel] [PATCH] audiodsp: Add VSX variant

2016-09-09 Thread Diego Biurrun
On Fri, Sep 09, 2016 at 03:06:46PM +0200, Luca Barbato wrote: > --- a/libavcodec/ppc/audiodsp.c > +++ b/libavcodec/ppc/audiodsp.c > @@ -61,12 +61,37 @@ static int32_t scalarproduct_int16_altivec(const int16_t > *v1, const int16_t *v2, > > +#if HAVE_VSX > +static int32_t

Re: [libav-devel] [PATCH] audiodsp: Add VSX variant

2016-09-09 Thread Luca Barbato
On 09/09/16 15:06, Luca Barbato wrote: > Signed-off-by: Luca Barbato > --- > libavcodec/ppc/audiodsp.c | 33 + > 1 file changed, 29 insertions(+), 4 deletions(-) > > While at it. > It passes the checkasm test on power8le. lu

[libav-devel] [PATCH] audiodsp: Add VSX variant

2016-09-09 Thread Luca Barbato
Signed-off-by: Luca Barbato --- libavcodec/ppc/audiodsp.c | 33 + 1 file changed, 29 insertions(+), 4 deletions(-) While at it. diff --git a/libavcodec/ppc/audiodsp.c b/libavcodec/ppc/audiodsp.c index 2a0a6d8..3320133 100644 ---