Skip to contentSkip to conversation

nodejs: not supported on old ARM cores

PATCH New Gustavo Zacarias 2 participants 2 messages Jul 12, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 nodejs: not supported on old ARM cores

nodejs needs the ARM blx instruction which is only supported in v5t+ cores (v5 ISA with thumb instructions).
Disable it for lower cores. Fixes:
http://autobuild.buildroot.net/results/89e/89ee5ba047a26a8c7a612d0285b08780b70efbd4/

---
 package/nodejs/Config.in | 2 ++
 1 file changed, 2 insertions(+)
package/nodejs/Config.in +2 -0
index aa65c6f..fea3480 100644
@@ -5,6 +5,8 @@ config BR2_PACKAGE_NODEJS
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_arm || BR2_i386 || BR2_x86_64
+	# ARM needs BLX, so v5t+
+	depends on !(BR2_arm7tdmi || BR2_arm720t || BR2_arm920t || BR2_arm922t || BR2_fa526)
 	# uses fork()
 	depends on BR2_USE_MMU
 	help
Signature · 1 line
1.8.1.5
#1 lore raw

PATCH 1/1 nodejs: not supported on old ARM cores

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gustavo" == Gustavo Zacarias <[email protected]> writes:

Gustavo> nodejs needs the ARM blx instruction which is only supported in v5t+ Gustavo> cores (v5 ISA with thumb instructions).
Gustavo> Disable it for lower cores. Fixes:
Gustavo> http://autobuild.buildroot.net/results/89e/89ee5ba047a26a8c7a612d0285b08780b70efbd4/

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard