comedi vmk80xx: make rudimentary_check a static function
authorJ. Ali Harlow <ali@avrc.city.ac.uk>
Wed, 18 May 2011 18:18:51 +0000 (19:18 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 18 May 2011 21:32:04 +0000 (14:32 -0700)
rudimentary_check is a macro with side-effects (it returns
on error) which is contary to CodingStyle. Replace it with
a static function.

Signed-off-by: J. Ali Harlow <ali@avrc.city.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/comedi/drivers/vmk80xx.c

index e4751334c563465c566829a893e60b4ac46eccfe..5d83bb667144c38b9702e58e63bb8a917409d6c5 100644 (file)
@@ -557,22 +557,24 @@ exit:
 #define DIR_IN  1
 #define DIR_OUT 2
 
-#define rudimentary_check(dir)                             \
-do {                                                       \
-       if (!dev)                                          \
-               return -EFAULT;                            \
-       if (!dev->probed)                                  \
-               return -ENODEV;                            \
-       if (!dev->attached)                                \
-               return -ENODEV;                            \
-       if ((dir) & DIR_IN) {                              \
-               if (test_bit(TRANS_IN_BUSY, &dev->flags))  \
-                       return -EBUSY;                     \
-       } else {  /* DIR_OUT */                            \
-               if (test_bit(TRANS_OUT_BUSY, &dev->flags)) \
-                       return -EBUSY;                     \
-       }                                                  \
-} while (0)
+static int rudimentary_check(struct vmk80xx_usb *dev, int dir)
+{
+       if (!dev)
+               return -EFAULT;
+       if (!dev->probed)
+               return -ENODEV;
+       if (!dev->attached)
+               return -ENODEV;
+       if (dir & DIR_IN) {
+               if (test_bit(TRANS_IN_BUSY, &dev->flags))
+                       return -EBUSY;
+       } else {  /* DIR_OUT */
+               if (test_bit(TRANS_OUT_BUSY, &dev->flags))
+                       return -EBUSY;
+       }
+
+       return 0;
+}
 
 static int vmk80xx_ai_rinsn(struct comedi_device *cdev,
                            struct comedi_subdevice *s,
@@ -585,7 +587,9 @@ static int vmk80xx_ai_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -636,7 +640,9 @@ static int vmk80xx_ao_winsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_OUT);
+       n = rudimentary_check(dev, DIR_OUT);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -679,7 +685,9 @@ static int vmk80xx_ao_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -713,7 +721,9 @@ static int vmk80xx_di_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -758,7 +768,9 @@ static int vmk80xx_do_winsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_OUT);
+       n = rudimentary_check(dev, DIR_OUT);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -805,7 +817,9 @@ static int vmk80xx_do_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -838,7 +852,9 @@ static int vmk80xx_cnt_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -886,7 +902,9 @@ static int vmk80xx_cnt_cinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_OUT);
+       n = rudimentary_check(dev, DIR_OUT);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
 
@@ -932,7 +950,9 @@ static int vmk80xx_cnt_winsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_OUT);
+       n = rudimentary_check(dev, DIR_OUT);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
        chan = CR_CHAN(insn->chanspec);
@@ -976,7 +996,9 @@ static int vmk80xx_pwm_rinsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_IN);
+       n = rudimentary_check(dev, DIR_IN);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);
 
@@ -1009,7 +1031,9 @@ static int vmk80xx_pwm_winsn(struct comedi_device *cdev,
 
        dbgvm("vmk80xx: %s\n", __func__);
 
-       rudimentary_check(DIR_OUT);
+       n = rudimentary_check(dev, DIR_OUT);
+       if (n)
+               return n;
 
        down(&dev->limit_sem);