staging: comedi: ni_tio_internal.h: replace NITIO_Gxx_Status_Reg()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 19 Dec 2013 23:31:43 +0000 (16:31 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Dec 2013 19:49:32 +0000 (11:49 -0800)
The shared "Status" registers are sequential in the enum ni_gpct_register.
Replace this inline CamelCase function with a simple define.

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.c
drivers/staging/comedi/drivers/ni_tio_internal.h
drivers/staging/comedi/drivers/ni_tiocmd.c

index 5203103084fcc4aa008ca9db8c85aa1a899e5ba3..7da8228f554ca4e659a7831c678c13f27e48279f 100644 (file)
@@ -425,7 +425,7 @@ static unsigned int ni_tio_counter_status(struct ni_gpct *counter)
 {
        unsigned int status = 0;
        const unsigned bits = read_register(counter,
-                                           NITIO_Gxx_Status_Reg(counter->
+                                           NITIO_STATUS_REG(counter->
                                                                 counter_index));
        if (bits & Gi_Armed_Bit(counter->counter_index)) {
                status |= COMEDI_COUNTER_ARMED;
@@ -1679,7 +1679,7 @@ EXPORT_SYMBOL_GPL(ni_tio_rinsn);
 static unsigned ni_tio_next_load_register(struct ni_gpct *counter)
 {
        const unsigned bits = read_register(counter,
-                                           NITIO_Gxx_Status_Reg(counter->
+                                           NITIO_STATUS_REG(counter->
                                                                 counter_index));
 
        if (bits & Gi_Next_Load_Source_Bit(counter->counter_index))
index 1ef5f35f4049c1be02d5f742b386ba8eb787a4f5..604fa6909f09d9c2dc136f74dacde5545bd823bf 100644 (file)
@@ -30,6 +30,7 @@
 #define NITIO_INPUT_SEL_REG(x)         (NITIO_G0_INPUT_SEL + (x))
 #define NITIO_CNT_MODE_REG(x)          (NITIO_G0_CNT_MODE + (x))
 #define NITIO_GATE2_REG(x)             (NITIO_G0_GATE2 + (x))
+#define NITIO_STATUS_REG(x)            (NITIO_G01_STATUS + ((x) / 2))
 
 static inline enum ni_gpct_register NITIO_Gxx_Joint_Reset_Reg(unsigned idx)
 {
@@ -70,19 +71,6 @@ static inline enum ni_gpct_register NITIO_Gxx_Joint_Status2_Reg(unsigned idx)
        return 0;
 }
 
-static inline enum ni_gpct_register NITIO_Gxx_Status_Reg(unsigned idx)
-{
-       switch (idx) {
-       case 0:
-       case 1:
-               return NITIO_G01_STATUS;
-       case 2:
-       case 3:
-               return NITIO_G23_STATUS;
-       }
-       return 0;
-}
-
 static inline enum ni_gpct_register NITIO_Gi_DMA_Config_Reg(unsigned idx)
 {
        switch (idx) {
index b3dad4f91656dd19b32cceeed185bfb756a87115..0ffb3a4c2a65b62c901a0e36638fd618ae131bdb 100644 (file)
@@ -354,7 +354,7 @@ void ni_tio_acknowledge_and_confirm(struct ni_gpct *counter, int *gate_error,
                                    int *stale_data)
 {
        const unsigned short gxx_status = read_register(counter,
-                                                       NITIO_Gxx_Status_Reg
+                                                       NITIO_STATUS_REG
                                                        (counter->
                                                         counter_index));
        const unsigned short gi_status = read_register(counter,