From a519583991c38d38503ab08357716513facc5725 Mon Sep 17 00:00:00 2001 From: Diego Biurrun Date: Fri, 12 Oct 2012 19:54:45 +0200 Subject: [PATCH] swscale: x86: Hide arch-specific initialization details Also give consistent names to init functions. --- libswscale/rgb2rgb.c | 2 +- libswscale/swscale.c | 4 ++-- libswscale/swscale_internal.h | 4 ++-- libswscale/x86/Makefile | 6 +++--- libswscale/x86/swscale.c | 10 +++++----- libswscale/x86/swscale_template.c | 2 +- libswscale/x86/yuv2rgb.c | 6 +++--- libswscale/yuv2rgb.c | 4 ++-- 8 files changed, 19 insertions(+), 19 deletions(-) diff --git a/libswscale/rgb2rgb.c b/libswscale/rgb2rgb.c index c261fe6ec1..21adfe11d2 100644 --- a/libswscale/rgb2rgb.c +++ b/libswscale/rgb2rgb.c @@ -129,7 +129,7 @@ void (*yuyvtoyuv422)(uint8_t *ydst, uint8_t *udst, uint8_t *vdst, av_cold void sws_rgb2rgb_init(void) { rgb2rgb_init_c(); - if (HAVE_MMX) + if (ARCH_X86) rgb2rgb_init_x86(); } diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 3a63fbe789..8b2090e761 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -772,8 +772,8 @@ SwsFunc ff_getSwsFunc(SwsContext *c) { sws_init_swScale_c(c); - if (HAVE_MMX) - ff_sws_init_swScale_mmx(c); + if (ARCH_X86) + ff_sws_init_swscale_x86(c); if (HAVE_ALTIVEC) ff_sws_init_swScale_altivec(c); diff --git a/libswscale/swscale_internal.h b/libswscale/swscale_internal.h index e8245b90d5..d9e300be1e 100644 --- a/libswscale/swscale_internal.h +++ b/libswscale/swscale_internal.h @@ -570,7 +570,7 @@ void ff_yuv2rgb_init_tables_altivec(SwsContext *c, const int inv_table[4], void updateMMXDitherTables(SwsContext *c, int dstY, int lumBufIndex, int chrBufIndex, int lastInLumBuf, int lastInChrBuf); -SwsFunc ff_yuv2rgb_init_mmx(SwsContext *c); +SwsFunc ff_yuv2rgb_init_x86(SwsContext *c); SwsFunc ff_yuv2rgb_init_vis(SwsContext *c); SwsFunc ff_yuv2rgb_init_altivec(SwsContext *c); SwsFunc ff_yuv2rgb_get_func_ptr_bfin(SwsContext *c); @@ -754,6 +754,6 @@ void ff_sws_init_output_funcs(SwsContext *c, yuv2packedX_fn *yuv2packedX, yuv2anyX_fn *yuv2anyX); void ff_sws_init_swScale_altivec(SwsContext *c); -void ff_sws_init_swScale_mmx(SwsContext *c); +void ff_sws_init_swscale_x86(SwsContext *c); #endif /* SWSCALE_SWSCALE_INTERNAL_H */ diff --git a/libswscale/x86/Makefile b/libswscale/x86/Makefile index 5416d48a4c..b94b14abbb 100644 --- a/libswscale/x86/Makefile +++ b/libswscale/x86/Makefile @@ -1,9 +1,9 @@ -OBJS-$(CONFIG_XMM_CLOBBER_TEST) += x86/w64xmmtest.o - -MMX-OBJS += x86/rgb2rgb.o \ +OBJS += x86/rgb2rgb.o \ x86/swscale.o \ x86/yuv2rgb.o \ +OBJS-$(CONFIG_XMM_CLOBBER_TEST) += x86/w64xmmtest.o + YASM-OBJS += x86/input.o \ x86/output.o \ x86/scale.o \ diff --git a/libswscale/x86/swscale.c b/libswscale/x86/swscale.c index 059589f681..0b5f870272 100644 --- a/libswscale/x86/swscale.c +++ b/libswscale/x86/swscale.c @@ -302,18 +302,18 @@ INPUT_FUNCS(sse2); INPUT_FUNCS(ssse3); INPUT_FUNCS(avx); -av_cold void ff_sws_init_swScale_mmx(SwsContext *c) +av_cold void ff_sws_init_swscale_x86(SwsContext *c) { int cpu_flags = av_get_cpu_flags(); -#if HAVE_INLINE_ASM +#if HAVE_MMX_INLINE if (cpu_flags & AV_CPU_FLAG_MMX) - sws_init_swScale_MMX(c); + sws_init_swscale_MMX(c); +#endif #if HAVE_MMXEXT_INLINE if (cpu_flags & AV_CPU_FLAG_MMXEXT) - sws_init_swScale_MMXEXT(c); + sws_init_swscale_MMXEXT(c); #endif -#endif /* HAVE_INLINE_ASM */ #define ASSIGN_SCALE_FUNC2(hscalefn, filtersize, opt1, opt2) do { \ if (c->srcBpc == 8) { \ diff --git a/libswscale/x86/swscale_template.c b/libswscale/x86/swscale_template.c index d89a26f6a5..80a3ad9a3d 100644 --- a/libswscale/x86/swscale_template.c +++ b/libswscale/x86/swscale_template.c @@ -1555,7 +1555,7 @@ static void RENAME(hcscale_fast)(SwsContext *c, int16_t *dst1, int16_t *dst2, } #endif /* COMPILE_TEMPLATE_MMXEXT */ -static av_cold void RENAME(sws_init_swScale)(SwsContext *c) +static av_cold void RENAME(sws_init_swscale)(SwsContext *c) { enum AVPixelFormat dstFormat = c->dstFormat; diff --git a/libswscale/x86/yuv2rgb.c b/libswscale/x86/yuv2rgb.c index 419d5133f9..9968a8d49d 100644 --- a/libswscale/x86/yuv2rgb.c +++ b/libswscale/x86/yuv2rgb.c @@ -69,9 +69,9 @@ DECLARE_ASM_CONST(8, uint64_t, pb_07) = 0x0707070707070707ULL; #endif /* HAVE_INLINE_ASM */ -av_cold SwsFunc ff_yuv2rgb_init_mmx(SwsContext *c) +av_cold SwsFunc ff_yuv2rgb_init_x86(SwsContext *c) { -#if HAVE_INLINE_ASM +#if HAVE_MMX_INLINE int cpu_flags = av_get_cpu_flags(); if (c->srcFormat != AV_PIX_FMT_YUV420P && @@ -111,7 +111,7 @@ av_cold SwsFunc ff_yuv2rgb_init_mmx(SwsContext *c) case AV_PIX_FMT_RGB555: return yuv420_rgb15_MMX; } } -#endif /* HAVE_INLINE_ASM */ +#endif /* HAVE_MMX_INLINE */ return NULL; } diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index b51044606b..7e17fcc0e3 100644 --- a/libswscale/yuv2rgb.c +++ b/libswscale/yuv2rgb.c @@ -560,8 +560,8 @@ SwsFunc ff_yuv2rgb_get_func_ptr(SwsContext *c) { SwsFunc t = NULL; - if (HAVE_MMX) - t = ff_yuv2rgb_init_mmx(c); + if (ARCH_X86) + t = ff_yuv2rgb_init_x86(c); else if (HAVE_VIS) t = ff_yuv2rgb_init_vis(c); else if (HAVE_ALTIVEC)