staging: sm750fb: change definition of DE_SOURCE fields
authorMike Rapoport <mike.rapoport@gmail.com>
Mon, 15 Feb 2016 17:53:57 +0000 (19:53 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 16 Feb 2016 02:26:17 +0000 (18:26 -0800)
Use stratight-forward definition of DE_SOURCE register fields
and use open-coded implementation for register manipulation

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/sm750fb/sm750_accel.c
drivers/staging/sm750fb/sm750_accel.h

index e150680a8cd14c496de4b6053e148ef2b09e6c08..da4eaea1e97509c18351bea3ebde6e0256ecba40 100644 (file)
@@ -256,9 +256,8 @@ unsigned int rop2)   /* ROP value */
        {
 
        write_dpr(accel, DE_SOURCE,
-                 FIELD_SET(0, DE_SOURCE, WRAP, DISABLE) |
-                 FIELD_VALUE(0, DE_SOURCE, X_K1, sx)   |
-                 FIELD_VALUE(0, DE_SOURCE, Y_K2, sy)); /* dpr0 */
+                 ((sx << DE_SOURCE_X_K1_SHIFT) & DE_SOURCE_X_K1_MASK) |
+                 (sy & DE_SOURCE_Y_K2_MASK)); /* dpr0 */
        write_dpr(accel, DE_DESTINATION,
                  FIELD_SET(0, DE_DESTINATION, WRAP, DISABLE) |
                  FIELD_VALUE(0, DE_DESTINATION, X,    dx)  |
@@ -354,8 +353,8 @@ int hw_imageblit(struct lynx_accel *accel,
         /* Note: For 2D Source in Host Write, only X_K1_MONO field is needed, and Y_K2 field is not used.
            For mono bitmap, use startBit for X_K1. */
        write_dpr(accel, DE_SOURCE,
-                 FIELD_SET(0, DE_SOURCE, WRAP, DISABLE)       |
-                 FIELD_VALUE(0, DE_SOURCE, X_K1_MONO, startBit)); /* dpr00 */
+                 (startBit << DE_SOURCE_X_K1_SHIFT) &
+                 DE_SOURCE_X_K1_MONO_MASK); /* dpr00 */
 
        write_dpr(accel, DE_DESTINATION,
                  FIELD_SET(0, DE_DESTINATION, WRAP, DISABLE) |
index 1ec66d28f456dfa947a13096f347721cbe0aaf13..0bbdccac448bfb8e862b1c9140b445a9dd6d6537 100644 (file)
 #define DE_PORT_ADDR_TYPE3 0x100000
 
 #define DE_SOURCE                                       0x0
-#define DE_SOURCE_WRAP                                  31:31
-#define DE_SOURCE_WRAP_DISABLE                          0
-#define DE_SOURCE_WRAP_ENABLE                           1
-#define DE_SOURCE_X_K1                                  29:16
-#define DE_SOURCE_Y_K2                                  15:0
-#define DE_SOURCE_X_K1_MONO                            20:16
+#define DE_SOURCE_WRAP                                  BIT(31)
+#define DE_SOURCE_X_K1_SHIFT                            16
+#define DE_SOURCE_X_K1_MASK                             (0x3fff << 16)
+#define DE_SOURCE_X_K1_MONO_MASK                       (0x1f << 16)
+#define DE_SOURCE_Y_K2_MASK                             0xffff
 
 #define DE_DESTINATION                                  0x4
 #define DE_DESTINATION_WRAP                             31:31