bfin: Refactor duplicated assembly-related macros
authorDiego Biurrun <diego@biurrun.de>
Wed, 22 Jan 2014 13:21:02 +0000 (05:21 -0800)
committerDiego Biurrun <diego@biurrun.de>
Thu, 13 Mar 2014 15:15:52 +0000 (08:15 -0700)
libavcodec/bfin/fdct_bfin.S
libavcodec/bfin/hpel_pixels_bfin.S
libavcodec/bfin/idct_bfin.S
libavcodec/bfin/pixels_bfin.S
libavcodec/bfin/vp3_idct_bfin.S
libavutil/bfin/asm.h [moved from libavcodec/bfin/config_bfin.h with 80% similarity]
libswscale/bfin/internal_bfin.S

index c923d76..2525e0d 100644 (file)
@@ -125,21 +125,16 @@ DCT BFINfdct: err_inf=2 err2=0.16425938 syserr=0.00795000 maxout=2098 blockSumEr
 DCT BFINfdct: 92.1 kdct/s
 */
 
-#include "config.h"
-#include "config_bfin.h"
-
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.B,"aw",@progbits
-#else
-.data
-#endif
+#include "libavutil/bfin/asm.h"
+
+SECTION_L1_DATA_B
+
 .align 4;
 dct_coeff:
 .short 0x5a82, 0x2d41, 0x187e, 0x3b21, 0x0c7c, 0x3ec5, 0x238e, 0x3537;
 
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.A,"aw",@progbits
-#endif
+SECTION_L1_DATA_A
+
 .align 4
 vtmp:   .space 128
 
index 8bed5c0..d311fdf 100644 (file)
@@ -19,7 +19,7 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include "config_bfin.h"
+#include "libavutil/bfin/asm.h"
 
 /*
   motion compensation
index 0eeffc2..dd427e7 100644 (file)
@@ -55,14 +55,9 @@ IDCT BFINidct: 88.3 kdct/s
 
 */
 
-#include "config.h"
-#include "config_bfin.h"
+#include "libavutil/bfin/asm.h"
 
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.B,"aw",@progbits
-#else
-.data
-#endif
+SECTION_L1_DATA_B
 
 .align 4;
 coefs:
@@ -77,9 +72,7 @@ coefs:
 .short 0x18F9;           //cos(7pi/16)
 .short 0x7D8A;           //cos(pi/16)
 
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.A,"aw",@progbits
-#endif
+SECTION_L1_DATA_A
 
 vtmp: .space 256
 
index 70d9de5..e19455b 100644 (file)
@@ -18,7 +18,8 @@
  * License along with Libav; if not, write to the Free Software
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
-#include "config_bfin.h"
+
+#include "libavutil/bfin/asm.h"
 
 DEFUN(put_pixels_clamped,mL1,
         (int16_t *block, uint8_t *dest, int line_size)):
index fec8d78..615cae9 100644 (file)
@@ -28,14 +28,9 @@ Registers Used  : A0, A1, R0-R7, I0-I3, B0, B2, B3, M0-M2, L0-L3, P0-P5, LC0.
 
 */
 
-#include "config.h"
-#include "config_bfin.h"
+#include "libavutil/bfin/asm.h"
 
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.B,"aw",@progbits
-#else
-.data
-#endif
+SECTION_L1_DATA_B
 
 .align 4;
 coefs:
@@ -50,9 +45,7 @@ coefs:
 .short 0x18F9;           //cos(7pi/16)
 .short 0x7D8A;           //cos(pi/16)
 
-#if defined(__FDPIC__) && CONFIG_SRAM
-.section .l1.data.A,"aw",@progbits
-#endif
+SECTION_L1_DATA_A
 
 vtmp: .space 256
 
similarity index 80%
rename from libavcodec/bfin/config_bfin.h
rename to libavutil/bfin/asm.h
index 63f9e32..6bfa0f3 100644 (file)
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-/*
- * low level assembler interface wrapper
- *
- * DEFUN(put_pixels_clamped,mL1,
- *      (int16_t *block, uint8_t *dest, int line_size)):
- *
- *    body
- *
- *    rts;
- */
-
-#ifndef AVCODEC_BFIN_CONFIG_BFIN_H
-#define AVCODEC_BFIN_CONFIG_BFIN_H
+#ifndef AVUTIL_BFIN_ASM_H
+#define AVUTIL_BFIN_ASM_H
 
 #include "config.h"
 
-#ifndef DEFUN
-
 #define mL3 .text
-#ifndef mL1
+
 #if defined(__FDPIC__) && CONFIG_SRAM
 #define mL1 .l1.text
+#define SECTION_L1_DATA_A .section .l1.data.A,"aw",@progbits
+#define SECTION_L1_DATA_B .section .l1.data.B,"aw",@progbits
 #else
 #define mL1 mL3
+#define SECTION_L1_DATA_A
+#define SECTION_L1_DATA_B .data
 #endif
-#endif /* mL1 */
 
 #define DEFUN(fname, where, interface)          \
     .section where;                             \
@@ -61,6 +51,4 @@
 #define RELOC(reg, got, obj) reg.L = obj; reg.H = obj
 #endif
 
-#endif /* DEFUN */
-
-#endif /* AVCODEC_BFIN_CONFIG_BFIN_H */
+#endif /* AVUTIL_BFIN_ASM_H */
index b007f07..dca8448 100644 (file)
@@ -93,23 +93,9 @@ huge variation on the reference codes on Blackfin I guess it must have
 to do with the memory system.
 */
 
-#define mL3 .text
-#if defined(__FDPIC__) && CONFIG_SRAM
-#define mL1 .l1.text
-#else
-#define mL1 mL3
-#endif
-#define MEM mL1
-
-#define DEFUN(fname,where,interface) \
-        .section where;              \
-        .global _ff_bfin_ ## fname;  \
-        .type _ff_bfin_ ## fname, STT_FUNC; \
-        .align 8;                    \
-        _ff_bfin_ ## fname
+#include "libavutil/bfin/asm.h"
 
-#define DEFUN_END(fname) \
-        .size _ff_bfin_ ## fname, . - _ff_bfin_ ## fname
+#define MEM mL1
 
 
 .text