staging: comedi: adl_pci9118: remove 'ai16bits' from private data
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 5 May 2014 16:35:43 +0000 (09:35 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 May 2014 12:18:55 +0000 (21:18 +0900)
This member of the private data is only set when the analog input
subdevice maxdata is 0xffff. Just check the s->maxdata and remove
the private data member.

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

index 394337e0561772094cd1ecbc33c0fce8017235e2..0e9fc0de8c5ab83e2b9a2ad75ee1704dc16727b2 100644 (file)
@@ -376,7 +376,6 @@ struct pci9118_private {
                                         * ptr to actual interrupt
                                         * AI function
                                         */
-       unsigned char ai16bits;         /* =1 16 bit card */
        unsigned char usedma;           /* =1 use DMA transfer and not INT */
        int softsshdelay;               /*
                                         * >0 use software S&H,
@@ -586,7 +585,7 @@ static int pci9118_insn_read_ai(struct comedi_device *dev,
                        return ret;
                }
 
-               if (devpriv->ai16bits) {
+               if (s->maxdata == 0xffff) {
                        data[n] =
                            (inl(dev->iobase +
                                 PCI9118_AD_DATA) & 0xffff) ^ 0x8000;
@@ -908,7 +907,7 @@ static void pci9118_ai_munge(struct comedi_device *dev,
        for (i = 0; i < num_samples; i++) {
                if (devpriv->usedma)
                        array[i] = be16_to_cpu(array[i]);
-               if (devpriv->ai16bits)
+               if (s->maxdata == 0xffff)
                        array[i] ^= 0x8000;
                else
                        array[i] = (array[i] >> 4) & 0x0fff;
@@ -933,7 +932,7 @@ static void interrupt_pci9118_ai_onesample(struct comedi_device *dev,
        sampl = inw(dev->iobase + PCI9118_AD_DATA);
 
 #ifdef PCI9118_PARANOIDCHECK
-       if (devpriv->ai16bits == 0) {
+       if (s->maxdata != 0xffff) {
                if ((sampl & 0x000f) != devpriv->chanlist[s->async->cur_chan]) {
                                                        /* data dropout! */
                        dev_info(dev->class_dev,
@@ -2017,15 +2016,6 @@ static int pci9118_common_attach(struct comedi_device *dev, int disable_irq,
        if (hw_err_mask)                /* disable some requested */
                devpriv->ai_maskharderr &= ~hw_err_mask;
 
-       switch (this_board->ai_maxdata) {
-       case 0xffff:
-               devpriv->ai16bits = 1;
-               break;
-       default:
-               devpriv->ai16bits = 0;
-               break;
-       }
-
        return 0;
 }