Clean up av_get_cpu_flag()

Instead of defining functions in per-arch header files included
by the main cpu.c, define them normally and call them from the
generic one.

Originally committed as revision 25084 to svn://svn.ffmpeg.org/ffmpeg/trunk

6 files changed, 17 insertions, 28 deletions

diff --git a/libavutil/Makefile b/libavutil/Makefile
index 545e1c23aa..22b5452f55 100644
--- a/libavutil/Makefile
+++ b/libavutil/Makefile
@@ -57,6 +57,10 @@ OBJS = adler32.o                                                        \
        tree.o                                                           \
        utils.o                                                          \
 
+OBJS-$(ARCH_ARM) += arm/cpu.o
+OBJS-$(ARCH_PPC) += ppc/cpu.o
+OBJS-$(ARCH_X86) += x86/cpu.o
+
 TESTPROGS = adler32 aes base64 cpu crc des lls md5 pca sha softfloat tree
 TESTPROGS-$(HAVE_LZO1X_999_COMPRESS) += lzo
 
diff --git a/libavutil/arm/cpu.h b/libavutil/arm/cpu.c
index 45ab761406..742c3e498d 100644
--- a/libavutil/arm/cpu.h
+++ b/libavutil/arm/cpu.c
@@ -16,14 +16,10 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#ifndef AVUTIL_ARM_CPU_H
-#define AVUTIL_ARM_CPU_H
-
+#include "libavutil/cpu.h"
 #include "config.h"
 
-int av_get_cpu_flags(void)
+int ff_get_cpu_flags_arm(void)
 {
     return HAVE_IWMMXT * AV_CPU_FLAG_IWMMXT;
 }
-
-#endif /* AVUTIL_ARM_CPU_H */
diff --git a/libavutil/cpu.c b/libavutil/cpu.c
index fb37edb879..34530eebaf 100644
--- a/libavutil/cpu.c
+++ b/libavutil/cpu.c
@@ -19,18 +19,13 @@
 #include "cpu.h"
 #include "config.h"
 
-#if   ARCH_ARM
-#   include "arm/cpu.h"
-#elif ARCH_PPC
-#   include "ppc/cpu.h"
-#elif ARCH_X86
-#   include "x86/cpu.h"
-#else
 int av_get_cpu_flags(void)
 {
+    if (ARCH_ARM) return ff_get_cpu_flags_arm();
+    if (ARCH_PPC) return ff_get_cpu_flags_ppc();
+    if (ARCH_X86) return ff_get_cpu_flags_x86();
     return 0;
 }
-#endif
 
 #ifdef TEST
 
diff --git a/libavutil/cpu.h b/libavutil/cpu.h
index 08e82f1d04..71cc26529a 100644
--- a/libavutil/cpu.h
+++ b/libavutil/cpu.h
@@ -21,8 +21,6 @@
 #ifndef AVUTIL_CPU_H
 #define AVUTIL_CPU_H
 
-#include "avutil.h"
-
 #define AV_CPU_FLAG_FORCE    0x80000000 /* force usage of selected flags (OR) */
 
     /* lower 16 bits - CPU features */
@@ -46,4 +44,9 @@
  */
 int av_get_cpu_flags(void);
 
+/* The following CPU-specific functions shall not be called directly. */
+int ff_get_cpu_flags_arm(void);
+int ff_get_cpu_flags_ppc(void);
+int ff_get_cpu_flags_x86(void);
+
 #endif  /* AVUTIL_CPU_H */
diff --git a/libavutil/ppc/cpu.h b/libavutil/ppc/cpu.c
index 684361c5ff..9dc9471b3e 100644
--- a/libavutil/ppc/cpu.h
+++ b/libavutil/ppc/cpu.c
@@ -16,9 +16,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#ifndef AVUTIL_PPC_CPU_H
-#define AVUTIL_PPC_CPU_H
-
 #ifdef __APPLE__
 #undef _POSIX_C_SOURCE
 #include <sys/sysctl.h>
@@ -32,13 +29,14 @@
 #include <proto/exec.h>
 #endif /* __APPLE__ */
 
+#include "libavutil/cpu.h"
 #include "config.h"
 
 /**
  * This function MAY rely on signal() or fork() in order to make sure AltiVec
  * is present.
  */
-int av_get_cpu_flags(void)
+int ff_get_cpu_flags_ppc(void)
 {
 #if HAVE_ALTIVEC
 #ifdef __AMIGAOS4__
@@ -84,5 +82,3 @@ int av_get_cpu_flags(void)
 #endif /* HAVE_ALTIVEC */
     return 0;
 }
-
-#endif /* AVUTIL_PPC_CPU_H */
diff --git a/libavutil/x86/cpu.h b/libavutil/x86/cpu.c
index 052d2c5477..4df3286f39 100644
--- a/libavutil/x86/cpu.h
+++ b/libavutil/x86/cpu.c
@@ -20,9 +20,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#ifndef AVUTIL_X86_CPU_H2
-#define AVUTIL_X86_CPU_H2
-
 #include <stdlib.h>
 #include "libavutil/x86_cpu.h"
 #include "libavutil/cpu.h"
@@ -38,7 +35,7 @@
          : "0" (index));
 
 /* Function to test if multimedia instructions are supported...  */
-int av_get_cpu_flags(void)
+int ff_get_cpu_flags_x86(void)
 {
     int rval = 0;
     int eax, ebx, ecx, edx;
@@ -124,5 +121,3 @@ int av_get_cpu_flags(void)
 
     return rval;
 }
-
-#endif /* AVUTIL_X86_CPU_H2 */