staging: comedi: ni_tio_internal: simplify ni_tio_counting_mode_registers_present()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Wed, 23 Mar 2016 22:36:43 +0000 (15:36 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
Only the e series gpct variant does not have counting mode registers.
Simplfy this function. For aesthetics, return bool instead of int.

This fixes the checkpatch.pl issues:
CHECK: spaces preferred around that '*' (ctx:ExV)
CHECK: Avoid crashing the kernel - try using WARN_ON & recovery code
       rather than BUG() or BUG_ON()

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

index 5343fce88ebfd8ad7d21160054c0baab298a34da..44dea4c4da20434a8995a331cf86b87ae7ef114c 100644 (file)
@@ -451,7 +451,7 @@ static void ni_tio_set_sync_mode(struct ni_gpct *counter, int force_alt_sync)
        unsigned mode;
        uint64_t clock_period_ps;
 
-       if (ni_tio_counting_mode_registers_present(counter_dev) == 0)
+       if (!ni_tio_counting_mode_registers_present(counter_dev))
                return;
 
        mode = ni_tio_get_soft_copy(counter, counting_mode_reg);
index c8ad66a9237fe7e11b54723e010c66aaba29d1cf..1e795fd6f288f2464e00293e8c14e20ec767616f 100644 (file)
@@ -174,21 +174,11 @@ static inline unsigned read_register(struct ni_gpct *counter,
        return counter->counter_dev->read_register(counter, reg);
 }
 
-static inline int ni_tio_counting_mode_registers_present(const struct
-                                                        ni_gpct_device
-                                                        *counter_dev)
+static inline bool
+ni_tio_counting_mode_registers_present(const struct ni_gpct_device *counter_dev)
 {
-       switch (counter_dev->variant) {
-       case ni_gpct_variant_e_series:
-               return 0;
-       case ni_gpct_variant_m_series:
-       case ni_gpct_variant_660x:
-               return 1;
-       default:
-               BUG();
-               break;
-       }
-       return 0;
+       /* m series and 660x variants have counting mode registers */
+       return counter_dev->variant != ni_gpct_variant_e_series;
 }
 
 void ni_tio_set_bits(struct ni_gpct *, enum ni_gpct_register reg,