staging: comedi: ni_mio_common: tidy up the Analog Output subdevice init
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 14 Jul 2014 19:23:56 +0000 (12:23 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 16 Jul 2014 20:24:43 +0000 (13:24 -0700)
For aesthetics, add some whitespace to the Analog Output subdevice init.

Also, remove the '#ifdef PCIDMA' here and only hook up the async command
support if we have an irq and the board either has a fifo or DMA is
supported.

The ni_pcimio driver is the only place PCIDMA is defined. That driver
is also the only user of ni_mio_common that allocates devpriv->mite. Use
that test instead to determine if dma is supported.

Move 'is_67xx' check and init_ao_67xx() call into the Analog Output
subdevice init. The 67xx boards all have analog outputs and this check
does not need to be done for the boards that do not have analog outputs.

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_mio_common.c

index 800f4e3af77b81d47fc947b38d992ba146d0f4c0..91b3fb4d12c22b95588c9482ce50d7e2057cc326 100644 (file)
@@ -5536,40 +5536,43 @@ static int ni_E_init(struct comedi_device *dev,
                s->type         = COMEDI_SUBD_UNUSED;
        }
 
-       /* analog output subdevice */
-
+       /* Analog Output subdevice */
        s = &dev->subdevices[NI_AO_SUBDEV];
        if (board->n_aochan) {
-               s->type = COMEDI_SUBD_AO;
-               s->subdev_flags = SDF_WRITABLE | SDF_DEGLITCH | SDF_GROUND;
+               s->type         = COMEDI_SUBD_AO;
+               s->subdev_flags = SDF_WRITABLE | SDF_DEGLITCH | SDF_GROUND;
                if (devpriv->is_m_series)
-                       s->subdev_flags |= SDF_SOFT_CALIBRATED;
-               s->n_chan = board->n_aochan;
-               s->maxdata = board->ao_maxdata;
-               s->range_table = board->ao_range_table;
-               s->insn_read = &ni_ao_insn_read;
-               s->insn_write = &ni_ao_insn_write;
-               s->insn_config = &ni_ao_insn_config;
-#ifdef PCIDMA
-               if (board->n_aochan) {
-                       s->async_dma_dir = DMA_TO_DEVICE;
-#else
-               if (board->ao_fifo_depth) {
-#endif
+                       s->subdev_flags |= SDF_SOFT_CALIBRATED;
+               s->n_chan       = board->n_aochan;
+               s->maxdata      = board->ao_maxdata;
+               s->range_table  = board->ao_range_table;
+               s->insn_read    = ni_ao_insn_read;
+               s->insn_write   = ni_ao_insn_write;
+               s->insn_config  = ni_ao_insn_config;
+
+               /*
+                * Along with the IRQ we need either a FIFO or DMA for
+                * async command support.
+                */
+               if (dev->irq && (board->ao_fifo_depth || devpriv->mite)) {
                        dev->write_subdev = s;
-                       s->subdev_flags |= SDF_CMD_WRITE;
-                       s->do_cmd = &ni_ao_cmd;
-                       s->do_cmdtest = &ni_ao_cmdtest;
-                       s->cancel = &ni_ao_reset;
-                       s->len_chanlist = board->n_aochan;
+                       s->subdev_flags |= SDF_CMD_WRITE;
+                       s->len_chanlist = s->n_chan;
+                       s->do_cmdtest   = ni_ao_cmdtest;
+                       s->do_cmd       = ni_ao_cmd;
+                       s->cancel       = ni_ao_reset;
                        if (!devpriv->is_m_series)
-                               s->munge = ni_ao_munge;
+                               s->munge        = ni_ao_munge;
+
+                       if (devpriv->mite)
+                               s->async_dma_dir = DMA_TO_DEVICE;
                }
+
+               if (devpriv->is_67xx)
+                       init_ao_67xx(dev, s);
        } else {
-               s->type = COMEDI_SUBD_UNUSED;
+               s->type         = COMEDI_SUBD_UNUSED;
        }
-       if (devpriv->is_67xx)
-               init_ao_67xx(dev, s);
 
        /* digital i/o subdevice */