staging: comedi: addi_apci_3xxx: remove 'iobase' from private data
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Wed, 12 Jun 2013 23:17:28 +0000 (16:17 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Jun 2013 21:33:44 +0000 (14:33 -0700)
The private data 'iobase' is the same as the comedi_device 'iobase'.
Use the one in the comedi_device instead.

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_3xxx.c

index 39e7fb679ab85892917be72bdb3d0ab05d419f4f..90f9749703945701d778b8f835c666ac99c747a2 100644 (file)
@@ -367,7 +367,6 @@ static const struct apci3xxx_boardinfo apci3xxx_boardtypes[] = {
 };
 
 struct apci3xxx_private {
-       int iobase;
        void __iomem *dw_AiBase;
        unsigned int ui_AiNbrofChannels;        /*  how many channels is measured */
        unsigned int ui_AiReadData[32];
@@ -445,9 +444,7 @@ static int apci3xxx_di_insn_bits(struct comedi_device *dev,
                                 struct comedi_insn *insn,
                                 unsigned int *data)
 {
-       struct apci3xxx_private *devpriv = dev->private;
-
-       data[1] = inl(devpriv->iobase + 32) & 0xf;
+       data[1] = inl(dev->iobase + 32) & 0xf;
 
        return insn->n;
 }
@@ -457,16 +454,15 @@ static int apci3xxx_do_insn_bits(struct comedi_device *dev,
                                 struct comedi_insn *insn,
                                 unsigned int *data)
 {
-       struct apci3xxx_private *devpriv = dev->private;
        unsigned int mask = data[0];
        unsigned int bits = data[1];
 
-       s->state = inl(devpriv->iobase + 48) & 0xf;
+       s->state = inl(dev->iobase + 48) & 0xf;
        if (mask) {
                s->state &= ~mask;
                s->state |= (bits & mask);
 
-               outl(s->state, devpriv->iobase + 48);
+               outl(s->state, dev->iobase + 48);
        }
 
        data[1] = s->state;
@@ -479,7 +475,6 @@ static int apci3xxx_dio_insn_config(struct comedi_device *dev,
                                    struct comedi_insn *insn,
                                    unsigned int *data)
 {
-       struct apci3xxx_private *devpriv = dev->private;
        unsigned int chan = CR_CHAN(insn->chanspec);
        unsigned int mask = 1 << chan;
        unsigned int bits;
@@ -512,7 +507,7 @@ static int apci3xxx_dio_insn_config(struct comedi_device *dev,
 
        /* update port 2 configuration */
        if (bits)
-               outl((s->io_bits >> 24) & 0xff, devpriv->iobase + 224);
+               outl((s->io_bits >> 24) & 0xff, dev->iobase + 224);
 
        return insn->n;
 }
@@ -522,7 +517,6 @@ static int apci3xxx_dio_insn_bits(struct comedi_device *dev,
                                  struct comedi_insn *insn,
                                  unsigned int *data)
 {
-       struct apci3xxx_private *devpriv = dev->private;
        unsigned int mask = data[0];
        unsigned int bits = data[1];
        unsigned int val;
@@ -534,17 +528,17 @@ static int apci3xxx_dio_insn_bits(struct comedi_device *dev,
                s->state |= (bits & mask);
 
                if (mask & 0xff)
-                       outl(s->state & 0xff, devpriv->iobase + 80);
+                       outl(s->state & 0xff, dev->iobase + 80);
                if (mask & 0xff0000)
-                       outl((s->state >> 16) & 0xff, devpriv->iobase + 112);
+                       outl((s->state >> 16) & 0xff, dev->iobase + 112);
        }
 
-       val = inl(devpriv->iobase + 80);
-       val |= (inl(devpriv->iobase + 64) << 8);
+       val = inl(dev->iobase + 80);
+       val |= (inl(dev->iobase + 64) << 8);
        if (s->io_bits & 0xff0000)
-               val |= (inl(devpriv->iobase + 112) << 16);
+               val |= (inl(dev->iobase + 112) << 16);
        else
-               val |= (inl(devpriv->iobase + 96) << 16);
+               val |= (inl(dev->iobase + 96) << 16);
 
        data[1] = val;
 
@@ -608,9 +602,7 @@ static int apci3xxx_auto_attach(struct comedi_device *dev,
        if (ret)
                return ret;
 
-       /* board has an ADDIDATA_9054 eeprom */
        dev->iobase = pci_resource_start(pcidev, 2);
-       devpriv->iobase = pci_resource_start(pcidev, 2);
        devpriv->dw_AiBase = pci_ioremap_bar(pcidev, 3);
 
        if (pcidev->irq > 0) {