diff mbox series

[1/2] package/wolfssl: disable asm on armv4

Message ID 20230726194453.1462570-1-bernd@kuhls.net
State Accepted
Headers show
Series [1/2] package/wolfssl: disable asm on armv4 | expand

Commit Message

Bernd Kuhls July 26, 2023, 7:44 p.m. UTC
Quoting
https://developer.arm.com/documentation/dui0489/i/arm-and-thumb-instructions/clz
"This ARM instruction is available in ARMv5T and above."

Fixes:
- armeb
  http://autobuild.buildroot.net/results/ea9/ea9b19e872b4b20b97fceec3f7e0a6a1109c4f12/
  /tmp/ccdRCOaB.s:27: Error: selected processor does not support `clz r5,r2' in ARM mode
- arm
  http://autobuild.buildroot.net/results/cb1/cb1c7ac68b3379d3f4b5efc897198bea0a904217/
  /tmp/cc72ECAj.s:6764: Error: selected processor does not support `clz r5,r0' in ARM mode
  /tmp/cc72ECAj.s:7110: Error: selected processor does not support `clz r5,lr' in ARM mode
  /tmp/cc72ECAj.s:8934: Error: selected processor does not support `clz r5,fp' in ARM mode

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
 package/wolfssl/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni July 27, 2023, 10:28 a.m. UTC | #1
On Wed, 26 Jul 2023 21:44:52 +0200
Bernd Kuhls <bernd@kuhls.net> wrote:

> Quoting
> https://developer.arm.com/documentation/dui0489/i/arm-and-thumb-instructions/clz
> "This ARM instruction is available in ARMv5T and above."
> 
> Fixes:
> - armeb
>   http://autobuild.buildroot.net/results/ea9/ea9b19e872b4b20b97fceec3f7e0a6a1109c4f12/
>   /tmp/ccdRCOaB.s:27: Error: selected processor does not support `clz r5,r2' in ARM mode
> - arm
>   http://autobuild.buildroot.net/results/cb1/cb1c7ac68b3379d3f4b5efc897198bea0a904217/
>   /tmp/cc72ECAj.s:6764: Error: selected processor does not support `clz r5,r0' in ARM mode
>   /tmp/cc72ECAj.s:7110: Error: selected processor does not support `clz r5,lr' in ARM mode
>   /tmp/cc72ECAj.s:8934: Error: selected processor does not support `clz r5,fp' in ARM mode
> 
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> ---
>  package/wolfssl/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks, both applied!

Thomas
Peter Korsgaard Aug. 30, 2023, 10:49 a.m. UTC | #2
>>>>> "Bernd" == Bernd Kuhls <bernd@kuhls.net> writes:

 > Quoting
 > https://developer.arm.com/documentation/dui0489/i/arm-and-thumb-instructions/clz
 > "This ARM instruction is available in ARMv5T and above."

 > Fixes:
 > - armeb
 >   http://autobuild.buildroot.net/results/ea9/ea9b19e872b4b20b97fceec3f7e0a6a1109c4f12/
 >   /tmp/ccdRCOaB.s:27: Error: selected processor does not support `clz r5,r2' in ARM mode
 > - arm
 >   http://autobuild.buildroot.net/results/cb1/cb1c7ac68b3379d3f4b5efc897198bea0a904217/
 >   /tmp/cc72ECAj.s:6764: Error: selected processor does not support `clz r5,r0' in ARM mode
 >   /tmp/cc72ECAj.s:7110: Error: selected processor does not support `clz r5,lr' in ARM mode
 >   /tmp/cc72ECAj.s:8934: Error: selected processor does not support `clz r5,fp' in ARM mode

 > Signed-off-by: Bernd Kuhls <bernd@kuhls.net>

Committed to 2023.02.x and 2023.05.x, thanks.
diff mbox series

Patch

diff --git a/package/wolfssl/Config.in b/package/wolfssl/Config.in
index c41bdb6888..9d173412fe 100644
--- a/package/wolfssl/Config.in
+++ b/package/wolfssl/Config.in
@@ -1,7 +1,7 @@ 
 config BR2_PACKAGE_WOLFSSL_ASM_SUPPORTS
 	bool
 	default y if BR2_aarch64
-	default y if BR2_arm || BR2_armeb
+	default y if (BR2_arm || BR2_armeb) && !BR2_ARM_CPU_ARMV4 # clz
 	default y if BR2_powerpc
 	default y if BR2_powerpc64 || BR2_powerpc64le
 	default y if BR2_mips64 && !BR2_MIPS_CPU_MIPS32R6