[MPlayer-dev-eng] [PATCH] Optimized libmpeg2 routines for ARM/Xscale
Siarhei Siamashka
siarhei.siamashka at gmail.com
Mon May 7 21:53:13 CEST 2007
On Monday 07 May 2007 15:17, Guillaume POIRIER wrote:
> On 5/5/07, Guillaume POIRIER <poirierg at gmail.com> wrote:
> > Hi there,
> >
> > Attached patch features $SUBJ. It's just a re post of a patch posted
> > by David Bateman while I was on hollidays, but never made it to SVN.
> > It was advised to send these changes to libmpeg2 team directly, but
> > since this project hasn't even acknowledged the message, I plan to
> > apply attached patch in a few days.
>
> No objection? I'll apply it tonite unless there's ojection since then.
This patch breaks compilation:
motion_comp.c: In function `mpeg2_mc_init':
motion_comp.c:71: error: `MPEG2_ACCEL_ARM' undeclared (first use in this
function)
motion_comp.c:71: error: (Each undeclared identifier is reported only once
motion_comp.c:71: error: for each function it appears in.)
These MPEG2_ACCEL_ARM and MPEG2_ACCEL_ARM_IWMMXT are not
defined anywhere.
In addition, there is a problem with PLD (cache prefetch) instruction, it is
only available on ARMv5TE, but this code is also getting compiled in when
configured for ARMv4. The same problem exists in ffmpeg as well, and it
was raised on ffmpeg-devel mailing list recently.
Other than that, I don't have any ARM device with IWMMXT support to test (only
have Nokia 770 and N800 here), so can't provide further feedback regarding
IWMMXT code.
Would you like to have ARMv5 optimizations [1] merged in MPlayer as well?
I was not sure if it would be a good idea to submit them to MPlayer instead of
upstream libmpeg2, so did not try that earlier.
1.
https://garage.maemo.org/plugins/scmsvn/viewcvs.php/trunk/libmpeg2/?root=mplayer
More information about the MPlayer-dev-eng
mailing list