From 6d6eb0816ad38fe8d3f34706a1f26622fb7bc782 Mon Sep 17 00:00:00 2001 From: Marcoen Hirschberg Date: Sun, 31 Dec 2006 08:56:59 +0000 Subject: fix for the inline asm in CLIP on the gigabeat by Steve Gotthard (fs#6488). Turns out that gcc rearranged the ASM block without volatile git-svn-id: svn://svn.rockbox.org/rockbox/trunk@11873 a1c6a512-1295-4272-9138-f99709370657 --- apps/plugins/mpegplayer/idct.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'apps/plugins') diff --git a/apps/plugins/mpegplayer/idct.c b/apps/plugins/mpegplayer/idct.c index 372d18a6dc..7ad78bfcd9 100644 --- a/apps/plugins/mpegplayer/idct.c +++ b/apps/plugins/mpegplayer/idct.c @@ -60,11 +60,10 @@ static inline unsigned CLIP(int value) ); return value; } -/* FIXME why doesn't this work on the gigabeat? */ -#elif defined CPU_ARM && CONFIG_CPU != S3C2440 +#elif defined CPU_ARM static inline unsigned CLIP(int value) { - asm ( /* Note: Uses knowledge that only the low byte of the result is used */ + asm volatile ( /* Note: Uses knowledge that only the low byte of the result is used */ "cmp %[v], #255 \n" "mvnhi %[v], %[v], asr #31 \n" : /* outputs */ @@ -302,7 +301,7 @@ void mpeg2_idct_init (uint32_t accel) mpeg2_idct_copy = mpeg2_idct_copy_c; mpeg2_idct_add = mpeg2_idct_add_c; -#if !defined(CPU_COLDFIRE) && !defined(CPU_ARM) || CONFIG_CPU == S3C2440 +#if !defined(CPU_COLDFIRE) && !defined(CPU_ARM) for (i = -3840; i < 3840 + 256; i++) CLIP(i) = (i < 0) ? 0 : ((i > 255) ? 255 : i); #endif -- cgit v1.2.3