Commit d3e8ab48 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: adv_pci1710: post increment 'subdev' in (*auto_attach)

For aesthetics, post-increment the 'subdev' index when used to get a
comedi_subdevice pointer instead of incrementing it after the subdevice
is initialized.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 73873325
...@@ -791,7 +791,7 @@ static int pci1710_auto_attach(struct comedi_device *dev, ...@@ -791,7 +791,7 @@ static int pci1710_auto_attach(struct comedi_device *dev,
subdev = 0; subdev = 0;
/* Analog Input subdevice */ /* Analog Input subdevice */
s = &dev->subdevices[subdev]; s = &dev->subdevices[subdev++];
s->type = COMEDI_SUBD_AI; s->type = COMEDI_SUBD_AI;
s->subdev_flags = SDF_READABLE | SDF_GROUND; s->subdev_flags = SDF_READABLE | SDF_GROUND;
if (!board->is_pci1711) if (!board->is_pci1711)
...@@ -817,11 +817,9 @@ static int pci1710_auto_attach(struct comedi_device *dev, ...@@ -817,11 +817,9 @@ static int pci1710_auto_attach(struct comedi_device *dev,
} }
} }
subdev++;
if (board->has_ao) { if (board->has_ao) {
/* Analog Output subdevice */ /* Analog Output subdevice */
s = &dev->subdevices[subdev]; s = &dev->subdevices[subdev++];
s->type = COMEDI_SUBD_AO; s->type = COMEDI_SUBD_AO;
s->subdev_flags = SDF_WRITABLE | SDF_GROUND; s->subdev_flags = SDF_WRITABLE | SDF_GROUND;
s->n_chan = 2; s->n_chan = 2;
...@@ -832,33 +830,29 @@ static int pci1710_auto_attach(struct comedi_device *dev, ...@@ -832,33 +830,29 @@ static int pci1710_auto_attach(struct comedi_device *dev,
ret = comedi_alloc_subdev_readback(s); ret = comedi_alloc_subdev_readback(s);
if (ret) if (ret)
return ret; return ret;
subdev++;
} }
if (!board->is_pci1713) { if (!board->is_pci1713) {
/* Digital Input subdevice */ /* Digital Input subdevice */
s = &dev->subdevices[subdev]; s = &dev->subdevices[subdev++];
s->type = COMEDI_SUBD_DI; s->type = COMEDI_SUBD_DI;
s->subdev_flags = SDF_READABLE; s->subdev_flags = SDF_READABLE;
s->n_chan = 16; s->n_chan = 16;
s->maxdata = 1; s->maxdata = 1;
s->range_table = &range_digital; s->range_table = &range_digital;
s->insn_bits = pci1710_di_insn_bits; s->insn_bits = pci1710_di_insn_bits;
subdev++;
/* Digital Output subdevice */ /* Digital Output subdevice */
s = &dev->subdevices[subdev]; s = &dev->subdevices[subdev++];
s->type = COMEDI_SUBD_DO; s->type = COMEDI_SUBD_DO;
s->subdev_flags = SDF_WRITABLE; s->subdev_flags = SDF_WRITABLE;
s->n_chan = 16; s->n_chan = 16;
s->maxdata = 1; s->maxdata = 1;
s->range_table = &range_digital; s->range_table = &range_digital;
s->insn_bits = pci1710_do_insn_bits; s->insn_bits = pci1710_do_insn_bits;
subdev++;
/* Counter subdevice (8254) */ /* Counter subdevice (8254) */
s = &dev->subdevices[subdev]; s = &dev->subdevices[subdev++];
comedi_8254_subdevice_init(s, dev->pacer); comedi_8254_subdevice_init(s, dev->pacer);
dev->pacer->insn_config = pci1710_counter_insn_config; dev->pacer->insn_config = pci1710_counter_insn_config;
...@@ -866,8 +860,6 @@ static int pci1710_auto_attach(struct comedi_device *dev, ...@@ -866,8 +860,6 @@ static int pci1710_auto_attach(struct comedi_device *dev,
/* counters 1 and 2 are used internally for the pacer */ /* counters 1 and 2 are used internally for the pacer */
comedi_8254_set_busy(dev->pacer, 1, true); comedi_8254_set_busy(dev->pacer, 1, true);
comedi_8254_set_busy(dev->pacer, 2, true); comedi_8254_set_busy(dev->pacer, 2, true);
subdev++;
} }
/* max_samples is half the FIFO size (2 bytes/sample) */ /* max_samples is half the FIFO size (2 bytes/sample) */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment