staging: comedi: addi_apci_16xx: only allocate needed subdevices
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Fri, 18 Jan 2013 17:46:17 +0000 (10:46 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Jan 2013 20:57:23 +0000 (12:57 -0800)
The addi-data "common" code always allocated 7 subdevices. This
driver only uses 1. Change the allocation and remove the unused
subdevices.

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

index 8078457dd2bed6970aec8d9e974de895a193cce1..5f1de019f5159f47c66bc01fd2b1a4e4f69b771f 100644 (file)
@@ -55,7 +55,7 @@ static int apci16xx_auto_attach(struct comedi_device *dev,
        const struct addi_board *this_board;
        struct addi_private *devpriv;
        struct comedi_subdevice *s;
-       int ret, n_subdevices;
+       int ret;
 
        this_board = addi_find_boardinfo(dev, pcidev);
        if (!this_board)
@@ -74,53 +74,24 @@ static int apci16xx_auto_attach(struct comedi_device *dev,
 
        dev->iobase = pci_resource_start(pcidev, 0);
 
-       n_subdevices = 7;
-       ret = comedi_alloc_subdevices(dev, n_subdevices);
+       ret = comedi_alloc_subdevices(dev, 1);
        if (ret)
                return ret;
 
-       /*  Allocate and Initialise AI Subdevice Structures */
+       /* Initialize the TTL digital i/o */
        s = &dev->subdevices[0];
-       s->type = COMEDI_SUBD_UNUSED;
-
-       /*  Allocate and Initialise AO Subdevice Structures */
-       s = &dev->subdevices[1];
-       s->type = COMEDI_SUBD_UNUSED;
-
-       /*  Allocate and Initialise DI Subdevice Structures */
-       s = &dev->subdevices[2];
-       s->type = COMEDI_SUBD_UNUSED;
-
-       /*  Allocate and Initialise DO Subdevice Structures */
-       s = &dev->subdevices[3];
-       s->type = COMEDI_SUBD_UNUSED;
-
-       /*  Allocate and Initialise Timer Subdevice Structures */
-       s = &dev->subdevices[4];
-       s->type = COMEDI_SUBD_UNUSED;
-
-       /*  Allocate and Initialise TTL */
-       s = &dev->subdevices[5];
-       if (this_board->i_NbrTTLChannel) {
-               s->type = COMEDI_SUBD_TTLIO;
-               s->subdev_flags =
-                       SDF_WRITEABLE | SDF_READABLE | SDF_GROUND | SDF_COMMON;
-               s->n_chan = this_board->i_NbrTTLChannel;
-               s->maxdata = 1;
-               s->io_bits = 0; /* all bits input */
-               s->len_chanlist = this_board->i_NbrTTLChannel;
-               s->range_table = &range_digital;
-               s->insn_config = this_board->ttl_config;
-               s->insn_bits = this_board->ttl_bits;
-               s->insn_read = this_board->ttl_read;
-               s->insn_write = this_board->ttl_write;
-       } else {
-               s->type = COMEDI_SUBD_UNUSED;
-       }
-
-       /* EEPROM */
-       s = &dev->subdevices[6];
-       s->type = COMEDI_SUBD_UNUSED;
+       s->type = COMEDI_SUBD_TTLIO;
+       s->subdev_flags =
+               SDF_WRITEABLE | SDF_READABLE | SDF_GROUND | SDF_COMMON;
+       s->n_chan = this_board->i_NbrTTLChannel;
+       s->maxdata = 1;
+       s->io_bits = 0; /* all bits input */
+       s->len_chanlist = this_board->i_NbrTTLChannel;
+       s->range_table = &range_digital;
+       s->insn_config = this_board->ttl_config;
+       s->insn_bits = this_board->ttl_bits;
+       s->insn_read = this_board->ttl_read;
+       s->insn_write = this_board->ttl_write;
 
        return 0;
 }