[FFmpeg-devel] [PATCH] Fix mm_flags, mm_support for ARM

[Siarhei Siamashka]

On Friday 27 June 2008, matthieu castet wrote:
> Benoit Fouet wrote:
> > Michael Niedermayer wrote:
> >> On Thu, Jun 26, 2008 at 10:45:20PM +0200, matthieu castet wrote:
> >
> > Matthieu, could you please resend the complete patch ?
>
> I attach them as 2 separate patch.
> fix_dct-test-arm.diff : allow to build dct-test on arm
> arm-icdt-test.diff : add some arm idct (there are other version that
> could be added, but I have no hardware to test).

> +#ifdef ARCH_ARMV4L
> + ?//{"IDCT_ARM", ? ? ? ?1, j_rev_dct_ARM, ? ? ?idct, MMX_PERM},
> /* should > be MMX_PERM perm, but produce strange result */ 

Yes, 'j_rev_dct_ARM' is broken, it produces quite noticeable artefacts on
decoding video. I think I already reported this issue here long ago. But as it
is armv4 only, I was not interested in investigating this problem further...

-- 
Best regards,
Siarhei Siamashka