staging: comedi: das08: rename and rewrite DAS08_EOC
authorIan Abbott <abbotti@mev.co.uk>
Fri, 5 Jun 2015 17:30:13 +0000 (18:30 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 13 Jun 2015 00:06:40 +0000 (17:06 -0700)
The `DAS08_EOC` macro contains a mask for the "end of A/D conversion"
bit in the status register.  The logic is reverse sense in that the bit
is set to 1 while the conversion is in progress and set to 0 when the
conversion is complete.  Rename the macro to `DAS08_STATUS_AI_BUSY` and
add a comment.  Also make use of the `BIT()` macro to define the value.

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

index eecbf4d95942e2a283b4a55ed98d0ae923903bf5..04843080b5e8dead9dbacb0781f1327233fa7ea5 100644 (file)
@@ -58,7 +58,7 @@
 #define DAS08_AI_MSB_REG       0x01    /* (R) AI most significant bits */
 #define DAS08_AI_TRIG_REG      0x01    /* (W) AI software trigger */
 #define DAS08_STATUS_REG       0x02    /* (R) status */
-#define   DAS08_EOC                    (1<<7)
+#define DAS08_STATUS_AI_BUSY   BIT(7)  /* AI conversion in progress */
 #define   DAS08_IRQ                    (1<<3)
 #define   DAS08_IP(x)                  (((x)>>4)&0x7)
 #define DAS08_CONTROL          2
@@ -206,7 +206,7 @@ static int das08_ai_eoc(struct comedi_device *dev,
        unsigned int status;
 
        status = inb(dev->iobase + DAS08_STATUS_REG);
-       if ((status & DAS08_EOC) == 0)
+       if ((status & DAS08_STATUS_AI_BUSY) == 0)
                return 0;
        return -EBUSY;
 }