staging: comedi: addi_apci_3120: simplify analog output boardinfo
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 14 Oct 2014 17:44:45 +0000 (10:44 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 29 Oct 2014 07:48:55 +0000 (15:48 +0800)
There are two boards supported by this driver. One with eight 14-bit analog
outputs and the other without analog outputs.

Add a bit field flag, 'has_ao', to the boardinfo and remove the 'i_NbrAoChannel'
and 'i_AoMaxdata' members. Use the new flag to determine if the analog output
subdevice needs to be initialized and open code the n_chan and maxdata.

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

index d555427d3359668f228cd9e0a046af98bb04189e..de1dabf72bb38d9879c8025814d3e5bcc045340d 100644 (file)
@@ -16,17 +16,15 @@ enum apci3120_boardid {
 
 struct apci3120_board {
        const char *name;
-       int i_NbrAoChannel;
        int i_AiMaxdata;
-       int i_AoMaxdata;
+       unsigned int has_ao:1;
 };
 
 static const struct apci3120_board apci3120_boardtypes[] = {
        [BOARD_APCI3120] = {
                .name                   = "apci3120",
-               .i_NbrAoChannel         = 8,
                .i_AiMaxdata            = 0xffff,
-               .i_AoMaxdata            = 0x3fff,
+               .has_ao                 = 1,
        },
        [BOARD_APCI3001] = {
                .name                   = "apci3001",
@@ -117,12 +115,11 @@ static int apci3120_auto_attach(struct comedi_device *dev,
 
        /*  Allocate and Initialise AO Subdevice Structures */
        s = &dev->subdevices[1];
-       if (this_board->i_NbrAoChannel) {
+       if (this_board->has_ao) {
                s->type = COMEDI_SUBD_AO;
                s->subdev_flags = SDF_WRITEABLE | SDF_GROUND | SDF_COMMON;
-               s->n_chan = this_board->i_NbrAoChannel;
-               s->maxdata = this_board->i_AoMaxdata;
-               s->len_chanlist = this_board->i_NbrAoChannel;
+               s->n_chan = 8;
+               s->maxdata = 0x3fff;
                s->range_table = &range_apci3120_ao;
                s->insn_write = apci3120_ao_insn_write;
        } else {