diff mbox

[1/1] package/libyuv: not available on aarch64

Message ID 1441545295-26498-1-git-send-email-bernd.kuhls@t-online.de
State Superseded
Headers show

Commit Message

Bernd Kuhls Sept. 6, 2015, 1:14 p.m. UTC
aarch64 is a NEON-enabled arch according to the source code[1] but the
files containing the NEON-specific functions[2] are not added to the
build using CMake.

Until this is fixed we disable libyuv for aarch64.

Fixes
http://autobuild.buildroot.net/results/0c5/0c559f1b1b9bc9f2e2a131e34f034ce4e34fa75b/
http://autobuild.buildroot.net/results/bf3/bf324985e9bbcd61196304de1d0313a0e216a0a8/
http://autobuild.buildroot.net/results/d55/d552e5e46c263ff9860192a993ee9ed11d7783a6/
http://autobuild.buildroot.net/results/2d7/2d73ccdfd5d06bd6dfdc4e303fe48a554460fa1c/
http://autobuild.buildroot.net/results/972/97207ed46a519368e242f0621e4363d522e21ee8/
http://autobuild.buildroot.net/results/d84/d84d54d85183296c57c9131d946e4517ddcaa84d/
http://autobuild.buildroot.net/results/c18/c189149b8d2a0242da06ec68863db23cd0fa65b8/
http://autobuild.buildroot.net/results/73a/73a026644179708692c43166a8ffba3ce0e19d95/

[1] https://freeswitch.org/stash/projects/SD/repos/libyuv/browse/include/libyuv/row.h#263
[2] https://freeswitch.org/stash/projects/SD/repos/libyuv/browse/BUILD.gn#101

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/libyuv/Config.in | 2 ++
 1 file changed, 2 insertions(+)

Comments

Yann E. MORIN Sept. 6, 2015, 8:15 p.m. UTC | #1
Bernd, All,

On 2015-09-06 15:14 +0200, Bernd Kuhls spake thusly:
> aarch64 is a NEON-enabled arch according to the source code[1] but the
> files containing the NEON-specific functions[2] are not added to the
> build using CMake.
> 
> Until this is fixed we disable libyuv for aarch64.

The fix looks pretty simple, something like:

    diff --git a/BUILD.gn b/BUILD.gn
    index 43a25e0..9c024f2 100644
    --- a/BUILD.gn
    +++ b/BUILD.gn
    @@ -15,7 +15,7 @@ config("libyuv_config") {
       ]
     }
     
    -use_neon = current_cpu == "arm" && (arm_use_neon || arm_optionally_use_neon)
    +use_neon = (current_cpu == "arm" || current_cpu == "arm64") && (arm_use_neon || arm_optionally_use_neon)
     
     source_set("libyuv") {
       sources = [

> Fixes
> http://autobuild.buildroot.net/results/0c5/0c559f1b1b9bc9f2e2a131e34f034ce4e34fa75b/
> http://autobuild.buildroot.net/results/bf3/bf324985e9bbcd61196304de1d0313a0e216a0a8/
> http://autobuild.buildroot.net/results/d55/d552e5e46c263ff9860192a993ee9ed11d7783a6/
> http://autobuild.buildroot.net/results/2d7/2d73ccdfd5d06bd6dfdc4e303fe48a554460fa1c/
> http://autobuild.buildroot.net/results/972/97207ed46a519368e242f0621e4363d522e21ee8/
> http://autobuild.buildroot.net/results/d84/d84d54d85183296c57c9131d946e4517ddcaa84d/
> http://autobuild.buildroot.net/results/c18/c189149b8d2a0242da06ec68863db23cd0fa65b8/
> http://autobuild.buildroot.net/results/73a/73a026644179708692c43166a8ffba3ce0e19d95/
> 
> [1] https://freeswitch.org/stash/projects/SD/repos/libyuv/browse/include/libyuv/row.h#263
> [2] https://freeswitch.org/stash/projects/SD/repos/libyuv/browse/BUILD.gn#101
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/libyuv/Config.in | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/libyuv/Config.in b/package/libyuv/Config.in
> index 29dcaa4..4a7c376 100644
> --- a/package/libyuv/Config.in
> +++ b/package/libyuv/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_LIBYUV
>  	bool "libyuv"
> +	depends on !BR2_aarch64 # not supported by buildsystem
>  	depends on BR2_INSTALL_LIBSTDCPP
>  	depends on !BR2_STATIC_LIBS
>  	help
> @@ -9,4 +10,5 @@ config BR2_PACKAGE_LIBYUV
>  	  https://code.google.com/p/libyuv
>  
>  comment "libyuv needs a toolchain w/ C++, dynamic library"
> +	depends on !BR2_aarch64
>  	depends on !BR2_INSTALL_LIBSTDCPP || BR2_STATIC_LIBS
> -- 
> 2.1.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

diff --git a/package/libyuv/Config.in b/package/libyuv/Config.in
index 29dcaa4..4a7c376 100644
--- a/package/libyuv/Config.in
+++ b/package/libyuv/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_LIBYUV
 	bool "libyuv"
+	depends on !BR2_aarch64 # not supported by buildsystem
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on !BR2_STATIC_LIBS
 	help
@@ -9,4 +10,5 @@  config BR2_PACKAGE_LIBYUV
 	  https://code.google.com/p/libyuv
 
 comment "libyuv needs a toolchain w/ C++, dynamic library"
+	depends on !BR2_aarch64
 	depends on !BR2_INSTALL_LIBSTDCPP || BR2_STATIC_LIBS