Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 2/2] mpg123: use code optimized for ARM NEON SIMD engine if available
@ 2013-11-22 22:00 Sven Neumann
  2013-11-22 22:37 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Sven Neumann @ 2013-11-22 22:00 UTC (permalink / raw)
  To: buildroot


Signed-off-by: Sven Neumann <neumann@teufel.de>
---
 package/mpg123/mpg123.mk | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/mpg123/mpg123.mk b/package/mpg123/mpg123.mk
index 3db2843..0d5b1bc 100644
--- a/package/mpg123/mpg123.mk
+++ b/package/mpg123/mpg123.mk
@@ -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
-- 
1.8.4.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH 2/2] mpg123: use code optimized for ARM NEON SIMD engine if available
  2013-11-22 22:00 [Buildroot] [PATCH 2/2] mpg123: use code optimized for ARM NEON SIMD engine if available Sven Neumann
@ 2013-11-22 22:37 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2013-11-22 22:37 UTC (permalink / raw)
  To: buildroot

>>>>> "Sven" == Sven Neumann <neumann@teufel.de> writes:

 > Signed-off-by: Sven Neumann <neumann@teufel.de>
 > ---
 >  package/mpg123/mpg123.mk | 4 ++++
 >  1 file changed, 4 insertions(+)

 > diff --git a/package/mpg123/mpg123.mk b/package/mpg123/mpg123.mk
 > index 3db2843..0d5b1bc 100644
 > --- a/package/mpg123/mpg123.mk
 > +++ b/package/mpg123/mpg123.mk
 > @@ -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)?

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-11-22 22:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-22 22:00 [Buildroot] [PATCH 2/2] mpg123: use code optimized for ARM NEON SIMD engine if available Sven Neumann
2013-11-22 22:37 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox