Skip to contentSkip to conversation

mpg123: use code optimized for ARM NEON SIMD engine if available

PATCH New Sven Neumann 2 participants 2 messages Nov 22, 2013

lore
Share & tools
lore

Sven Neumann

author
#0 lore raw

PATCH 2/2 mpg123: use code optimized for ARM NEON SIMD engine if available

Signed-off-by Sven Neumann <[email protected]>
---
 package/mpg123/mpg123.mk | 4 ++++
 1 file changed, 4 insertions(+)
package/mpg123/mpg123.mk +4 -0
index 3db2843..0d5b1bc 100644
@@ -15,8 +15,12 @@ MPG123_LICENSE_FILES = COPYING
 MPG123_CPU = $(if $(BR2_SOFT_FLOAT),generic_nofpu,generic_fpu)
 
 ifeq ($(BR2_arm),y)
+ifeq ($(BR2_ARM_FPU_NEON),y)
+MPG123_CPU = neon
+else
 MPG123_CPU = arm_nofpu
 endif
+endif
 
 ifeq ($(BR2_i386),y)
 MPG123_CPU = x86
Signature · 1 line
1.8.4.2
#1 lore raw

PATCH 2/2 mpg123: use code optimized for ARM NEON SIMD engine if available

Quoted 12 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Sven" == Sven Neumann <[email protected]> writes:

Signed-off-by Sven Neumann <[email protected]>
---
 package/mpg123/mpg123.mk | 4 ++++
 1 file changed, 4 insertions(+)
package/mpg123/mpg123.mk +0 -0
index 3db2843..0d5b1bc 100644

package/mpg123/mpg123.mk MPG123_LICENSE_FILES = COPYING · line 20

Quoted 6 lines
package/mpg123/mpg123.mk +3 -0
@@ -15,8 +15,12 @@ MPG123_LICENSE_FILES = COPYING
 MPG123_CPU = $(if $(BR2_SOFT_FLOAT),generic_nofpu,generic_fpu)
 
 ifeq ($(BR2_arm),y)
+ifeq ($(BR2_ARM_FPU_NEON),y)
+MPG123_CPU = neon
+else

I don't know anything about the mpg123 code, but are you sure you need ARM_FPU_NEON (E.G. do all fpu calculation using neon) and not just BR2_ARM_CPU_HAS_NEON (E.G. cpu has neon support)?

Signature · 1 line
Bye, Peter Korsgaard