staging: comedi: ni_660x: remove enum ni_register_width
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 22 Mar 2016 18:10:13 +0000 (11:10 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
All the registers are defined struct NI_660xRegisterData and they are
either 2 or 4 bytes in size. Remove the enum and just use a char member
to define the size as 2 or 4 bytes.

Simplify the ni_660x_{write,read}_register() functions and remove the
unnecessary BUG() in each.

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_660x.c

index 10bb8394fd99d8cb7166a9e366c1510acd7252b7..dbbeb968ae5971e98721a0cb339bfbb056e3dd44 100644 (file)
@@ -158,12 +158,6 @@ enum ni_660x_register {
 
 #define NI660X_IO_CFG(x)       (NI660X_IO_CFG_0_1 + ((x) / 2))
 
-enum ni_660x_register_width {
-       DATA_1B,
-       DATA_2B,
-       DATA_4B
-};
-
 enum ni_660x_register_direction {
        NI_660x_READ,
        NI_660x_WRITE,
@@ -189,108 +183,108 @@ static inline unsigned NI_660X_GPCT_SUBDEV(unsigned index)
 struct NI_660xRegisterData {
        int offset;             /*  Offset from base address from GPCT chip */
        enum ni_660x_register_direction direction;
-       enum ni_660x_register_width size; /* 1 byte, 2 bytes, or 4 bytes */
+       char size;              /* 2 or 4 bytes */
 };
 
 static const struct NI_660xRegisterData registerData[NI660X_NUM_REGS] = {
-       [NI660X_G0_INT_ACK]             = {0x004, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_STATUS]              = {0x004, NI_660x_READ, DATA_2B},
-       [NI660X_G1_INT_ACK]             = {0x006, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_STATUS]              = {0x006, NI_660x_READ, DATA_2B},
-       [NI660X_G01_STATUS]             = {0x008, NI_660x_READ, DATA_2B},
-       [NI660X_G0_CMD]                 = {0x00c, NI_660x_WRITE, DATA_2B},
-       [NI660X_STC_DIO_PARALLEL_INPUT] = {0x00e, NI_660x_READ, DATA_2B},
-       [NI660X_G1_CMD]                 = {0x00e, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_HW_SAVE]             = {0x010, NI_660x_READ, DATA_4B},
-       [NI660X_G1_HW_SAVE]             = {0x014, NI_660x_READ, DATA_4B},
-       [NI660X_STC_DIO_OUTPUT]         = {0x014, NI_660x_WRITE, DATA_2B},
-       [NI660X_STC_DIO_CONTROL]        = {0x016, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_SW_SAVE]             = {0x018, NI_660x_READ, DATA_4B},
-       [NI660X_G1_SW_SAVE]             = {0x01c, NI_660x_READ, DATA_4B},
-       [NI660X_G0_MODE]                = {0x034, NI_660x_WRITE, DATA_2B},
-       [NI660X_G01_STATUS1]            = {0x036, NI_660x_READ, DATA_2B},
-       [NI660X_G1_MODE]                = {0x036, NI_660x_WRITE, DATA_2B},
-       [NI660X_STC_DIO_SERIAL_INPUT]   = {0x038, NI_660x_READ, DATA_2B},
-       [NI660X_G0_LOADA]               = {0x038, NI_660x_WRITE, DATA_4B},
-       [NI660X_G01_STATUS2]            = {0x03a, NI_660x_READ, DATA_2B},
-       [NI660X_G0_LOADB]               = {0x03c, NI_660x_WRITE, DATA_4B},
-       [NI660X_G1_LOADA]               = {0x040, NI_660x_WRITE, DATA_4B},
-       [NI660X_G1_LOADB]               = {0x044, NI_660x_WRITE, DATA_4B},
-       [NI660X_G0_INPUT_SEL]           = {0x048, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_INPUT_SEL]           = {0x04a, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_AUTO_INC]            = {0x088, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_AUTO_INC]            = {0x08a, NI_660x_WRITE, DATA_2B},
-       [NI660X_G01_RESET]              = {0x090, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_INT_ENA]             = {0x092, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_INT_ENA]             = {0x096, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_CNT_MODE]            = {0x0b0, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_CNT_MODE]            = {0x0b2, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_GATE2]               = {0x0b4, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_GATE2]               = {0x0b6, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_DMA_CFG]             = {0x0b8, NI_660x_WRITE, DATA_2B},
-       [NI660X_G0_DMA_STATUS]          = {0x0b8, NI_660x_READ, DATA_2B},
-       [NI660X_G1_DMA_CFG]             = {0x0ba, NI_660x_WRITE, DATA_2B},
-       [NI660X_G1_DMA_STATUS]          = {0x0ba, NI_660x_READ, DATA_2B},
-       [NI660X_G2_INT_ACK]             = {0x104, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_STATUS]              = {0x104, NI_660x_READ, DATA_2B},
-       [NI660X_G3_INT_ACK]             = {0x106, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_STATUS]              = {0x106, NI_660x_READ, DATA_2B},
-       [NI660X_G23_STATUS]             = {0x108, NI_660x_READ, DATA_2B},
-       [NI660X_G2_CMD]                 = {0x10c, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_CMD]                 = {0x10e, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_HW_SAVE]             = {0x110, NI_660x_READ, DATA_4B},
-       [NI660X_G3_HW_SAVE]             = {0x114, NI_660x_READ, DATA_4B},
-       [NI660X_G2_SW_SAVE]             = {0x118, NI_660x_READ, DATA_4B},
-       [NI660X_G3_SW_SAVE]             = {0x11c, NI_660x_READ, DATA_4B},
-       [NI660X_G2_MODE]                = {0x134, NI_660x_WRITE, DATA_2B},
-       [NI660X_G23_STATUS1]            = {0x136, NI_660x_READ, DATA_2B},
-       [NI660X_G3_MODE]                = {0x136, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_LOADA]               = {0x138, NI_660x_WRITE, DATA_4B},
-       [NI660X_G23_STATUS2]            = {0x13a, NI_660x_READ, DATA_2B},
-       [NI660X_G2_LOADB]               = {0x13c, NI_660x_WRITE, DATA_4B},
-       [NI660X_G3_LOADA]               = {0x140, NI_660x_WRITE, DATA_4B},
-       [NI660X_G3_LOADB]               = {0x144, NI_660x_WRITE, DATA_4B},
-       [NI660X_G2_INPUT_SEL]           = {0x148, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_INPUT_SEL]           = {0x14a, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_AUTO_INC]            = {0x188, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_AUTO_INC]            = {0x18a, NI_660x_WRITE, DATA_2B},
-       [NI660X_G23_RESET]              = {0x190, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_INT_ENA]             = {0x192, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_INT_ENA]             = {0x196, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_CNT_MODE]            = {0x1b0, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_CNT_MODE]            = {0x1b2, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_GATE2]               = {0x1b6, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_GATE2]               = {0x1b4, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_DMA_CFG]             = {0x1b8, NI_660x_WRITE, DATA_2B},
-       [NI660X_G2_DMA_STATUS]          = {0x1b8, NI_660x_READ, DATA_2B},
-       [NI660X_G3_DMA_CFG]             = {0x1ba, NI_660x_WRITE, DATA_2B},
-       [NI660X_G3_DMA_STATUS]          = {0x1ba, NI_660x_READ, DATA_2B},
-       [NI660X_DIO32_INPUT]            = {0x414, NI_660x_READ, DATA_4B},
-       [NI660X_DIO32_OUTPUT]           = {0x510, NI_660x_WRITE, DATA_4B},
-       [NI660X_CLK_CFG]                = {0x73c, NI_660x_WRITE, DATA_4B},
-       [NI660X_GLOBAL_INT_STATUS]      = {0x754, NI_660x_READ, DATA_4B},
-       [NI660X_DMA_CFG]                = {0x76c, NI_660x_WRITE, DATA_4B},
-       [NI660X_GLOBAL_INT_CFG]         = {0x770, NI_660x_WRITE, DATA_4B},
-       [NI660X_IO_CFG_0_1]             = {0x77c, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_2_3]             = {0x77e, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_4_5]             = {0x780, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_6_7]             = {0x782, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_8_9]             = {0x784, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_10_11]           = {0x786, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_12_13]           = {0x788, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_14_15]           = {0x78a, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_16_17]           = {0x78c, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_18_19]           = {0x78e, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_20_21]           = {0x790, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_22_23]           = {0x792, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_24_25]           = {0x794, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_26_27]           = {0x796, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_28_29]           = {0x798, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_30_31]           = {0x79a, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_32_33]           = {0x79c, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_34_35]           = {0x79e, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_36_37]           = {0x7a0, NI_660x_READ_WRITE, DATA_2B},
-       [NI660X_IO_CFG_38_39]           = {0x7a2, NI_660x_READ_WRITE, DATA_2B}
+       [NI660X_G0_INT_ACK]             = { 0x004, NI_660x_WRITE, 2 },
+       [NI660X_G0_STATUS]              = { 0x004, NI_660x_READ, 2 },
+       [NI660X_G1_INT_ACK]             = { 0x006, NI_660x_WRITE, 2 },
+       [NI660X_G1_STATUS]              = { 0x006, NI_660x_READ, 2 },
+       [NI660X_G01_STATUS]             = { 0x008, NI_660x_READ, 2 },
+       [NI660X_G0_CMD]                 = { 0x00c, NI_660x_WRITE, 2 },
+       [NI660X_STC_DIO_PARALLEL_INPUT] = { 0x00e, NI_660x_READ, 2 },
+       [NI660X_G1_CMD]                 = { 0x00e, NI_660x_WRITE, 2 },
+       [NI660X_G0_HW_SAVE]             = { 0x010, NI_660x_READ, 4 },
+       [NI660X_G1_HW_SAVE]             = { 0x014, NI_660x_READ, 4 },
+       [NI660X_STC_DIO_OUTPUT]         = { 0x014, NI_660x_WRITE, 2 },
+       [NI660X_STC_DIO_CONTROL]        = { 0x016, NI_660x_WRITE, 2 },
+       [NI660X_G0_SW_SAVE]             = { 0x018, NI_660x_READ, 4 },
+       [NI660X_G1_SW_SAVE]             = { 0x01c, NI_660x_READ, 4 },
+       [NI660X_G0_MODE]                = { 0x034, NI_660x_WRITE, 2 },
+       [NI660X_G01_STATUS1]            = { 0x036, NI_660x_READ, 2 },
+       [NI660X_G1_MODE]                = { 0x036, NI_660x_WRITE, 2 },
+       [NI660X_STC_DIO_SERIAL_INPUT]   = { 0x038, NI_660x_READ, 2 },
+       [NI660X_G0_LOADA]               = { 0x038, NI_660x_WRITE, 4 },
+       [NI660X_G01_STATUS2]            = { 0x03a, NI_660x_READ, 2 },
+       [NI660X_G0_LOADB]               = { 0x03c, NI_660x_WRITE, 4 },
+       [NI660X_G1_LOADA]               = { 0x040, NI_660x_WRITE, 4 },
+       [NI660X_G1_LOADB]               = { 0x044, NI_660x_WRITE, 4 },
+       [NI660X_G0_INPUT_SEL]           = { 0x048, NI_660x_WRITE, 2 },
+       [NI660X_G1_INPUT_SEL]           = { 0x04a, NI_660x_WRITE, 2 },
+       [NI660X_G0_AUTO_INC]            = { 0x088, NI_660x_WRITE, 2 },
+       [NI660X_G1_AUTO_INC]            = { 0x08a, NI_660x_WRITE, 2 },
+       [NI660X_G01_RESET]              = { 0x090, NI_660x_WRITE, 2 },
+       [NI660X_G0_INT_ENA]             = { 0x092, NI_660x_WRITE, 2 },
+       [NI660X_G1_INT_ENA]             = { 0x096, NI_660x_WRITE, 2 },
+       [NI660X_G0_CNT_MODE]            = { 0x0b0, NI_660x_WRITE, 2 },
+       [NI660X_G1_CNT_MODE]            = { 0x0b2, NI_660x_WRITE, 2 },
+       [NI660X_G0_GATE2]               = { 0x0b4, NI_660x_WRITE, 2 },
+       [NI660X_G1_GATE2]               = { 0x0b6, NI_660x_WRITE, 2 },
+       [NI660X_G0_DMA_CFG]             = { 0x0b8, NI_660x_WRITE, 2 },
+       [NI660X_G0_DMA_STATUS]          = { 0x0b8, NI_660x_READ, 2 },
+       [NI660X_G1_DMA_CFG]             = { 0x0ba, NI_660x_WRITE, 2 },
+       [NI660X_G1_DMA_STATUS]          = { 0x0ba, NI_660x_READ, 2 },
+       [NI660X_G2_INT_ACK]             = { 0x104, NI_660x_WRITE, 2 },
+       [NI660X_G2_STATUS]              = { 0x104, NI_660x_READ, 2 },
+       [NI660X_G3_INT_ACK]             = { 0x106, NI_660x_WRITE, 2 },
+       [NI660X_G3_STATUS]              = { 0x106, NI_660x_READ, 2 },
+       [NI660X_G23_STATUS]             = { 0x108, NI_660x_READ, 2 },
+       [NI660X_G2_CMD]                 = { 0x10c, NI_660x_WRITE, 2 },
+       [NI660X_G3_CMD]                 = { 0x10e, NI_660x_WRITE, 2 },
+       [NI660X_G2_HW_SAVE]             = { 0x110, NI_660x_READ, 4 },
+       [NI660X_G3_HW_SAVE]             = { 0x114, NI_660x_READ, 4 },
+       [NI660X_G2_SW_SAVE]             = { 0x118, NI_660x_READ, 4 },
+       [NI660X_G3_SW_SAVE]             = { 0x11c, NI_660x_READ, 4 },
+       [NI660X_G2_MODE]                = { 0x134, NI_660x_WRITE, 2 },
+       [NI660X_G23_STATUS1]            = { 0x136, NI_660x_READ, 2 },
+       [NI660X_G3_MODE]                = { 0x136, NI_660x_WRITE, 2 },
+       [NI660X_G2_LOADA]               = { 0x138, NI_660x_WRITE, 4 },
+       [NI660X_G23_STATUS2]            = { 0x13a, NI_660x_READ, 2 },
+       [NI660X_G2_LOADB]               = { 0x13c, NI_660x_WRITE, 4 },
+       [NI660X_G3_LOADA]               = { 0x140, NI_660x_WRITE, 4 },
+       [NI660X_G3_LOADB]               = { 0x144, NI_660x_WRITE, 4 },
+       [NI660X_G2_INPUT_SEL]           = { 0x148, NI_660x_WRITE, 2 },
+       [NI660X_G3_INPUT_SEL]           = { 0x14a, NI_660x_WRITE, 2 },
+       [NI660X_G2_AUTO_INC]            = { 0x188, NI_660x_WRITE, 2 },
+       [NI660X_G3_AUTO_INC]            = { 0x18a, NI_660x_WRITE, 2 },
+       [NI660X_G23_RESET]              = { 0x190, NI_660x_WRITE, 2 },
+       [NI660X_G2_INT_ENA]             = { 0x192, NI_660x_WRITE, 2 },
+       [NI660X_G3_INT_ENA]             = { 0x196, NI_660x_WRITE, 2 },
+       [NI660X_G2_CNT_MODE]            = { 0x1b0, NI_660x_WRITE, 2 },
+       [NI660X_G3_CNT_MODE]            = { 0x1b2, NI_660x_WRITE, 2 },
+       [NI660X_G3_GATE2]               = { 0x1b6, NI_660x_WRITE, 2 },
+       [NI660X_G2_GATE2]               = { 0x1b4, NI_660x_WRITE, 2 },
+       [NI660X_G2_DMA_CFG]             = { 0x1b8, NI_660x_WRITE, 2 },
+       [NI660X_G2_DMA_STATUS]          = { 0x1b8, NI_660x_READ, 2 },
+       [NI660X_G3_DMA_CFG]             = { 0x1ba, NI_660x_WRITE, 2 },
+       [NI660X_G3_DMA_STATUS]          = { 0x1ba, NI_660x_READ, 2 },
+       [NI660X_DIO32_INPUT]            = { 0x414, NI_660x_READ, 4 },
+       [NI660X_DIO32_OUTPUT]           = { 0x510, NI_660x_WRITE, 4 },
+       [NI660X_CLK_CFG]                = { 0x73c, NI_660x_WRITE, 4 },
+       [NI660X_GLOBAL_INT_STATUS]      = { 0x754, NI_660x_READ, 4 },
+       [NI660X_DMA_CFG]                = { 0x76c, NI_660x_WRITE, 4 },
+       [NI660X_GLOBAL_INT_CFG]         = { 0x770, NI_660x_WRITE, 4 },
+       [NI660X_IO_CFG_0_1]             = { 0x77c, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_2_3]             = { 0x77e, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_4_5]             = { 0x780, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_6_7]             = { 0x782, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_8_9]             = { 0x784, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_10_11]           = { 0x786, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_12_13]           = { 0x788, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_14_15]           = { 0x78a, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_16_17]           = { 0x78c, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_18_19]           = { 0x78e, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_20_21]           = { 0x790, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_22_23]           = { 0x792, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_24_25]           = { 0x794, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_26_27]           = { 0x796, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_28_29]           = { 0x798, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_30_31]           = { 0x79a, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_32_33]           = { 0x79c, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_34_35]           = { 0x79e, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_36_37]           = { 0x7a0, NI_660x_READ_WRITE, 2 },
+       [NI660X_IO_CFG_38_39]           = { 0x7a2, NI_660x_READ_WRITE, 2 }
 };
 
 /* kind of ENABLE for the second counter */
@@ -579,17 +573,10 @@ static inline void ni_660x_write_register(struct comedi_device *dev,
 {
        unsigned int addr = GPCT_OFFSET[chip] + registerData[reg].offset;
 
-       switch (registerData[reg].size) {
-       case DATA_2B:
+       if (registerData[reg].size == 2)
                writew(bits, dev->mmio + addr);
-               break;
-       case DATA_4B:
+       else
                writel(bits, dev->mmio + addr);
-               break;
-       default:
-               BUG();
-               break;
-       }
 }
 
 static inline unsigned ni_660x_read_register(struct comedi_device *dev,
@@ -598,16 +585,9 @@ static inline unsigned ni_660x_read_register(struct comedi_device *dev,
 {
        unsigned int addr = GPCT_OFFSET[chip] + registerData[reg].offset;
 
-       switch (registerData[reg].size) {
-       case DATA_2B:
+       if (registerData[reg].size == 2)
                return readw(dev->mmio + addr);
-       case DATA_4B:
-               return readl(dev->mmio + addr);
-       default:
-               BUG();
-               break;
-       }
-       return 0;
+       return readl(dev->mmio + addr);
 }
 
 static void ni_gpct_write_register(struct ni_gpct *counter, unsigned bits,