Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 302073 - [PATCH] media-video/mplayer fails to build if -mfpu=neon is set on ARM
Summary: [PATCH] media-video/mplayer fails to build if -mfpu=neon is set on ARM
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: ARM Linux
: High normal (vote)
Assignee: Gentoo Media-video project
URL: http://archives.free.net.ph/message/2...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-24 15:06 UTC by Raúl Porcel (RETIRED)
Modified: 2010-01-31 17:58 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Raúl Porcel (RETIRED) gentoo-dev 2010-01-24 15:06:43 UTC 
armv7a-unknown-linux-gnueabi-gcc -DHAVE_AV_CONFIG_H -I.. -I..    -c -o arm/dsputil_neon.o arm/dsputil_neon.S
arm/dsputil_neon.S: Assembler messages:
arm/dsputil_neon.S:255: Error: bad instruction `vld1.64 {d0,d1},[r1],r2'
arm/dsputil_neon.S:255: Error: bad instruction `vld1.64 {d2,d3},[r1],r2'
arm/dsputil_neon.S:255: Error: bad instruction `vld1.64 {d4,d5},[r1],r2'

etc etc...


mplayer enables neon source code building if -mfpu=neon is on CFLAGS, but then our compiler is unable to compile neon code if -mfpu=neon isn't on *FLAGS(AS,CXX,C,CPP)...upstream fixed this some days ago.

Can we please get this patch applied to mplayer-20091026 and  and 20091124 so i can keyword it ~arm? Needless to say 9999 works fine.

Thanks
Comment 1 Reimar Döffinger 2010-01-26 19:19:05 UTC 
Just wanted to say, feel free to contact me directly if you have issues with MPlayer on ARM, I have only tried cross-compiling without actually running anything.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2010-01-31 17:58:40 UTC 
+  31 Jan 2010; Samuli Suominen <ssuominen@gentoo.org>
+  mplayer-1.0_rc4_p20091026-r1.ebuild,
+  +files/mplayer-1.0_rc4_p20091026-arm_neon.patch:
+  Fix building with ARM NEON wrt #302073 by Raúl Porcel.

Untested.   I hope it's fine for this version.

Skipped mplayer-1.0_rc4_p20091124-r1 on purpose, I doubt this specific version will ever be unmasked.