summaryrefslogtreecommitdiffstats
path: root/package/opencv3
diff options
context:
space:
mode:
authorBernd Kuhls <bernd.kuhls@t-online.de>2018-07-30 18:08:36 +0200
committerThomas Petazzoni <thomas.petazzoni@bootlin.com>2018-07-31 22:05:38 +0200
commit141cf77903764efad4238abd4cd1391260c0210d (patch)
tree6c69d6c8b90540ad0e377dcc08eddef2f9493b47 /package/opencv3
parent88a161b3886c96b9bd6fc96b8f3850d422af7a5b (diff)
downloadbuildroot-141cf77903764efad4238abd4cd1391260c0210d.tar.gz
buildroot-141cf77903764efad4238abd4cd1391260c0210d.zip
package/opencv3: fix aarch64 build
Commit a17402e42d8c996af239cfdb536e74188d6c6245 ("opencv3: conditionally enable NEON and VFPv3 optimizations") broke the build on AArch64, because the VFPv3 optimizations are not supported by OpenCV on this architecture: https://github.com/opencv/opencv/blob/3.4/cmake/OpenCVCompilerOptimizations.cmake#L275 However BR2_ARM_CPU_HAS_VFPV3 is always true on AArch64, causing this build failure for every AArch64 build of OpenCV 3. This commit fixes that by enabling VFPV3 only on ARM, and not on AArch64. Fixes: http://autobuild.buildroot.net/results/d38fb092cffe086f93b469d1181da77fba0994bd/ Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Diffstat (limited to 'package/opencv3')
-rw-r--r--package/opencv3/opencv3.mk9
1 files changed, 7 insertions, 2 deletions
diff --git a/package/opencv3/opencv3.mk b/package/opencv3/opencv3.mk
index 53b771ef25..e8afa7cccf 100644
--- a/package/opencv3/opencv3.mk
+++ b/package/opencv3/opencv3.mk
@@ -100,8 +100,13 @@ OPENCV3_CONF_OPTS += \
# adding '-mcpu=G3 -mtune=G5' to them, which is already handled by Buildroot.
OPENCV3_CONF_OPTS += \
-DENABLE_POWERPC=OFF \
- -DENABLE_NEON=$(if $(BR2_ARM_CPU_HAS_NEON),ON,OFF) \
- -DENABLE_VFPV3=$(if $(BR2_ARM_CPU_HAS_VFPV3),ON,OFF)
+ -DENABLE_NEON=$(if $(BR2_ARM_CPU_HAS_NEON),ON,OFF)
+
+ifeq ($(BR2_ARCH_IS_64):$(BR2_ARM_CPU_HAS_VFPV3),:y)
+OPENCV3_CONF_OPTS += -DENABLE_VFPV3=ON
+else
+OPENCV3_CONF_OPTS += -DENABLE_VFPV3=OFF
+endif
# Cuda stuff
OPENCV3_CONF_OPTS += \
OpenPOWER on IntegriCloud