staging: comedi: das08: use comedi_timeout()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 10 Feb 2014 18:49:18 +0000 (11:49 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Feb 2014 17:28:45 +0000 (09:28 -0800)
Use comedi_timeout() to wait for the analog input end-of-conversion.

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

index 3c0e458..b991631 100644 (file)
@@ -201,17 +201,29 @@ static const int *const das08_gainlists[] = {
        das08_pgm_gainlist,
 };
 
-#define TIMEOUT 100000
+static int das08_ai_eoc(struct comedi_device *dev,
+                       struct comedi_subdevice *s,
+                       struct comedi_insn *insn,
+                       unsigned long context)
+{
+       unsigned int status;
+
+       status = inb(dev->iobase + DAS08_STATUS);
+       if ((status & DAS08_EOC) == 0)
+               return 0;
+       return -EBUSY;
+}
 
 static int das08_ai_rinsn(struct comedi_device *dev, struct comedi_subdevice *s,
                          struct comedi_insn *insn, unsigned int *data)
 {
        const struct das08_board_struct *thisboard = comedi_board(dev);
        struct das08_private_struct *devpriv = dev->private;
-       int i, n;
+       int n;
        int chan;
        int range;
        int lsb, msb;
+       int ret;
 
        chan = CR_CHAN(insn->chanspec);
        range = CR_RANGE(insn->chanspec);
@@ -244,14 +256,12 @@ static int das08_ai_rinsn(struct comedi_device *dev, struct comedi_subdevice *s,
                /* trigger conversion */
                outb_p(0, dev->iobase + DAS08_TRIG_12BIT);
 
-               for (i = 0; i < TIMEOUT; i++) {
-                       if (!(inb(dev->iobase + DAS08_STATUS) & DAS08_EOC))
-                               break;
-               }
-               if (i == TIMEOUT) {
+               ret = comedi_timeout(dev, s, insn, das08_ai_eoc, 0);
+               if (ret) {
                        dev_err(dev->class_dev, "timeout\n");
-                       return -ETIME;
+                       return ret;
                }
+
                msb = inb(dev->iobase + DAS08_MSB);
                lsb = inb(dev->iobase + DAS08_LSB);
                if (thisboard->ai_encoding == das08_encode12) {