staging: comedi: addi_apci_3501: remove devpriv->iobase usage
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Wed, 23 Jan 2013 19:39:47 +0000 (12:39 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 25 Jan 2013 20:00:28 +0000 (12:00 -0800)
The iobase address stored in devpriv->iobase is also stored in
dev->iobase. Use that instead.

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-data/hwdrv_apci3501.c
drivers/staging/comedi/drivers/addi_apci_3501.c

index 74b6eacf5e932c6e0cd9b32b98390878b1d8a50e..41a3477c153ed62ff915fe5cf2e1cc25f1ce76ed 100644 (file)
@@ -118,7 +118,7 @@ static int i_APCI3501_ConfigAnalogOutput(struct comedi_device *dev,
        struct addi_private *devpriv = dev->private;
 
        outl(data[0],
-               devpriv->iobase + APCI3501_ANALOG_OUTPUT +
+               dev->iobase + APCI3501_ANALOG_OUTPUT +
                APCI3501_AO_VOLT_MODE);
 
        if (data[0]) {
@@ -181,10 +181,10 @@ static int i_APCI3501_WriteAnalogOutput(struct comedi_device *dev,
                printk("\nIn WriteAnalogOutput :: Not Valid Channel\n");
        }                       /*  end if((ul_Channel_no<0)||(ul_Channel_no>7)) */
 
-       ul_DAC_Ready = inl(devpriv->iobase + APCI3501_ANALOG_OUTPUT);
+       ul_DAC_Ready = inl(dev->iobase + APCI3501_ANALOG_OUTPUT);
 
        while (ul_DAC_Ready == 0) {
-               ul_DAC_Ready = inl(devpriv->iobase + APCI3501_ANALOG_OUTPUT);
+               ul_DAC_Ready = inl(dev->iobase + APCI3501_ANALOG_OUTPUT);
                ul_DAC_Ready = (ul_DAC_Ready >> 8) & 1;
        }
 
@@ -195,7 +195,7 @@ static int i_APCI3501_WriteAnalogOutput(struct comedi_device *dev,
                        (unsigned int) ((*data << 0x8) & 0x7FFFFF00L) |
                        (unsigned int) (ul_Polarity));
                outl(ul_Command1,
-                       devpriv->iobase + APCI3501_ANALOG_OUTPUT +
+                       dev->iobase + APCI3501_ANALOG_OUTPUT +
                        APCI3501_AO_PROG);
        }
 
@@ -242,67 +242,67 @@ static int i_APCI3501_ConfigTimerCounterWatchdog(struct comedi_device *dev,
 
                devpriv->b_TimerSelectMode = ADDIDATA_WATCHDOG;
                /* Disable the watchdog */
-               outl(0x0, devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);     /* disable Wa */
+               outl(0x0, dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG); /* disable Wa */
 
                if (data[1] == 1) {
                        /* Enable TIMER int & DISABLE ALL THE OTHER int SOURCES */
                        outl(0x02,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                } else {
-                       outl(0x0, devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);     /* disable Timer interrupt */
+                       outl(0x0, dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG); /* disable Timer interrupt */
                }
 
                /* Loading the Timebase value */
                outl(data[2],
-                       devpriv->iobase + APCI3501_WATCHDOG +
+                       dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_TIMEBASE);
 
                /* Loading the Reload value */
                outl(data[3],
-                       devpriv->iobase + APCI3501_WATCHDOG +
+                       dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_RELOAD_VALUE);
                /* Set the mode */
-               ul_Command1 = inl(devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG) | 0xFFF819E0UL;      /* e2->e0 */
+               ul_Command1 = inl(dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG) | 0xFFF819E0UL;  /* e2->e0 */
                outl(ul_Command1,
-                       devpriv->iobase + APCI3501_WATCHDOG +
+                       dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_PROG);
        }                       /* end if(data[0]==ADDIDATA_WATCHDOG) */
 
        else if (data[0] == ADDIDATA_TIMER) {
                /* First Stop The Timer */
                ul_Command1 =
-                       inl(devpriv->iobase + APCI3501_WATCHDOG +
+                       inl(dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_PROG);
                ul_Command1 = ul_Command1 & 0xFFFFF9FEUL;
-               outl(ul_Command1, devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);     /* Stop The Timer */
+               outl(ul_Command1, dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG); /* Stop The Timer */
                devpriv->b_TimerSelectMode = ADDIDATA_TIMER;
                if (data[1] == 1) {
                        /* Enable TIMER int & DISABLE ALL THE OTHER int SOURCES */
                        outl(0x02,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                } else {
-                       outl(0x0, devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);     /* disable Timer interrupt */
+                       outl(0x0, dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG); /* disable Timer interrupt */
                }
 
                /*  Loading Timebase */
                outl(data[2],
-                       devpriv->iobase + APCI3501_WATCHDOG +
+                       dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_TIMEBASE);
 
                /* Loading the Reload value */
                outl(data[3],
-                       devpriv->iobase + APCI3501_WATCHDOG +
+                       dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_RELOAD_VALUE);
 
-               /*  printk ("\nTimer Address :: %x\n", (devpriv->iobase+APCI3501_WATCHDOG)); */
+               /*  printk ("\nTimer Address :: %x\n", (dev->iobase + APCI3501_WATCHDOG)); */
                ul_Command1 =
-                       inl(devpriv->iobase + APCI3501_WATCHDOG +
+                       inl(dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_PROG);
                ul_Command1 =
                        (ul_Command1 & 0xFFF719E2UL) | 2UL << 13UL | 0x10UL;
-               outl(ul_Command1, devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);     /* mode 2 */
+               outl(ul_Command1, dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG); /* mode 2 */
 
        }                       /* end if(data[0]==ADDIDATA_TIMER) */
 
@@ -347,12 +347,12 @@ static int i_APCI3501_StartStopWriteTimerCounterWatchdog(struct comedi_device *d
 
                if (data[1] == 1) {
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = (ul_Command1 & 0xFFFFF9FFUL) | 0x1UL;
                        /* Enable the Watchdog */
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                }
 
@@ -360,19 +360,19 @@ static int i_APCI3501_StartStopWriteTimerCounterWatchdog(struct comedi_device *d
                {
                        /* Stop The Watchdog */
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = ul_Command1 & 0xFFFFF9FEUL;
                        outl(0x0,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                } else if (data[1] == 2) {
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = (ul_Command1 & 0xFFFFF9FFUL) | 0x200UL;
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                }               /* if(data[1]==2) */
        }                       /*  end if (devpriv->b_TimerSelectMode==ADDIDATA_WATCHDOG) */
@@ -381,37 +381,37 @@ static int i_APCI3501_StartStopWriteTimerCounterWatchdog(struct comedi_device *d
                if (data[1] == 1) {
 
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = (ul_Command1 & 0xFFFFF9FFUL) | 0x1UL;
                        /* Enable the Timer */
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                } else if (data[1] == 0) {
                        /* Stop The Timer */
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = ul_Command1 & 0xFFFFF9FEUL;
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                }
 
                else if (data[1] == 2) {
                        /* Trigger the Timer */
                        ul_Command1 =
-                               inl(devpriv->iobase + APCI3501_WATCHDOG +
+                               inl(dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                        ul_Command1 = (ul_Command1 & 0xFFFFF9FFUL) | 0x200UL;
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_WATCHDOG +
+                               dev->iobase + APCI3501_WATCHDOG +
                                APCI3501_TCW_PROG);
                }
 
        }                       /*  end if (devpriv->b_TimerSelectMode==ADDIDATA_TIMER) */
-       i_Temp = inl(devpriv->iobase + APCI3501_WATCHDOG +
+       i_Temp = inl(dev->iobase + APCI3501_WATCHDOG +
                APCI3501_TCW_TRIG_STATUS) & 0x1;
        return insn->n;
 }
@@ -449,16 +449,16 @@ static int i_APCI3501_ReadTimerCounterWatchdog(struct comedi_device *dev,
 
        if (devpriv->b_TimerSelectMode == ADDIDATA_WATCHDOG) {
                data[0] =
-                       inl(devpriv->iobase + APCI3501_WATCHDOG +
+                       inl(dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_TRIG_STATUS) & 0x1;
-               data[1] = inl(devpriv->iobase + APCI3501_WATCHDOG);
+               data[1] = inl(dev->iobase + APCI3501_WATCHDOG);
        }                       /*  end if  (devpriv->b_TimerSelectMode==ADDIDATA_WATCHDOG) */
 
        else if (devpriv->b_TimerSelectMode == ADDIDATA_TIMER) {
                data[0] =
-                       inl(devpriv->iobase + APCI3501_WATCHDOG +
+                       inl(dev->iobase + APCI3501_WATCHDOG +
                        APCI3501_TCW_TRIG_STATUS) & 0x1;
-               data[1] = inl(devpriv->iobase + APCI3501_WATCHDOG);
+               data[1] = inl(dev->iobase + APCI3501_WATCHDOG);
        }                       /*  end if  (devpriv->b_TimerSelectMode==ADDIDATA_TIMER) */
 
        else if ((devpriv->b_TimerSelectMode != ADDIDATA_TIMER)
index 3f8f26ac4df74c573faaf418e7864b4b1fc59d46..16a348cd69fb4563142fb9e3486611a6a2e41d83 100644 (file)
@@ -28,9 +28,7 @@ static int apci3501_di_insn_bits(struct comedi_device *dev,
                                 struct comedi_insn *insn,
                                 unsigned int *data)
 {
-       struct addi_private *devpriv = dev->private;
-
-       data[1] = inl(devpriv->iobase + APCI3501_DIGITAL_IP) & 0x3;
+       data[1] = inl(dev->iobase + APCI3501_DIGITAL_IP) & 0x3;
 
        return insn->n;
 }
@@ -40,16 +38,15 @@ static int apci3501_do_insn_bits(struct comedi_device *dev,
                                 struct comedi_insn *insn,
                                 unsigned int *data)
 {
-       struct addi_private *devpriv = dev->private;
        unsigned int mask = data[0];
        unsigned int bits = data[1];
 
-       s->state = inl(devpriv->iobase + APCI3501_DIGITAL_OP);
+       s->state = inl(dev->iobase + APCI3501_DIGITAL_OP);
        if (mask) {
                s->state &= ~mask;
                s->state |= (bits & mask);
 
-               outl(s->state, devpriv->iobase + APCI3501_DIGITAL_OP);
+               outl(s->state, dev->iobase + APCI3501_DIGITAL_OP);
        }
 
        data[1] = s->state;
@@ -84,14 +81,14 @@ static irqreturn_t apci3501_interrupt(int irq, void *d)
 
        /*  Disable Interrupt */
        ul_Command1 =
-               inl(devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
+               inl(dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
 
        ul_Command1 = (ul_Command1 & 0xFFFFF9FDul);
        outl(ul_Command1,
-               devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
+               dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
 
        ui_Timer_AOWatchdog =
-               inl(devpriv->iobase + APCI3501_WATCHDOG +
+               inl(dev->iobase + APCI3501_WATCHDOG +
                APCI3501_TCW_IRQ) & 0x1;
 
        if ((!ui_Timer_AOWatchdog)) {
@@ -102,11 +99,11 @@ static irqreturn_t apci3501_interrupt(int irq, void *d)
        /* Enable Interrupt Send a signal to from kernel to user space */
        send_sig(SIGIO, devpriv->tsk_Current, 0);
        ul_Command1 =
-               inl(devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
+               inl(dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
        ul_Command1 = ((ul_Command1 & 0xFFFFF9FDul) | 1 << 1);
        outl(ul_Command1,
-               devpriv->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
-       i_temp = inl(devpriv->iobase + APCI3501_WATCHDOG +
+               dev->iobase + APCI3501_WATCHDOG + APCI3501_TCW_PROG);
+       i_temp = inl(dev->iobase + APCI3501_WATCHDOG +
                APCI3501_TCW_TRIG_STATUS) & 0x1;
 
        return IRQ_HANDLED;
@@ -114,22 +111,21 @@ static irqreturn_t apci3501_interrupt(int irq, void *d)
 
 static int apci3501_reset(struct comedi_device *dev)
 {
-       struct addi_private *devpriv = dev->private;
        int i_Count = 0, i_temp = 0;
        unsigned int ul_Command1 = 0, ul_Polarity, ul_DAC_Ready = 0;
 
-       outl(0x0, devpriv->iobase + APCI3501_DIGITAL_OP);
-       outl(1, devpriv->iobase + APCI3501_ANALOG_OUTPUT +
+       outl(0x0, dev->iobase + APCI3501_DIGITAL_OP);
+       outl(1, dev->iobase + APCI3501_ANALOG_OUTPUT +
                APCI3501_AO_VOLT_MODE);
 
        ul_Polarity = 0x80000000;
 
        for (i_Count = 0; i_Count <= 7; i_Count++) {
-               ul_DAC_Ready = inl(devpriv->iobase + APCI3501_ANALOG_OUTPUT);
+               ul_DAC_Ready = inl(dev->iobase + APCI3501_ANALOG_OUTPUT);
 
                while (ul_DAC_Ready == 0) {
                        ul_DAC_Ready =
-                               inl(devpriv->iobase + APCI3501_ANALOG_OUTPUT);
+                               inl(dev->iobase + APCI3501_ANALOG_OUTPUT);
                        ul_DAC_Ready = (ul_DAC_Ready >> 8) & 1;
                }
 
@@ -140,7 +136,7 @@ static int apci3501_reset(struct comedi_device *dev)
                                (unsigned int) ((i_temp << 0x8) & 0x7FFFFF00L) |
                                (unsigned int) (ul_Polarity));
                        outl(ul_Command1,
-                               devpriv->iobase + APCI3501_ANALOG_OUTPUT +
+                               dev->iobase + APCI3501_ANALOG_OUTPUT +
                                APCI3501_AO_PROG);
                }
        }
@@ -190,7 +186,6 @@ static int apci3501_auto_attach(struct comedi_device *dev,
                return ret;
 
        dev->iobase = pci_resource_start(pcidev, 1);
-       devpriv->iobase = dev->iobase;
        devpriv->i_IobaseAmcc = pci_resource_start(pcidev, 0);
        devpriv->i_IobaseAddon = pci_resource_start(pcidev, 2);
        devpriv->i_IobaseReserved = pci_resource_start(pcidev, 3);