staging: comedi: ni_tio_internal.h: tidy up bit defines
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Wed, 23 Mar 2016 22:36:37 +0000 (15:36 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
Fix the checkpatch.pl issues:
CHECK: Prefer using the BIT macro

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/ni_tio_internal.h

index 2bceae4..c51e01e 100644 (file)
 #define NITIO_AUTO_INC_REG(x)          (NITIO_G0_AUTO_INC + (x))
 #define GI_AUTO_INC_MASK               0xff
 #define NITIO_CMD_REG(x)               (NITIO_G0_CMD + (x))
-#define GI_ARM                         (1 << 0)
-#define GI_SAVE_TRACE                  (1 << 1)
-#define GI_LOAD                                (1 << 2)
-#define GI_DISARM                      (1 << 4)
+#define GI_ARM                         BIT(0)
+#define GI_SAVE_TRACE                  BIT(1)
+#define GI_LOAD                                BIT(2)
+#define GI_DISARM                      BIT(4)
 #define GI_CNT_DIR(x)                  (((x) & 0x3) << 5)
-#define GI_CNT_DIR_MASK                        (3 << 5)
-#define GI_WRITE_SWITCH                        (1 << 7)
-#define GI_SYNC_GATE                   (1 << 8)
-#define GI_LITTLE_BIG_ENDIAN           (1 << 9)
-#define GI_BANK_SWITCH_START           (1 << 10)
-#define GI_BANK_SWITCH_MODE            (1 << 11)
-#define GI_BANK_SWITCH_ENABLE          (1 << 12)
-#define GI_ARM_COPY                    (1 << 13)
-#define GI_SAVE_TRACE_COPY             (1 << 14)
-#define GI_DISARM_COPY                 (1 << 15)
+#define GI_CNT_DIR_MASK                        GI_CNT_DIR(3)
+#define GI_WRITE_SWITCH                        BIT(7)
+#define GI_SYNC_GATE                   BIT(8)
+#define GI_LITTLE_BIG_ENDIAN           BIT(9)
+#define GI_BANK_SWITCH_START           BIT(10)
+#define GI_BANK_SWITCH_MODE            BIT(11)
+#define GI_BANK_SWITCH_ENABLE          BIT(12)
+#define GI_ARM_COPY                    BIT(13)
+#define GI_SAVE_TRACE_COPY             BIT(14)
+#define GI_DISARM_COPY                 BIT(15)
 #define NITIO_HW_SAVE_REG(x)           (NITIO_G0_HW_SAVE + (x))
 #define NITIO_SW_SAVE_REG(x)           (NITIO_G0_SW_SAVE + (x))
 #define NITIO_MODE_REG(x)              (NITIO_G0_MODE + (x))
-#define GI_GATING_DISABLED             (0 << 0)
-#define GI_LEVEL_GATING                        (1 << 0)
-#define GI_RISING_EDGE_GATING          (2 << 0)
-#define GI_FALLING_EDGE_GATING         (3 << 0)
-#define GI_GATING_MODE_MASK            (3 << 0)
-#define GI_GATE_ON_BOTH_EDGES          (1 << 2)
-#define GI_EDGE_GATE_STARTS_STOPS      (0 << 3)
-#define GI_EDGE_GATE_STOPS_STARTS      (1 << 3)
-#define GI_EDGE_GATE_STARTS            (2 << 3)
-#define GI_EDGE_GATE_NO_STARTS_OR_STOPS        (3 << 3)
-#define GI_EDGE_GATE_MODE_MASK         (3 << 3)
-#define GI_STOP_ON_GATE                        (0 << 5)
-#define GI_STOP_ON_GATE_OR_TC          (1 << 5)
-#define GI_STOP_ON_GATE_OR_SECOND_TC   (2 << 5)
-#define GI_STOP_MODE_MASK              (3 << 5)
-#define GI_LOAD_SRC_SEL                        (1 << 7)
-#define GI_OUTPUT_TC_PULSE             (1 << 8)
-#define GI_OUTPUT_TC_TOGGLE            (2 << 8)
-#define GI_OUTPUT_TC_OR_GATE_TOGGLE    (3 << 8)
-#define GI_OUTPUT_MODE_MASK            (3 << 8)
-#define GI_NO_HARDWARE_DISARM          (0 << 10)
-#define GI_DISARM_AT_TC                        (1 << 10)
-#define GI_DISARM_AT_GATE              (2 << 10)
-#define GI_DISARM_AT_TC_OR_GATE                (3 << 10)
-#define GI_COUNTING_ONCE_MASK          (3 << 10)
-#define GI_LOADING_ON_TC               (1 << 12)
-#define GI_GATE_POL_INVERT             (1 << 13)
-#define GI_LOADING_ON_GATE             (1 << 14)
-#define GI_RELOAD_SRC_SWITCHING                (1 << 15)
+#define GI_GATING_MODE(x)              (((x) & 0x3) << 0)
+#define GI_GATING_DISABLED             GI_GATING_MODE(0)
+#define GI_LEVEL_GATING                        GI_GATING_MODE(1)
+#define GI_RISING_EDGE_GATING          GI_GATING_MODE(2)
+#define GI_FALLING_EDGE_GATING         GI_GATING_MODE(3)
+#define GI_GATING_MODE_MASK            GI_GATING_MODE(3)
+#define GI_GATE_ON_BOTH_EDGES          BIT(2)
+#define GI_EDGE_GATE_MODE(x)           (((x) & 0x3) << 3)
+#define GI_EDGE_GATE_STARTS_STOPS      GI_EDGE_GATE_MODE(0)
+#define GI_EDGE_GATE_STOPS_STARTS      GI_EDGE_GATE_MODE(1)
+#define GI_EDGE_GATE_STARTS            GI_EDGE_GATE_MODE(2)
+#define GI_EDGE_GATE_NO_STARTS_OR_STOPS        GI_EDGE_GATE_MODE(3)
+#define GI_EDGE_GATE_MODE_MASK         GI_EDGE_GATE_MODE(3)
+#define GI_STOP_MODE(x)                        (((x) & 0x3) << 5)
+#define GI_STOP_ON_GATE                        GI_STOP_MODE(0)
+#define GI_STOP_ON_GATE_OR_TC          GI_STOP_MODE(1)
+#define GI_STOP_ON_GATE_OR_SECOND_TC   GI_STOP_MODE(2)
+#define GI_STOP_MODE_MASK              GI_STOP_MODE(3)
+#define GI_LOAD_SRC_SEL                        BIT(7)
+#define GI_OUTPUT_MODE(x)              (((x) & 0x3) << 8)
+#define GI_OUTPUT_TC_PULSE             GI_OUTPUT_MODE(1)
+#define GI_OUTPUT_TC_TOGGLE            GI_OUTPUT_MODE(2)
+#define GI_OUTPUT_TC_OR_GATE_TOGGLE    GI_OUTPUT_MODE(3)
+#define GI_OUTPUT_MODE_MASK            GI_OUTPUT_MODE(3)
+#define GI_COUNTING_ONCE(x)            (((x) & 0x3) << 10)
+#define GI_NO_HARDWARE_DISARM          GI_COUNTING_ONCE(0)
+#define GI_DISARM_AT_TC                        GI_COUNTING_ONCE(1)
+#define GI_DISARM_AT_GATE              GI_COUNTING_ONCE(2)
+#define GI_DISARM_AT_TC_OR_GATE                GI_COUNTING_ONCE(3)
+#define GI_COUNTING_ONCE_MASK          GI_COUNTING_ONCE(3)
+#define GI_LOADING_ON_TC               BIT(12)
+#define GI_GATE_POL_INVERT             BIT(13)
+#define GI_LOADING_ON_GATE             BIT(14)
+#define GI_RELOAD_SRC_SWITCHING                BIT(15)
 #define NITIO_LOADA_REG(x)             (NITIO_G0_LOADA + (x))
 #define NITIO_LOADB_REG(x)             (NITIO_G0_LOADB + (x))
 #define NITIO_INPUT_SEL_REG(x)         (NITIO_G0_INPUT_SEL + (x))
-#define GI_READ_ACKS_IRQ               (1 << 0)
-#define GI_WRITE_ACKS_IRQ              (1 << 1)
+#define GI_READ_ACKS_IRQ               BIT(0)
+#define GI_WRITE_ACKS_IRQ              BIT(1)
 #define GI_BITS_TO_SRC(x)              (((x) >> 2) & 0x1f)
 #define GI_SRC_SEL(x)                  (((x) & 0x1f) << 2)
-#define GI_SRC_SEL_MASK                        (0x1f << 2)
+#define GI_SRC_SEL_MASK                        GI_SRC_SEL(0x1f)
 #define GI_BITS_TO_GATE(x)             (((x) >> 7) & 0x1f)
 #define GI_GATE_SEL(x)                 (((x) & 0x1f) << 7)
-#define GI_GATE_SEL_MASK               (0x1f << 7)
-#define GI_GATE_SEL_LOAD_SRC           (1 << 12)
-#define GI_OR_GATE                     (1 << 13)
-#define GI_OUTPUT_POL_INVERT           (1 << 14)
-#define GI_SRC_POL_INVERT              (1 << 15)
+#define GI_GATE_SEL_MASK               GI_GATE_SEL(0x1f)
+#define GI_GATE_SEL_LOAD_SRC           BIT(12)
+#define GI_OR_GATE                     BIT(13)
+#define GI_OUTPUT_POL_INVERT           BIT(14)
+#define GI_SRC_POL_INVERT              BIT(15)
 #define NITIO_CNT_MODE_REG(x)          (NITIO_G0_CNT_MODE + (x))
 #define GI_CNT_MODE(x)                 (((x) & 0x7) << 0)
 #define GI_CNT_MODE_NORMAL             GI_CNT_MODE(0)
 #define GI_CNT_MODE_QUADX4             GI_CNT_MODE(3)
 #define GI_CNT_MODE_TWO_PULSE          GI_CNT_MODE(4)
 #define GI_CNT_MODE_SYNC_SRC           GI_CNT_MODE(6)
-#define GI_CNT_MODE_MASK               (7 << 0)
-#define GI_INDEX_MODE                  (1 << 4)
+#define GI_CNT_MODE_MASK               GI_CNT_MODE(7)
+#define GI_INDEX_MODE                  BIT(4)
 #define GI_INDEX_PHASE(x)              (((x) & 0x3) << 5)
-#define GI_INDEX_PHASE_MASK            (3 << 5)
-#define GI_HW_ARM_ENA                  (1 << 7)
+#define GI_INDEX_PHASE_MASK            GI_INDEX_PHASE(3)
+#define GI_HW_ARM_ENA                  BIT(7)
 #define GI_HW_ARM_SEL(x)               ((x) << 8)
-#define GI_660X_HW_ARM_SEL_MASK                (0x7 << 8)
-#define GI_M_HW_ARM_SEL_MASK           (0x1f << 8)
-#define GI_660X_PRESCALE_X8            (1 << 12)
-#define GI_M_PRESCALE_X8               (1 << 13)
-#define GI_660X_ALT_SYNC               (1 << 13)
-#define GI_M_ALT_SYNC                  (1 << 14)
-#define GI_660X_PRESCALE_X2            (1 << 14)
-#define GI_M_PRESCALE_X2               (1 << 15)
+#define GI_660X_HW_ARM_SEL_MASK                GI_HW_ARM_SEL(0x7)
+#define GI_M_HW_ARM_SEL_MASK           GI_HW_ARM_SEL(0x1f)
+#define GI_660X_PRESCALE_X8            BIT(12)
+#define GI_M_PRESCALE_X8               BIT(13)
+#define GI_660X_ALT_SYNC               BIT(13)
+#define GI_M_ALT_SYNC                  BIT(14)
+#define GI_660X_PRESCALE_X2            BIT(14)
+#define GI_M_PRESCALE_X2               BIT(15)
 #define NITIO_GATE2_REG(x)             (NITIO_G0_GATE2 + (x))
-#define GI_GATE2_MODE                  (1 << 0)
+#define GI_GATE2_MODE                  BIT(0)
 #define GI_BITS_TO_GATE2(x)            (((x) >> 7) & 0x1f)
 #define GI_GATE2_SEL(x)                        (((x) & 0x1f) << 7)
-#define GI_GATE2_SEL_MASK              (0x1f << 7)
-#define GI_GATE2_POL_INVERT            (1 << 13)
-#define GI_GATE2_SUBSEL                        (1 << 14)
-#define GI_SRC_SUBSEL                  (1 << 15)
+#define GI_GATE2_SEL_MASK              GI_GATE2_SEL(0x1f)
+#define GI_GATE2_POL_INVERT            BIT(13)
+#define GI_GATE2_SUBSEL                        BIT(14)
+#define GI_SRC_SUBSEL                  BIT(15)
 #define NITIO_SHARED_STATUS_REG(x)     (NITIO_G01_STATUS + ((x) / 2))
-#define GI_SAVE(x)                     (((x) % 2) ? (1 << 1) : (1 << 0))
-#define GI_COUNTING(x)                 (((x) % 2) ? (1 << 3) : (1 << 2))
-#define GI_NEXT_LOAD_SRC(x)            (((x) % 2) ? (1 << 5) : (1 << 4))
-#define GI_STALE_DATA(x)               (((x) % 2) ? (1 << 7) : (1 << 6))
-#define GI_ARMED(x)                    (((x) % 2) ? (1 << 9) : (1 << 8))
-#define GI_NO_LOAD_BETWEEN_GATES(x)    (((x) % 2) ? (1 << 11) : (1 << 10))
-#define GI_TC_ERROR(x)                 (((x) % 2) ? (1 << 13) : (1 << 12))
-#define GI_GATE_ERROR(x)               (((x) % 2) ? (1 << 15) : (1 << 14))
+#define GI_SAVE(x)                     (((x) % 2) ? BIT(1) : BIT(0))
+#define GI_COUNTING(x)                 (((x) % 2) ? BIT(3) : BIT(2))
+#define GI_NEXT_LOAD_SRC(x)            (((x) % 2) ? BIT(5) : BIT(4))
+#define GI_STALE_DATA(x)               (((x) % 2) ? BIT(7) : BIT(6))
+#define GI_ARMED(x)                    (((x) % 2) ? BIT(9) : BIT(8))
+#define GI_NO_LOAD_BETWEEN_GATES(x)    (((x) % 2) ? BIT(11) : BIT(10))
+#define GI_TC_ERROR(x)                 (((x) % 2) ? BIT(13) : BIT(12))
+#define GI_GATE_ERROR(x)               (((x) % 2) ? BIT(15) : BIT(14))
 #define NITIO_RESET_REG(x)             (NITIO_G01_RESET + ((x) / 2))
-#define GI_RESET(x)                    (1 << (2 + ((x) % 2)))
+#define GI_RESET(x)                    BIT(2 + ((x) % 2))
 #define NITIO_STATUS1_REG(x)           (NITIO_G01_STATUS1 + ((x) / 2))
 #define NITIO_STATUS2_REG(x)           (NITIO_G01_STATUS2 + ((x) / 2))
-#define GI_OUTPUT(x)                   (((x) % 2) ? (1 << 1) : (1 << 0))
-#define GI_HW_SAVE(x)                  (((x) % 2) ? (1 << 13) : (1 << 12))
-#define GI_PERMANENT_STALE(x)          (((x) % 2) ? (1 << 15) : (1 << 14))
+#define GI_OUTPUT(x)                   (((x) % 2) ? BIT(1) : BIT(0))
+#define GI_HW_SAVE(x)                  (((x) % 2) ? BIT(13) : BIT(12))
+#define GI_PERMANENT_STALE(x)          (((x) % 2) ? BIT(15) : BIT(14))
 #define NITIO_DMA_CFG_REG(x)           (NITIO_G0_DMA_CFG + (x))
-#define GI_DMA_ENABLE                  (1 << 0)
-#define GI_DMA_WRITE                   (1 << 1)
-#define GI_DMA_INT_ENA                 (1 << 2)
-#define GI_DMA_RESET                   (1 << 3)
-#define GI_DMA_BANKSW_ERROR            (1 << 4)
+#define GI_DMA_ENABLE                  BIT(0)
+#define GI_DMA_WRITE                   BIT(1)
+#define GI_DMA_INT_ENA                 BIT(2)
+#define GI_DMA_RESET                   BIT(3)
+#define GI_DMA_BANKSW_ERROR            BIT(4)
 #define NITIO_DMA_STATUS_REG(x)                (NITIO_G0_DMA_STATUS + (x))
-#define GI_DMA_READBANK                        (1 << 13)
-#define GI_DRQ_ERROR                   (1 << 14)
-#define GI_DRQ_STATUS                  (1 << 15)
+#define GI_DMA_READBANK                        BIT(13)
+#define GI_DRQ_ERROR                   BIT(14)
+#define GI_DRQ_STATUS                  BIT(15)
 #define NITIO_ABZ_REG(x)               (NITIO_G0_ABZ + (x))
 #define NITIO_INT_ACK_REG(x)           (NITIO_G0_INT_ACK + (x))
-#define GI_GATE_ERROR_CONFIRM(x)       (((x) % 2) ? (1 << 1) : (1 << 5))
-#define GI_TC_ERROR_CONFIRM(x)         (((x) % 2) ? (1 << 2) : (1 << 6))
-#define GI_TC_INTERRUPT_ACK            (1 << 14)
-#define GI_GATE_INTERRUPT_ACK          (1 << 15)
+#define GI_GATE_ERROR_CONFIRM(x)       (((x) % 2) ? BIT(1) : BIT(5))
+#define GI_TC_ERROR_CONFIRM(x)         (((x) % 2) ? BIT(2) : BIT(6))
+#define GI_TC_INTERRUPT_ACK            BIT(14)
+#define GI_GATE_INTERRUPT_ACK          BIT(15)
 #define NITIO_STATUS_REG(x)            (NITIO_G0_STATUS + (x))
-#define GI_GATE_INTERRUPT              (1 << 2)
-#define GI_TC                          (1 << 3)
-#define GI_INTERRUPT                   (1 << 15)
+#define GI_GATE_INTERRUPT              BIT(2)
+#define GI_TC                          BIT(3)
+#define GI_INTERRUPT                   BIT(15)
 #define NITIO_INT_ENA_REG(x)           (NITIO_G0_INT_ENA + (x))
-#define GI_TC_INTERRUPT_ENABLE(x)      (((x) % 2) ? (1 << 9) : (1 << 6))
-#define GI_GATE_INTERRUPT_ENABLE(x)    (((x) % 2) ? (1 << 10) : (1 << 8))
+#define GI_TC_INTERRUPT_ENABLE(x)      (((x) % 2) ? BIT(9) : BIT(6))
+#define GI_GATE_INTERRUPT_ENABLE(x)    (((x) % 2) ? BIT(10) : BIT(8))
 
 static inline void write_register(struct ni_gpct *counter, unsigned bits,
                                  enum ni_gpct_register reg)