From 68d0385533698e49747a070a03bd7a06652288bd Mon Sep 17 00:00:00 2001 From: Giulio Benetti Date: Mon, 15 Aug 2022 17:48:51 +0200 Subject: [PATCH] Revert "configs/olimex_a33_olinuxino: use NEON+VFPv4 as FPU strategy" The gcc man page states that specifying Neon as part of the fpu setting has no effect, unless the -funsafe-math-optimizations is also specified, because Neon is not compliant with IEEE 754: ``` If the selected floating-point hardware includes the NEON extension (e.g. -mfpu=neon), note that floating-point operations are not generated by GCC's auto-vectorization pass unless -funsafe-math-optimizations is also specified. This is because NEON hardware does not fully implement the IEEE 754 standard for floating-point arithmetic (in particular denormal values are treated as zero), so the use of NEON instructions may lead to a loss of precision. ``` -funsafe-math-optimizations must be explictly specified per package to really use NEON as FPU, but it's something that is left to the user as well as setting BR2_ARM_FPU_NEON_VFPV4. This way the default BR2_ARM_FPU_VFPV4D16 is used as previously. So let's revert the offending patch. This reverts commit 23329364e206d5ca67a9eb9c733d3d556ccdbac4. Signed-off-by: Giulio Benetti Signed-off-by: Yann E. MORIN --- configs/olimex_a33_olinuxino_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/configs/olimex_a33_olinuxino_defconfig b/configs/olimex_a33_olinuxino_defconfig index 27924b9e92..cdbc63efe1 100644 --- a/configs/olimex_a33_olinuxino_defconfig +++ b/configs/olimex_a33_olinuxino_defconfig @@ -1,7 +1,6 @@ # Architecture BR2_arm=y BR2_cortex_a7=y -BR2_ARM_FPU_NEON_VFPV4=y # Linux headers same as kernel, a 5.14 series BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_5_14=y