staging: comedi: pcmmio: tidy up the ao subdevice init
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 9 Dec 2013 22:30:50 +0000 (15:30 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 17 Dec 2013 18:01:50 +0000 (10:01 -0800)
Add some whitespace to the analog output subdevice init and reorder the
init to follow the "norm" in comedi drivers.

Remove the init of len_chanlist. This member is only used for subdevices
the support async commands. The core will default the value correctly.

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/pcmmio.c

index 9338f0c..39dff77 100644 (file)
@@ -1016,16 +1016,15 @@ static int pcmmio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        outb(0,
             dev->iobase + PCMMIO_AI_2ND_ADC_OFFSET + PCMMIO_AI_RESOURCE_REG);
 
-       /* Next, AO */
+       /* Analog Output subdevice */
        s = &dev->subdevices[1];
-       s->maxdata = 0xffff;
-       s->range_table = &pcmmio_ao_ranges;
-       s->subdev_flags = SDF_READABLE;
-       s->type = COMEDI_SUBD_AO;
-       s->n_chan = 8;
-       s->len_chanlist = s->n_chan;
-       s->insn_read = pcmmio_ao_insn_read;
-       s->insn_write = pcmmio_ao_insn_write;
+       s->type         = COMEDI_SUBD_AO;
+       s->subdev_flags = SDF_READABLE;
+       s->n_chan       = 8;
+       s->maxdata      = 0xffff;
+       s->range_table  = &pcmmio_ao_ranges;
+       s->insn_read    = pcmmio_ao_insn_read;
+       s->insn_write   = pcmmio_ao_insn_write;
 
        /* initialize the resource enable register by clearing it */
        outb(0, dev->iobase + PCMMIO_AO_RESOURCE_ENA_REG);