Staging: fix assorted typos all over the place
[safe/jmp/linux-2.6] / drivers / staging / comedi / drivers / amplc_pci230.c
index ee5f203..091a1a5 100644 (file)
@@ -187,9 +187,11 @@ Extra triggered scan functionality, interrupt bug-fix added by Steve Sharples.
 Support for PCI230+/260+, more triggered scan functionality, and workarounds
 for (or detection of) various hardware problems added by Ian Abbott.
 */
+
 #include "../comedidev.h"
 
 #include <linux/delay.h>
+#include <linux/interrupt.h>
 
 #include "comedi_pci.h"
 #include "8253.h"
@@ -443,7 +445,7 @@ enum {
  * Board descriptions for the two boards supported.
  */
 
-typedef struct pci230_board_struct {
+struct pci230_board {
        const char *name;
        unsigned short id;
        int ai_chans;
@@ -452,66 +454,67 @@ typedef struct pci230_board_struct {
        int ao_bits;
        int have_dio;
        unsigned int min_hwver; /* Minimum hardware version supported. */
-} pci230_board;
-static const pci230_board pci230_boards[] = {
+};
+static const struct pci230_board pci230_boards[] = {
        {
-             name:     "pci230+",
-             id:       PCI_DEVICE_ID_PCI230,
-             ai_chans:16,
-             ai_bits:  16,
-             ao_chans:2,
-             ao_bits:  12,
-             have_dio:1,
-             min_hwver:1,
-               },
+        .name = "pci230+",
+        .id = PCI_DEVICE_ID_PCI230,
+        .ai_chans = 16,
+        .ai_bits = 16,
+        .ao_chans = 2,
+        .ao_bits = 12,
+        .have_dio = 1,
+        .min_hwver = 1,
+        },
        {
-             name:     "pci260+",
-             id:       PCI_DEVICE_ID_PCI260,
-             ai_chans:16,
-             ai_bits:  16,
-             ao_chans:0,
-             ao_bits:  0,
-             have_dio:0,
-             min_hwver:1,
-               },
+        .name = "pci260+",
+        .id = PCI_DEVICE_ID_PCI260,
+        .ai_chans = 16,
+        .ai_bits = 16,
+        .ao_chans = 0,
+        .ao_bits = 0,
+        .have_dio = 0,
+        .min_hwver = 1,
+        },
        {
-             name:     "pci230",
-             id:       PCI_DEVICE_ID_PCI230,
-             ai_chans:16,
-             ai_bits:  12,
-             ao_chans:2,
-             ao_bits:  12,
-             have_dio:1,
-               },
+        .name = "pci230",
+        .id = PCI_DEVICE_ID_PCI230,
+        .ai_chans = 16,
+        .ai_bits = 12,
+        .ao_chans = 2,
+        .ao_bits = 12,
+        .have_dio = 1,
+        },
        {
-             name:     "pci260",
-             id:       PCI_DEVICE_ID_PCI260,
-             ai_chans:16,
-             ai_bits:  12,
-             ao_chans:0,
-             ao_bits:  0,
-             have_dio:0,
-               },
+        .name = "pci260",
+        .id = PCI_DEVICE_ID_PCI260,
+        .ai_chans = 16,
+        .ai_bits = 12,
+        .ao_chans = 0,
+        .ao_bits = 0,
+        .have_dio = 0,
+        },
        {
-             name:     "amplc_pci230", /* Wildcard matches any above */
-             id:       PCI_DEVICE_ID_INVALID,
-               },
+        .name = "amplc_pci230",        /* Wildcard matches any above */
+        .id = PCI_DEVICE_ID_INVALID,
+        },
 };
 
 static DEFINE_PCI_DEVICE_TABLE(pci230_pci_table) = {
-       {PCI_VENDOR_ID_AMPLICON, PCI_DEVICE_ID_PCI230, PCI_ANY_ID, PCI_ANY_ID,
-               0, 0, 0},
-       {PCI_VENDOR_ID_AMPLICON, PCI_DEVICE_ID_PCI260, PCI_ANY_ID, PCI_ANY_ID,
-               0, 0, 0},
-       {0}
+       {
+       PCI_VENDOR_ID_AMPLICON, PCI_DEVICE_ID_PCI230, PCI_ANY_ID,
+                   PCI_ANY_ID, 0, 0, 0}, {
+       PCI_VENDOR_ID_AMPLICON, PCI_DEVICE_ID_PCI260, PCI_ANY_ID,
+                   PCI_ANY_ID, 0, 0, 0}, {
+       0}
 };
 
 MODULE_DEVICE_TABLE(pci, pci230_pci_table);
 /*
  * Useful for shorthand access to the particular board structure
  */
-#define n_pci230_boards (sizeof(pci230_boards)/sizeof(pci230_boards[0]))
-#define thisboard ((const pci230_board *)dev->board_ptr)
+#define n_pci230_boards ARRAY_SIZE(pci230_boards)
+#define thisboard ((const struct pci230_board *)dev->board_ptr)
 
 /* this structure is for data unique to this hardware driver.  If
    several hardware drivers keep similar information in this structure,
@@ -568,15 +571,15 @@ static const unsigned int pci230_timebase[8] = {
 };
 
 /* PCI230 analogue input range table */
-static const comedi_lrange pci230_ai_range = { 7, {
-                       BIP_RANGE(10),
-                       BIP_RANGE(5),
-                       BIP_RANGE(2.5),
-                       BIP_RANGE(1.25),
-                       UNI_RANGE(10),
-                       UNI_RANGE(5),
-                       UNI_RANGE(2.5)
-       }
+static const struct comedi_lrange pci230_ai_range = { 7, {
+                                                         BIP_RANGE(10),
+                                                         BIP_RANGE(5),
+                                                         BIP_RANGE(2.5),
+                                                         BIP_RANGE(1.25),
+                                                         UNI_RANGE(10),
+                                                         UNI_RANGE(5),
+                                                         UNI_RANGE(2.5)
+                                                         }
 };
 
 /* PCI230 analogue gain bits for each input range. */
@@ -586,64 +589,78 @@ static const unsigned char pci230_ai_gain[7] = { 0, 1, 2, 3, 1, 2, 3 };
 static const unsigned char pci230_ai_bipolar[7] = { 1, 1, 1, 1, 0, 0, 0 };
 
 /* PCI230 analogue output range table */
-static const comedi_lrange pci230_ao_range = { 2, {
-                       UNI_RANGE(10),
-                       BIP_RANGE(10)
-       }
+static const struct comedi_lrange pci230_ao_range = { 2, {
+                                                         UNI_RANGE(10),
+                                                         BIP_RANGE(10)
+                                                         }
 };
 
 /* PCI230 daccon bipolar flag for each analogue output range. */
 static const unsigned char pci230_ao_bipolar[2] = { 0, 1 };
 
 /*
- * The comedi_driver structure tells the Comedi core module
+ * The struct comedi_driver structure tells the Comedi core module
  * which functions to call to configure/deconfigure (attach/detach)
  * the board, and also about the kernel module that contains
  * the device code.
  */
-static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it);
-static int pci230_detach(struct comedi_device * dev);
-static comedi_driver driver_amplc_pci230 = {
-      driver_name:"amplc_pci230",
-      module:THIS_MODULE,
-      attach:pci230_attach,
-      detach:pci230_detach,
-      board_name:&pci230_boards[0].name,
-      offset:sizeof(pci230_boards[0]),
-      num_names:sizeof(pci230_boards) / sizeof(pci230_boards[0]),
+static int pci230_attach(struct comedi_device *dev,
+                        struct comedi_devconfig *it);
+static int pci230_detach(struct comedi_device *dev);
+static struct comedi_driver driver_amplc_pci230 = {
+       .driver_name = "amplc_pci230",
+       .module = THIS_MODULE,
+       .attach = pci230_attach,
+       .detach = pci230_detach,
+       .board_name = &pci230_boards[0].name,
+       .offset = sizeof(pci230_boards[0]),
+       .num_names = ARRAY_SIZE(pci230_boards),
 };
 
 COMEDI_PCI_INITCLEANUP(driver_amplc_pci230, pci230_pci_table);
 
-static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data);
-static int pci230_ao_winsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data);
-static int pci230_ao_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data);
-static void pci230_ct_setup_ns_mode(struct comedi_device * dev, unsigned int ct,
-       unsigned int mode, uint64_t ns, unsigned int round);
+static int pci230_ai_rinsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data);
+static int pci230_ao_winsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data);
+static int pci230_ao_rinsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data);
+static void pci230_ct_setup_ns_mode(struct comedi_device *dev, unsigned int ct,
+                                   unsigned int mode, uint64_t ns,
+                                   unsigned int round);
 static void pci230_ns_to_single_timer(unsigned int *ns, unsigned int round);
-static void pci230_cancel_ct(struct comedi_device * dev, unsigned int ct);
-static irqreturn_t pci230_interrupt(int irq, void *d PT_REGS_ARG);
-static int pci230_ao_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_cmd * cmd);
-static int pci230_ao_cmd(struct comedi_device * dev, struct comedi_subdevice * s);
-static int pci230_ao_cancel(struct comedi_device * dev, struct comedi_subdevice * s);
-static void pci230_ao_stop(struct comedi_device * dev, struct comedi_subdevice * s);
-static void pci230_handle_ao_nofifo(struct comedi_device * dev, struct comedi_subdevice * s);
-static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subdevice * s);
-static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_cmd * cmd);
-static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s);
-static int pci230_ai_cancel(struct comedi_device * dev, struct comedi_subdevice * s);
-static void pci230_ai_stop(struct comedi_device * dev, struct comedi_subdevice * s);
-static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice * s);
-
-static short pci230_ai_read(struct comedi_device * dev)
+static void pci230_cancel_ct(struct comedi_device *dev, unsigned int ct);
+static irqreturn_t pci230_interrupt(int irq, void *d);
+static int pci230_ao_cmdtest(struct comedi_device *dev,
+                            struct comedi_subdevice *s,
+                            struct comedi_cmd *cmd);
+static int pci230_ao_cmd(struct comedi_device *dev, struct comedi_subdevice *s);
+static int pci230_ao_cancel(struct comedi_device *dev,
+                           struct comedi_subdevice *s);
+static void pci230_ao_stop(struct comedi_device *dev,
+                          struct comedi_subdevice *s);
+static void pci230_handle_ao_nofifo(struct comedi_device *dev,
+                                   struct comedi_subdevice *s);
+static int pci230_handle_ao_fifo(struct comedi_device *dev,
+                                struct comedi_subdevice *s);
+static int pci230_ai_cmdtest(struct comedi_device *dev,
+                            struct comedi_subdevice *s,
+                            struct comedi_cmd *cmd);
+static int pci230_ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s);
+static int pci230_ai_cancel(struct comedi_device *dev,
+                           struct comedi_subdevice *s);
+static void pci230_ai_stop(struct comedi_device *dev,
+                          struct comedi_subdevice *s);
+static void pci230_handle_ai(struct comedi_device *dev,
+                            struct comedi_subdevice *s);
+
+static short pci230_ai_read(struct comedi_device *dev)
 {
        /* Read sample. */
-       short data = (short) inw(dev->iobase + PCI230_ADCDATA);
+       short data = (short)inw(dev->iobase + PCI230_ADCDATA);
 
        /* PCI230 is 12 bit - stored in upper bits of 16 bit register (lower
         * four bits reserved for expansion). */
@@ -658,8 +675,8 @@ static short pci230_ai_read(struct comedi_device * dev)
        return data;
 }
 
-static inline unsigned short pci230_ao_mangle_datum(struct comedi_device * dev,
-       short datum)
+static inline unsigned short pci230_ao_mangle_datum(struct comedi_device *dev,
+                                                   short datum)
 {
        /* If a bipolar range was specified, mangle it (straight binary->twos
         * complement). */
@@ -674,26 +691,28 @@ static inline unsigned short pci230_ao_mangle_datum(struct comedi_device * dev,
        return (unsigned short)datum;
 }
 
-static inline void pci230_ao_write_nofifo(struct comedi_device * dev, short datum,
-       unsigned int chan)
+static inline void pci230_ao_write_nofifo(struct comedi_device *dev,
+                                         short datum, unsigned int chan)
 {
        /* Store unmangled datum to be read back later. */
        devpriv->ao_readback[chan] = datum;
 
        /* Write mangled datum to appropriate DACOUT register. */
        outw(pci230_ao_mangle_datum(dev, datum), dev->iobase + (((chan) == 0)
-                       ? PCI230_DACOUT1 : PCI230_DACOUT2));
+                                                               ? PCI230_DACOUT1
+                                                               :
+                                                               PCI230_DACOUT2));
 }
 
-static inline void pci230_ao_write_fifo(struct comedi_device * dev, short datum,
-       unsigned int chan)
+static inline void pci230_ao_write_fifo(struct comedi_device *dev, short datum,
+                                       unsigned int chan)
 {
        /* Store unmangled datum to be read back later. */
        devpriv->ao_readback[chan] = datum;
 
        /* Write mangled datum to appropriate DACDATA register. */
        outw(pci230_ao_mangle_datum(dev, datum),
-               dev->iobase + PCI230P2_DACDATA);
+            dev->iobase + PCI230P2_DACDATA);
 }
 
 /*
@@ -702,7 +721,7 @@ static inline void pci230_ao_write_fifo(struct comedi_device * dev, short datum,
  * in the driver structure, dev->board_ptr contains that
  * address.
  */
-static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
+static int pci230_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 {
        struct comedi_subdevice *s;
        unsigned long iobase1, iobase2;
@@ -711,7 +730,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        int i = 0, irq_hdl, rc;
 
        printk("comedi%d: amplc_pci230: attach %s %d,%d\n", dev->minor,
-               thisboard->name, it->options[0], it->options[1]);
+              thisboard->name, it->options[0], it->options[1]);
 
        /* Allocate the private structure area using alloc_private().
         * Macro defined in comedidev.h - memsets struct fields to 0. */
@@ -724,12 +743,12 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        spin_lock_init(&devpriv->ao_stop_spinlock);
        /* Find card */
        for (pci_dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, NULL);
-               pci_dev != NULL;
-               pci_dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, pci_dev)) {
+            pci_dev != NULL;
+            pci_dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, pci_dev)) {
                if (it->options[0] || it->options[1]) {
                        /* Match against bus/slot options. */
                        if (it->options[0] != pci_dev->bus->number ||
-                               it->options[1] != PCI_SLOT(pci_dev->devfn))
+                           it->options[1] != PCI_SLOT(pci_dev->devfn))
                                continue;
                }
                if (pci_dev->vendor != PCI_VENDOR_ID_AMPLICON)
@@ -746,7 +765,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
                                                 * First check length of
                                                 * registers. */
                                                if (pci_resource_len(pci_dev, 3)
-                                                       < 32) {
+                                                   < 32) {
                                                        /* Not a '+' model. */
                                                        continue;
                                                }
@@ -788,7 +807,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        }
        if (!pci_dev) {
                printk("comedi%d: No %s card found\n", dev->minor,
-                       thisboard->name);
+                      thisboard->name);
                return -EIO;
        }
        devpriv->pci_dev = pci_dev;
@@ -801,7 +820,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        /* Enable PCI device and reserve I/O spaces. */
        if (comedi_pci_enable(pci_dev, "amplc_pci230") < 0) {
                printk("comedi%d: failed to enable PCI device "
-                       "and request regions\n", dev->minor);
+                      "and request regions\n", dev->minor);
                return -EIO;
        }
 
@@ -811,7 +830,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        iobase2 = pci_resource_start(pci_dev, 3);
 
        printk("comedi%d: %s I/O region 1 0x%04lx I/O region 2 0x%04lx\n",
-               dev->minor, dev->board_name, iobase1, iobase2);
+              dev->minor, dev->board_name, iobase1, iobase2);
 
        devpriv->iobase1 = iobase1;
        dev->iobase = iobase2;
@@ -827,9 +846,9 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
                devpriv->hwver = inw(dev->iobase + PCI230P_HWVER);
                if (devpriv->hwver < thisboard->min_hwver) {
                        printk("comedi%d: %s - bad hardware version "
-                               "- got %u, need %u\n", dev->minor,
-                               dev->board_name, devpriv->hwver,
-                               thisboard->min_hwver);
+                              "- got %u, need %u\n", dev->minor,
+                              dev->board_name, devpriv->hwver,
+                              thisboard->min_hwver);
                        return -EIO;
                }
                if (devpriv->hwver > 0) {
@@ -842,7 +861,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
                                extfunc |= PCI230P_EXTFUNC_GAT_EXTTRIG;
                        }
                        if ((thisboard->ao_chans > 0)
-                               && (devpriv->hwver >= 2)) {
+                           && (devpriv->hwver >= 2)) {
                                /* Enable DAC FIFO functionality. */
                                extfunc |= PCI230P2_EXTFUNC_DACFIFO;
                        }
@@ -852,8 +871,8 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
                        /* Temporarily enable DAC FIFO, reset it and disable
                         * FIFO wraparound. */
                        outw(devpriv->daccon | PCI230P2_DAC_FIFO_EN
-                               | PCI230P2_DAC_FIFO_RESET,
-                               dev->iobase + PCI230_DACCON);
+                            | PCI230P2_DAC_FIFO_RESET,
+                            dev->iobase + PCI230_DACCON);
                        /* Clear DAC FIFO channel enable register. */
                        outw(0, dev->iobase + PCI230P2_DACEN);
                        /* Disable DAC FIFO. */
@@ -867,23 +886,23 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        /* Set ADC to a reasonable state. */
        devpriv->adcg = 0;
        devpriv->adccon = PCI230_ADC_TRIG_NONE | PCI230_ADC_IM_SE
-               | PCI230_ADC_IR_BIP;
+           | PCI230_ADC_IR_BIP;
        outw(1 << 0, dev->iobase + PCI230_ADCEN);
        outw(devpriv->adcg, dev->iobase + PCI230_ADCG);
        outw(devpriv->adccon | PCI230_ADC_FIFO_RESET,
-               dev->iobase + PCI230_ADCCON);
+            dev->iobase + PCI230_ADCCON);
 
        /* Register the interrupt handler. */
-       irq_hdl = comedi_request_irq(devpriv->pci_dev->irq, pci230_interrupt,
-               IRQF_SHARED, "amplc_pci230", dev);
+       irq_hdl = request_irq(devpriv->pci_dev->irq, pci230_interrupt,
+                             IRQF_SHARED, "amplc_pci230", dev);
        if (irq_hdl < 0) {
                printk("comedi%d: unable to register irq, "
-                       "commands will not be available %d\n", dev->minor,
-                       devpriv->pci_dev->irq);
+                      "commands will not be available %d\n", dev->minor,
+                      devpriv->pci_dev->irq);
        } else {
                dev->irq = devpriv->pci_dev->irq;
                printk("comedi%d: registered irq %u\n", dev->minor,
-                       devpriv->pci_dev->irq);
+                      devpriv->pci_dev->irq);
        }
 
        /*
@@ -939,7 +958,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
        /* digital i/o subdevice */
        if (thisboard->have_dio) {
                rc = subdev_8255_init(dev, s, NULL,
-                       (devpriv->iobase1 + PCI230_PPI_X_BASE));
+                                     (devpriv->iobase1 + PCI230_PPI_X_BASE));
                if (rc < 0)
                        return rc;
        } else {
@@ -959,7 +978,7 @@ static int pci230_attach(struct comedi_device * dev, comedi_devconfig * it)
  * allocated by _attach().  dev->private and dev->subdevices are
  * deallocated automatically by the core.
  */
-static int pci230_detach(struct comedi_device * dev)
+static int pci230_detach(struct comedi_device *dev)
 {
        printk("comedi%d: amplc_pci230: remove\n", dev->minor);
 
@@ -968,7 +987,7 @@ static int pci230_detach(struct comedi_device * dev)
                subdev_8255_cleanup(dev, dev->subdevices + 2);
 
        if (dev->irq)
-               comedi_free_irq(dev->irq, dev);
+               free_irq(dev->irq, dev);
 
        if (devpriv) {
                if (devpriv->pci_dev) {
@@ -982,8 +1001,8 @@ static int pci230_detach(struct comedi_device * dev)
        return 0;
 }
 
-static int get_resources(struct comedi_device * dev, unsigned int res_mask,
-       unsigned char owner)
+static int get_resources(struct comedi_device *dev, unsigned int res_mask,
+                        unsigned char owner)
 {
        int ok;
        unsigned int i;
@@ -993,9 +1012,9 @@ static int get_resources(struct comedi_device * dev, unsigned int res_mask,
 
        ok = 1;
        claimed = 0;
-       comedi_spin_lock_irqsave(&devpriv->res_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->res_spinlock, irqflags);
        for (b = 1, i = 0; (i < NUM_RESOURCES)
-               && (res_mask != 0); b <<= 1, i++) {
+            && (res_mask != 0); b <<= 1, i++) {
                if ((res_mask & b) != 0) {
                        res_mask &= ~b;
                        if (devpriv->res_owner[i] == OWNER_NONE) {
@@ -1005,7 +1024,7 @@ static int get_resources(struct comedi_device * dev, unsigned int res_mask,
                                for (b = 1, i = 0; claimed != 0; b <<= 1, i++) {
                                        if ((claimed & b) != 0) {
                                                devpriv->res_owner[i]
-                                                       = OWNER_NONE;
+                                                   = OWNER_NONE;
                                                claimed &= ~b;
                                        }
                                }
@@ -1014,26 +1033,26 @@ static int get_resources(struct comedi_device * dev, unsigned int res_mask,
                        }
                }
        }
-       comedi_spin_unlock_irqrestore(&devpriv->res_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->res_spinlock, irqflags);
        return ok;
 }
 
-static inline int get_one_resource(struct comedi_device * dev, unsigned int resource,
-       unsigned char owner)
+static inline int get_one_resource(struct comedi_device *dev,
+                                  unsigned int resource, unsigned char owner)
 {
        return get_resources(dev, (1U << resource), owner);
 }
 
-static void put_resources(struct comedi_device * dev, unsigned int res_mask,
-       unsigned char owner)
+static void put_resources(struct comedi_device *dev, unsigned int res_mask,
+                         unsigned char owner)
 {
        unsigned int i;
        unsigned int b;
        unsigned long irqflags;
 
-       comedi_spin_lock_irqsave(&devpriv->res_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->res_spinlock, irqflags);
        for (b = 1, i = 0; (i < NUM_RESOURCES)
-               && (res_mask != 0); b <<= 1, i++) {
+            && (res_mask != 0); b <<= 1, i++) {
                if ((res_mask & b) != 0) {
                        res_mask &= ~b;
                        if (devpriv->res_owner[i] == owner) {
@@ -1041,16 +1060,17 @@ static void put_resources(struct comedi_device * dev, unsigned int res_mask,
                        }
                }
        }
-       comedi_spin_unlock_irqrestore(&devpriv->res_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->res_spinlock, irqflags);
 }
 
-static inline void put_one_resource(struct comedi_device * dev, unsigned int resource,
-       unsigned char owner)
+static inline void put_one_resource(struct comedi_device *dev,
+                                   unsigned int resource, unsigned char owner)
 {
        put_resources(dev, (1U << resource), owner);
 }
 
-static inline void put_all_resources(struct comedi_device * dev, unsigned char owner)
+static inline void put_all_resources(struct comedi_device *dev,
+                                    unsigned char owner)
 {
        put_resources(dev, (1U << NUM_RESOURCES) - 1, owner);
 }
@@ -1058,8 +1078,9 @@ static inline void put_all_resources(struct comedi_device * dev, unsigned char o
 /*
  *  COMEDI_SUBD_AI instruction;
  */
-static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data)
+static int pci230_ai_rinsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data)
 {
        unsigned int n, i;
        unsigned int chan, range, aref;
@@ -1110,7 +1131,7 @@ static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
                adccon |= PCI230_ADC_IM_SE;
        }
        devpriv->adcg = (devpriv->adcg & ~(3 << gainshift))
-               | (pci230_ai_gain[range] << gainshift);
+           | (pci230_ai_gain[range] << gainshift);
        if (devpriv->ai_bipolar) {
                adccon |= PCI230_ADC_IR_BIP;
        } else {
@@ -1133,9 +1154,9 @@ static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
                /* Trigger conversion by toggling Z2-CT2 output (finish with
                 * output high). */
                i8254_set_mode(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, 2,
-                       I8254_MODE0);
+                              I8254_MODE0);
                i8254_set_mode(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, 2,
-                       I8254_MODE1);
+                              I8254_MODE1);
 
 #define TIMEOUT 100
                /* wait for conversion to end */
@@ -1143,12 +1164,12 @@ static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
                        status = inw(dev->iobase + PCI230_ADCCON);
                        if (!(status & PCI230_ADC_FIFO_EMPTY))
                                break;
-                       comedi_udelay(1);
+                       udelay(1);
                }
                if (i == TIMEOUT) {
-                       /* rt_printk() should be used instead of printk()
+                       /* printk() should be used instead of printk()
                         * whenever the code can be called from real-time. */
-                       rt_printk("timeout\n");
+                       printk("timeout\n");
                        return -ETIMEDOUT;
                }
 
@@ -1163,8 +1184,9 @@ static int pci230_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
 /*
  *  COMEDI_SUBD_AO instructions;
  */
-static int pci230_ao_winsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data)
+static int pci230_ao_winsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data)
 {
        int i;
        int chan, range;
@@ -1191,8 +1213,9 @@ static int pci230_ao_winsn(struct comedi_device * dev, struct comedi_subdevice *
 
 /* AO subdevices should have a read insn as well as a write insn.
  * Usually this means copying a value stored in devpriv. */
-static int pci230_ao_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data)
+static int pci230_ao_rinsn(struct comedi_device *dev,
+                          struct comedi_subdevice *s, struct comedi_insn *insn,
+                          unsigned int *data)
 {
        int i;
        int chan = CR_CHAN(insn->chanspec);
@@ -1203,8 +1226,8 @@ static int pci230_ao_rinsn(struct comedi_device * dev, struct comedi_subdevice *
        return i;
 }
 
-static int pci230_ao_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_cmd * cmd)
+static int pci230_ao_cmdtest(struct comedi_device *dev,
+                            struct comedi_subdevice *s, struct comedi_cmd *cmd)
 {
        int err = 0;
        unsigned int tmp;
@@ -1315,17 +1338,16 @@ static int pci230_ao_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                /* Trigger number must be 0. */
                if ((cmd->scan_begin_arg & ~CR_FLAGS_MASK) != 0) {
                        cmd->scan_begin_arg = COMBINE(cmd->scan_begin_arg, 0,
-                               ~CR_FLAGS_MASK);
+                                                     ~CR_FLAGS_MASK);
                        err++;
                }
                /* The only flags allowed are CR_EDGE and CR_INVERT.  The
                 * CR_EDGE flag is ignored. */
                if ((cmd->scan_begin_arg
-                               & (CR_FLAGS_MASK & ~(CR_EDGE | CR_INVERT))) !=
-                       0) {
+                    & (CR_FLAGS_MASK & ~(CR_EDGE | CR_INVERT))) != 0) {
                        cmd->scan_begin_arg =
-                               COMBINE(cmd->scan_begin_arg, 0,
-                               CR_FLAGS_MASK & ~(CR_EDGE | CR_INVERT));
+                           COMBINE(cmd->scan_begin_arg, 0,
+                                   CR_FLAGS_MASK & ~(CR_EDGE | CR_INVERT));
                        err++;
                }
                break;
@@ -1359,7 +1381,7 @@ static int pci230_ao_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        if (cmd->scan_begin_src == TRIG_TIMER) {
                tmp = cmd->scan_begin_arg;
                pci230_ns_to_single_timer(&cmd->scan_begin_arg,
-                       cmd->flags & TRIG_ROUND_MASK);
+                                         cmd->flags & TRIG_ROUND_MASK);
                if (tmp != cmd->scan_begin_arg)
                        err++;
        }
@@ -1416,42 +1438,44 @@ static int pci230_ao_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        return 0;
 }
 
-static int pci230_ao_inttrig_scan_begin(struct comedi_device * dev,
-       struct comedi_subdevice * s, unsigned int trig_num)
+static int pci230_ao_inttrig_scan_begin(struct comedi_device *dev,
+                                       struct comedi_subdevice *s,
+                                       unsigned int trig_num)
 {
        unsigned long irqflags;
 
        if (trig_num != 0)
                return -EINVAL;
 
-       comedi_spin_lock_irqsave(&devpriv->ao_stop_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->ao_stop_spinlock, irqflags);
        if (test_bit(AO_CMD_STARTED, &devpriv->state)) {
                /* Perform scan. */
                if (devpriv->hwver < 2) {
                        /* Not using DAC FIFO. */
-                       comedi_spin_unlock_irqrestore(&devpriv->
-                               ao_stop_spinlock, irqflags);
+                       spin_unlock_irqrestore(&devpriv->ao_stop_spinlock,
+                                              irqflags);
                        pci230_handle_ao_nofifo(dev, s);
                        comedi_event(dev, s);
                } else {
                        /* Using DAC FIFO. */
                        /* Read DACSWTRIG register to trigger conversion. */
                        inw(dev->iobase + PCI230P2_DACSWTRIG);
-                       comedi_spin_unlock_irqrestore(&devpriv->
-                               ao_stop_spinlock, irqflags);
+                       spin_unlock_irqrestore(&devpriv->ao_stop_spinlock,
+                                              irqflags);
                }
                /* Delay.  Should driver be responsible for this? */
                /* XXX TODO: See if DAC busy bit can be used. */
-               comedi_udelay(8);
+               udelay(8);
        }
 
        return 1;
 }
 
-static void pci230_ao_start(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_ao_start(struct comedi_device *dev,
+                           struct comedi_subdevice *s)
 {
-       comedi_async *async = s->async;
-       comedi_cmd *cmd = &async->cmd;
+       struct comedi_async *async = s->async;
+       struct comedi_cmd *cmd = &async->cmd;
        unsigned long irqflags;
 
        set_bit(AO_CMD_STARTED, &devpriv->state);
@@ -1497,7 +1521,8 @@ static void pci230_ao_start(struct comedi_device * dev, struct comedi_subdevice
                                break;
                        }
                        devpriv->daccon = (devpriv->daccon
-                               & ~PCI230P2_DAC_TRIG_MASK) | scantrig;
+                                          & ~PCI230P2_DAC_TRIG_MASK) |
+                           scantrig;
                        outw(devpriv->daccon, dev->iobase + PCI230_DACCON);
 
                }
@@ -1506,18 +1531,18 @@ static void pci230_ao_start(struct comedi_device * dev, struct comedi_subdevice
                        if (devpriv->hwver < 2) {
                                /* Not using DAC FIFO. */
                                /* Enable CT1 timer interrupt. */
-                               comedi_spin_lock_irqsave(&devpriv->isr_spinlock,
-                                       irqflags);
+                               spin_lock_irqsave(&devpriv->isr_spinlock,
+                                                 irqflags);
                                devpriv->int_en |= PCI230_INT_ZCLK_CT1;
                                devpriv->ier |= PCI230_INT_ZCLK_CT1;
                                outb(devpriv->ier,
-                                       devpriv->iobase1 + PCI230_INT_SCE);
-                               comedi_spin_unlock_irqrestore(&devpriv->
-                                       isr_spinlock, irqflags);
+                                    devpriv->iobase1 + PCI230_INT_SCE);
+                               spin_unlock_irqrestore(&devpriv->isr_spinlock,
+                                                      irqflags);
                        }
                        /* Set CT1 gate high to start counting. */
                        outb(GAT_CONFIG(1, GAT_VCC),
-                               devpriv->iobase1 + PCI230_ZGAT_SCE);
+                            devpriv->iobase1 + PCI230_ZGAT_SCE);
                        break;
                case TRIG_INT:
                        async->inttrig = pci230_ao_inttrig_scan_begin;
@@ -1525,19 +1550,19 @@ static void pci230_ao_start(struct comedi_device * dev, struct comedi_subdevice
                }
                if (devpriv->hwver >= 2) {
                        /* Using DAC FIFO.  Enable DAC FIFO interrupt. */
-                       comedi_spin_lock_irqsave(&devpriv->isr_spinlock,
-                               irqflags);
+                       spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
                        devpriv->int_en |= PCI230P2_INT_DAC;
                        devpriv->ier |= PCI230P2_INT_DAC;
                        outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
-                       comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock,
-                               irqflags);
+                       spin_unlock_irqrestore(&devpriv->isr_spinlock,
+                                              irqflags);
                }
        }
 }
 
-static int pci230_ao_inttrig_start(struct comedi_device * dev, struct comedi_subdevice * s,
-       unsigned int trig_num)
+static int pci230_ao_inttrig_start(struct comedi_device *dev,
+                                  struct comedi_subdevice *s,
+                                  unsigned int trig_num)
 {
        if (trig_num != 0)
                return -EINVAL;
@@ -1548,13 +1573,13 @@ static int pci230_ao_inttrig_start(struct comedi_device * dev, struct comedi_sub
        return 1;
 }
 
-static int pci230_ao_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
+static int pci230_ao_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 {
        unsigned short daccon;
        unsigned int range;
 
        /* Get the command. */
-       comedi_cmd *cmd = &s->async->cmd;
+       struct comedi_cmd *cmd = &s->async->cmd;
 
        if (cmd->scan_begin_src == TRIG_TIMER) {
                /* Claim Z2-CT1. */
@@ -1598,24 +1623,25 @@ static int pci230_ao_cmd(struct comedi_device * dev, struct comedi_subdevice * s
                 * N.B. DAC FIFO interrupts are currently disabled.
                 */
                daccon |= PCI230P2_DAC_FIFO_EN | PCI230P2_DAC_FIFO_RESET
-                       | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR
-                       | PCI230P2_DAC_TRIG_NONE | PCI230P2_DAC_INT_FIFO_NHALF;
+                   | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR
+                   | PCI230P2_DAC_TRIG_NONE | PCI230P2_DAC_INT_FIFO_NHALF;
        }
 
        /* Set DACCON. */
        outw(daccon, dev->iobase + PCI230_DACCON);
        /* Preserve most of DACCON apart from write-only, transient bits. */
        devpriv->daccon = daccon
-               & ~(PCI230P2_DAC_FIFO_RESET | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR);
+           & ~(PCI230P2_DAC_FIFO_RESET | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR);
 
        if (cmd->scan_begin_src == TRIG_TIMER) {
                /* Set the counter timer 1 to the specified scan frequency. */
                /* cmd->scan_begin_arg is sampling period in ns */
                /* gate it off for now. */
                outb(GAT_CONFIG(1, GAT_GND),
-                       devpriv->iobase1 + PCI230_ZGAT_SCE);
+                    devpriv->iobase1 + PCI230_ZGAT_SCE);
                pci230_ct_setup_ns_mode(dev, 1, I8254_MODE3,
-                       cmd->scan_begin_arg, cmd->flags & TRIG_ROUND_MASK);
+                                       cmd->scan_begin_arg,
+                                       cmd->flags & TRIG_ROUND_MASK);
        }
 
        /* N.B. cmd->start_src == TRIG_INT */
@@ -1624,7 +1650,7 @@ static int pci230_ao_cmd(struct comedi_device * dev, struct comedi_subdevice * s
        return 0;
 }
 
-static int pci230_ai_check_scan_period(comedi_cmd * cmd)
+static int pci230_ai_check_scan_period(struct comedi_cmd *cmd)
 {
        unsigned int min_scan_period, chanlist_len;
        int err = 0;
@@ -1635,7 +1661,7 @@ static int pci230_ai_check_scan_period(comedi_cmd * cmd)
        }
        min_scan_period = chanlist_len * cmd->convert_arg;
        if ((min_scan_period < chanlist_len)
-               || (min_scan_period < cmd->convert_arg)) {
+           || (min_scan_period < cmd->convert_arg)) {
                /* Arithmetic overflow. */
                min_scan_period = UINT_MAX;
                err++;
@@ -1648,8 +1674,8 @@ static int pci230_ai_check_scan_period(comedi_cmd * cmd)
        return !err;
 }
 
-static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_cmd * cmd)
+static int pci230_ai_cmdtest(struct comedi_device *dev,
+                            struct comedi_subdevice *s, struct comedi_cmd *cmd)
 {
        int err = 0;
        unsigned int tmp;
@@ -1677,7 +1703,7 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
         * EXTTRIG/EXTCONVCLK input on pin 17 instead. */
        if ((thisboard->have_dio) || (thisboard->min_hwver > 0)) {
                cmd->scan_begin_src &= TRIG_FOLLOW | TRIG_TIMER | TRIG_INT
-                       | TRIG_EXT;
+                   | TRIG_EXT;
        } else {
                cmd->scan_begin_src &= TRIG_FOLLOW | TRIG_TIMER | TRIG_INT;
        }
@@ -1721,7 +1747,7 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        /* If scan_begin_src is not TRIG_FOLLOW, then a monostable will be
         * set up to generate a fixed number of timed conversion pulses. */
        if ((cmd->scan_begin_src != TRIG_FOLLOW)
-               && (cmd->convert_src != TRIG_TIMER))
+           && (cmd->convert_src != TRIG_TIMER))
                err++;
 
        if (err)
@@ -1786,17 +1812,17 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                        /* Trigger number must be 0. */
                        if ((cmd->convert_arg & ~CR_FLAGS_MASK) != 0) {
                                cmd->convert_arg = COMBINE(cmd->convert_arg, 0,
-                                       ~CR_FLAGS_MASK);
+                                                          ~CR_FLAGS_MASK);
                                err++;
                        }
                        /* The only flags allowed are CR_INVERT and CR_EDGE.
                         * CR_EDGE is required. */
                        if ((cmd->convert_arg & (CR_FLAGS_MASK & ~CR_INVERT))
-                               != CR_EDGE) {
+                           != CR_EDGE) {
                                /* Set CR_EDGE, preserve CR_INVERT. */
                                cmd->convert_arg =
-                                       COMBINE(cmd->start_arg, (CR_EDGE | 0),
-                                       CR_FLAGS_MASK & ~CR_INVERT);
+                                   COMBINE(cmd->start_arg, (CR_EDGE | 0),
+                                           CR_FLAGS_MASK & ~CR_INVERT);
                                err++;
                        }
                } else {
@@ -1834,13 +1860,13 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                 * of CT2 (sample convert trigger is CT2) */
                if ((cmd->scan_begin_arg & ~CR_FLAGS_MASK) != 0) {
                        cmd->scan_begin_arg = COMBINE(cmd->scan_begin_arg, 0,
-                               ~CR_FLAGS_MASK);
+                                                     ~CR_FLAGS_MASK);
                        err++;
                }
                /* The only flag allowed is CR_EDGE, which is ignored. */
                if ((cmd->scan_begin_arg & CR_FLAGS_MASK & ~CR_EDGE) != 0) {
                        cmd->scan_begin_arg = COMBINE(cmd->scan_begin_arg, 0,
-                               CR_FLAGS_MASK & ~CR_EDGE);
+                                                     CR_FLAGS_MASK & ~CR_EDGE);
                        err++;
                }
        } else if (cmd->scan_begin_src == TRIG_TIMER) {
@@ -1865,7 +1891,7 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        if (cmd->convert_src == TRIG_TIMER) {
                tmp = cmd->convert_arg;
                pci230_ns_to_single_timer(&cmd->convert_arg,
-                       cmd->flags & TRIG_ROUND_MASK);
+                                         cmd->flags & TRIG_ROUND_MASK);
                if (tmp != cmd->convert_arg)
                        err++;
        }
@@ -1874,11 +1900,11 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                /* N.B. cmd->convert_arg is also TRIG_TIMER */
                tmp = cmd->scan_begin_arg;
                pci230_ns_to_single_timer(&cmd->scan_begin_arg,
-                       cmd->flags & TRIG_ROUND_MASK);
+                                         cmd->flags & TRIG_ROUND_MASK);
                if (!pci230_ai_check_scan_period(cmd)) {
                        /* Was below minimum required.  Round up. */
                        pci230_ns_to_single_timer(&cmd->scan_begin_arg,
-                               TRIG_ROUND_UP);
+                                                 TRIG_ROUND_UP);
                        pci230_ai_check_scan_period(cmd);
                }
                if (tmp != cmd->scan_begin_arg)
@@ -1919,20 +1945,19 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                         * differential.  (These are remapped in software.  In
                         * hardware, only the even channels are available.) */
                        if ((aref == AREF_DIFF)
-                               && (chan >= (s->n_chan / 2))) {
+                           && (chan >= (s->n_chan / 2))) {
                                errors |= diffchan_err;
                        }
                        if (n > 0) {
                                /* Channel numbers must strictly increase or
                                 * subsequence must repeat exactly. */
                                if ((chan <= prev_chan)
-                                       && (subseq_len == 0)) {
+                                   && (subseq_len == 0)) {
                                        subseq_len = n;
                                }
                                if ((subseq_len > 0)
-                                       && (cmd->chanlist[n] !=
-                                               cmd->chanlist[n %
-                                                       subseq_len])) {
+                                   && (cmd->chanlist[n] !=
+                                       cmd->chanlist[n % subseq_len])) {
                                        errors |= seq_err;
                                }
                                /* Channels must have same AREF. */
@@ -1946,8 +1971,8 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                                /* Single-ended channel pairs must have same
                                 * range.  */
                                if ((aref != AREF_DIFF)
-                                       && (((chan ^ prev_chan) & ~1) == 0)
-                                       && (range != prev_range)) {
+                                   && (((chan ^ prev_chan) & ~1) == 0)
+                                   && (range != prev_range)) {
                                        errors |= rangepair_err;
                                }
                        }
@@ -1981,7 +2006,7 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                         * does, and we can't tell them apart!
                         */
                        if ((subseq_len > 1)
-                               && (CR_CHAN(cmd->chanlist[0]) != 0)) {
+                           && (CR_CHAN(cmd->chanlist[0]) != 0)) {
                                errors |= buggy_chan0_err;
                        }
                }
@@ -2019,11 +2044,11 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
                        if ((errors & buggy_chan0_err) != 0) {
                                /* Use printk instead of DPRINTK here. */
                                printk("comedi: comedi%d: amplc_pci230: "
-                                       "ai_cmdtest: Buggy PCI230+/260+ "
-                                       "h/w version %u requires first channel "
-                                       "of multi-channel sequence to be 0 "
-                                       "(corrected in h/w version 4)\n",
-                                       dev->minor, devpriv->hwver);
+                                      "ai_cmdtest: Buggy PCI230+/260+ "
+                                      "h/w version %u requires first channel "
+                                      "of multi-channel sequence to be 0 "
+                                      "(corrected in h/w version 4)\n",
+                                      dev->minor, devpriv->hwver);
                        }
                }
        }
@@ -2034,10 +2059,10 @@ static int pci230_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        return 0;
 }
 
-static void pci230_ai_update_fifo_trigger_level(struct comedi_device * dev,
-       struct comedi_subdevice * s)
+static void pci230_ai_update_fifo_trigger_level(struct comedi_device *dev,
+                                               struct comedi_subdevice *s)
 {
-       comedi_cmd *cmd = &s->async->cmd;
+       struct comedi_cmd *cmd = &s->async->cmd;
        unsigned int scanlen = cmd->scan_end_arg;
        unsigned int wake;
        unsigned short triglev;
@@ -2048,13 +2073,12 @@ static void pci230_ai_update_fifo_trigger_level(struct comedi_device * dev,
                wake = scanlen - devpriv->ai_scan_pos;
        } else {
                if (devpriv->ai_continuous
-                       || (devpriv->ai_scan_count
-                               >= PCI230_ADC_FIFOLEVEL_HALFFULL)
-                       || (scanlen >= PCI230_ADC_FIFOLEVEL_HALFFULL)) {
+                   || (devpriv->ai_scan_count >= PCI230_ADC_FIFOLEVEL_HALFFULL)
+                   || (scanlen >= PCI230_ADC_FIFOLEVEL_HALFFULL)) {
                        wake = PCI230_ADC_FIFOLEVEL_HALFFULL;
                } else {
                        wake = (devpriv->ai_scan_count * scanlen)
-                               - devpriv->ai_scan_pos;
+                           - devpriv->ai_scan_pos;
                }
        }
        if (wake >= PCI230_ADC_FIFOLEVEL_HALFFULL) {
@@ -2078,51 +2102,51 @@ static void pci230_ai_update_fifo_trigger_level(struct comedi_device * dev,
        }
 }
 
-static int pci230_ai_inttrig_convert(struct comedi_device * dev, struct comedi_subdevice * s,
-       unsigned int trig_num)
+static int pci230_ai_inttrig_convert(struct comedi_device *dev,
+                                    struct comedi_subdevice *s,
+                                    unsigned int trig_num)
 {
        unsigned long irqflags;
 
        if (trig_num != 0)
                return -EINVAL;
 
-       comedi_spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
        if (test_bit(AI_CMD_STARTED, &devpriv->state)) {
                unsigned int delayus;
 
                /* Trigger conversion by toggling Z2-CT2 output.  Finish
                 * with output high. */
                i8254_set_mode(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, 2,
-                       I8254_MODE0);
+                              I8254_MODE0);
                i8254_set_mode(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, 2,
-                       I8254_MODE1);
+                              I8254_MODE1);
                /* Delay.  Should driver be responsible for this?  An
                 * alternative would be to wait until conversion is complete,
                 * but we can't tell when it's complete because the ADC busy
                 * bit has a different meaning when FIFO enabled (and when
                 * FIFO not enabled, it only works for software triggers). */
                if (((devpriv->adccon & PCI230_ADC_IM_MASK)
-                               == PCI230_ADC_IM_DIF)
-                       && (devpriv->hwver == 0)) {
+                    == PCI230_ADC_IM_DIF)
+                   && (devpriv->hwver == 0)) {
                        /* PCI230/260 in differential mode */
                        delayus = 8;
                } else {
                        /* single-ended or PCI230+/260+ */
                        delayus = 4;
                }
-               comedi_spin_unlock_irqrestore(&devpriv->ai_stop_spinlock,
-                       irqflags);
-               comedi_udelay(delayus);
+               spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
+               udelay(delayus);
        } else {
-               comedi_spin_unlock_irqrestore(&devpriv->ai_stop_spinlock,
-                       irqflags);
+               spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
        }
 
        return 1;
 }
 
-static int pci230_ai_inttrig_scan_begin(struct comedi_device * dev,
-       struct comedi_subdevice * s, unsigned int trig_num)
+static int pci230_ai_inttrig_scan_begin(struct comedi_device *dev,
+                                       struct comedi_subdevice *s,
+                                       unsigned int trig_num)
 {
        unsigned long irqflags;
        unsigned char zgat;
@@ -2130,7 +2154,7 @@ static int pci230_ai_inttrig_scan_begin(struct comedi_device * dev,
        if (trig_num != 0)
                return -EINVAL;
 
-       comedi_spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
        if (test_bit(AI_CMD_STARTED, &devpriv->state)) {
                /* Trigger scan by waggling CT0 gate source. */
                zgat = GAT_CONFIG(0, GAT_GND);
@@ -2138,17 +2162,18 @@ static int pci230_ai_inttrig_scan_begin(struct comedi_device * dev,
                zgat = GAT_CONFIG(0, GAT_VCC);
                outb(zgat, devpriv->iobase1 + PCI230_ZGAT_SCE);
        }
-       comedi_spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
 
        return 1;
 }
 
-static void pci230_ai_start(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_ai_start(struct comedi_device *dev,
+                           struct comedi_subdevice *s)
 {
        unsigned long irqflags;
        unsigned short conv;
-       comedi_async *async = s->async;
-       comedi_cmd *cmd = &async->cmd;
+       struct comedi_async *async = s->async;
+       struct comedi_cmd *cmd = &async->cmd;
 
        set_bit(AI_CMD_STARTED, &devpriv->state);
        if (!devpriv->ai_continuous && (devpriv->ai_scan_count == 0)) {
@@ -2158,11 +2183,11 @@ static void pci230_ai_start(struct comedi_device * dev, struct comedi_subdevice
                comedi_event(dev, s);
        } else {
                /* Enable ADC FIFO trigger level interrupt. */
-               comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+               spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
                devpriv->int_en |= PCI230_INT_ADC;
                devpriv->ier |= PCI230_INT_ADC;
                outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
-               comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+               spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
 
                /* Update conversion trigger source which is currently set
                 * to CT2 output, which is currently stuck high. */
@@ -2201,7 +2226,7 @@ static void pci230_ai_start(struct comedi_device * dev, struct comedi_subdevice
                        break;
                }
                devpriv->adccon = (devpriv->adccon & ~PCI230_ADC_TRIG_MASK)
-                       | conv;
+                   | conv;
                outw(devpriv->adccon, dev->iobase + PCI230_ADCCON);
                if (cmd->convert_src == TRIG_INT) {
                        async->inttrig = pci230_ai_inttrig_convert;
@@ -2265,11 +2290,11 @@ static void pci230_ai_start(struct comedi_device * dev, struct comedi_subdevice
                                         * gated on to start counting. */
                                        zgat = GAT_CONFIG(1, GAT_VCC);
                                        outb(zgat, devpriv->iobase1
-                                               + PCI230_ZGAT_SCE);
+                                            + PCI230_ZGAT_SCE);
                                        break;
                                case TRIG_INT:
                                        async->inttrig =
-                                               pci230_ai_inttrig_scan_begin;
+                                           pci230_ai_inttrig_scan_begin;
                                        break;
                                }
                        }
@@ -2280,8 +2305,9 @@ static void pci230_ai_start(struct comedi_device * dev, struct comedi_subdevice
        }
 }
 
-static int pci230_ai_inttrig_start(struct comedi_device * dev, struct comedi_subdevice * s,
-       unsigned int trig_num)
+static int pci230_ai_inttrig_start(struct comedi_device *dev,
+                                  struct comedi_subdevice *s,
+                                  unsigned int trig_num)
 {
        if (trig_num != 0)
                return -EINVAL;
@@ -2292,7 +2318,7 @@ static int pci230_ai_inttrig_start(struct comedi_device * dev, struct comedi_sub
        return 1;
 }
 
-static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
+static int pci230_ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 {
        unsigned int i, chan, range, diff;
        unsigned int res_mask;
@@ -2300,8 +2326,8 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
        unsigned char zgat;
 
        /* Get the command. */
-       comedi_async *async = s->async;
-       comedi_cmd *cmd = &async->cmd;
+       struct comedi_async *async = s->async;
+       struct comedi_cmd *cmd = &async->cmd;
 
        /*
         * Determine which shared resources are needed.
@@ -2392,7 +2418,7 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
                        adcen |= 1 << chan;
                }
                devpriv->adcg = (devpriv->adcg & ~(3 << gainshift))
-                       | (pci230_ai_gain[range] << gainshift);
+                   | (pci230_ai_gain[range] << gainshift);
        }
 
        /* Set channel scan list. */
@@ -2424,7 +2450,7 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
         * FIFO reset (above) and the second FIFO reset (below). Setting the
         * channel gains and scan list _before_ the first FIFO reset also
         * helps, though only slightly. */
-       comedi_udelay(25);
+       udelay(25);
 
        /* Reset FIFO again. */
        outw(adccon | PCI230_ADC_FIFO_RESET, dev->iobase + PCI230_ADCCON);
@@ -2437,7 +2463,7 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
                outb(zgat, devpriv->iobase1 + PCI230_ZGAT_SCE);
                /* Set counter/timer 2 to the specified conversion period. */
                pci230_ct_setup_ns_mode(dev, 2, I8254_MODE3, cmd->convert_arg,
-                       cmd->flags & TRIG_ROUND_MASK);
+                                       cmd->flags & TRIG_ROUND_MASK);
                if (cmd->scan_begin_src != TRIG_FOLLOW) {
                        /*
                         * Set up monostable on CT0 output for scan timing.  A
@@ -2454,8 +2480,9 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
                        zgat = GAT_CONFIG(0, GAT_VCC);
                        outb(zgat, devpriv->iobase1 + PCI230_ZGAT_SCE);
                        pci230_ct_setup_ns_mode(dev, 0, I8254_MODE1,
-                               ((uint64_t) cmd->convert_arg
-                                       * cmd->scan_end_arg), TRIG_ROUND_UP);
+                                               ((uint64_t) cmd->convert_arg
+                                                * cmd->scan_end_arg),
+                                               TRIG_ROUND_UP);
                        if (cmd->scan_begin_src == TRIG_TIMER) {
                                /*
                                 * Monostable on CT0 will be triggered by
@@ -2466,8 +2493,10 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
                                zgat = GAT_CONFIG(1, GAT_GND);
                                outb(zgat, devpriv->iobase1 + PCI230_ZGAT_SCE);
                                pci230_ct_setup_ns_mode(dev, 1, I8254_MODE3,
-                                       cmd->scan_begin_arg,
-                                       cmd->flags & TRIG_ROUND_MASK);
+                                                       cmd->scan_begin_arg,
+                                                       cmd->
+                                                       flags &
+                                                       TRIG_ROUND_MASK);
                        }
                }
        }
@@ -2483,7 +2512,7 @@ static int pci230_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s
 }
 
 static unsigned int divide_ns(uint64_t ns, unsigned int timebase,
-       unsigned int round_mode)
+                             unsigned int round_mode)
 {
        uint64_t div;
        unsigned int rem;
@@ -2508,7 +2537,7 @@ static unsigned int divide_ns(uint64_t ns, unsigned int timebase,
 /* Given desired period in ns, returns the required internal clock source
  * and gets the initial count. */
 static unsigned int pci230_choose_clk_count(uint64_t ns, unsigned int *count,
-       unsigned int round_mode)
+                                           unsigned int round_mode)
 {
        unsigned int clk_src, cnt;
 
@@ -2532,8 +2561,9 @@ static void pci230_ns_to_single_timer(unsigned int *ns, unsigned int round)
        return;
 }
 
-static void pci230_ct_setup_ns_mode(struct comedi_device * dev, unsigned int ct,
-       unsigned int mode, uint64_t ns, unsigned int round)
+static void pci230_ct_setup_ns_mode(struct comedi_device *dev, unsigned int ct,
+                                   unsigned int mode, uint64_t ns,
+                                   unsigned int round)
 {
        unsigned int clk_src;
        unsigned int count;
@@ -2551,18 +2581,18 @@ static void pci230_ct_setup_ns_mode(struct comedi_device * dev, unsigned int ct,
        i8254_write(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, ct, count);
 }
 
-static void pci230_cancel_ct(struct comedi_device * dev, unsigned int ct)
+static void pci230_cancel_ct(struct comedi_device *dev, unsigned int ct)
 {
        i8254_set_mode(devpriv->iobase1 + PCI230_Z2_CT_BASE, 0, ct,
-               I8254_MODE1);
+                      I8254_MODE1);
        /* Counter ct, 8254 mode 1, initial count not written. */
 }
 
 /* Interrupt handler */
-static irqreturn_t pci230_interrupt(int irq, void *d PT_REGS_ARG)
+static irqreturn_t pci230_interrupt(int irq, void *d)
 {
        unsigned char status_int, valid_status_int;
-       struct comedi_device *dev = (struct comedi_device *) d;
+       struct comedi_device *dev = (struct comedi_device *)d;
        struct comedi_subdevice *s;
        unsigned long irqflags;
 
@@ -2573,7 +2603,7 @@ static irqreturn_t pci230_interrupt(int irq, void *d PT_REGS_ARG)
                return IRQ_NONE;
        }
 
-       comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        valid_status_int = devpriv->int_en & status_int;
        /* Disable triggered interrupts.
         * (Only those interrupts that need re-enabling, are, later in the
@@ -2582,7 +2612,7 @@ static irqreturn_t pci230_interrupt(int irq, void *d PT_REGS_ARG)
        outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
        devpriv->intr_running = 1;
        devpriv->intr_cpuid = THISCPU;
-       comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
 
        /*
         * Check the source of interrupt and handle it.
@@ -2611,23 +2641,24 @@ static irqreturn_t pci230_interrupt(int irq, void *d PT_REGS_ARG)
        }
 
        /* Reenable interrupts. */
-       comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        if (devpriv->ier != devpriv->int_en) {
                devpriv->ier = devpriv->int_en;
                outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
        }
        devpriv->intr_running = 0;
-       comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
 
        return IRQ_HANDLED;
 }
 
-static void pci230_handle_ao_nofifo(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_handle_ao_nofifo(struct comedi_device *dev,
+                                   struct comedi_subdevice *s)
 {
        short data;
        int i, ret;
-       comedi_async *async = s->async;
-       comedi_cmd *cmd = &async->cmd;
+       struct comedi_async *async = s->async;
+       struct comedi_cmd *cmd = &async->cmd;
 
        if (!devpriv->ao_continuous && (devpriv->ao_scan_count == 0)) {
                return;
@@ -2659,10 +2690,11 @@ static void pci230_handle_ao_nofifo(struct comedi_device * dev, struct comedi_su
 
 /* Loads DAC FIFO (if using it) from buffer. */
 /* Returns 0 if AO finished due to completion or error, 1 if still going. */
-static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subdevice * s)
+static int pci230_handle_ao_fifo(struct comedi_device *dev,
+                                struct comedi_subdevice *s)
 {
-       comedi_async *async = s->async;
-       comedi_cmd *cmd = &async->cmd;
+       struct comedi_async *async = s->async;
+       struct comedi_cmd *cmd = &async->cmd;
        unsigned int num_scans;
        unsigned int room;
        unsigned short dacstat;
@@ -2697,7 +2729,7 @@ static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subde
                 * (otherwise there will be loads of "DAC FIFO not half full"
                 * interrupts). */
                if ((num_scans == 0)
-                       && ((dacstat & PCI230P2_DAC_FIFO_HALF) == 0)) {
+                   && ((dacstat & PCI230P2_DAC_FIFO_HALF) == 0)) {
                        comedi_error(dev, "AO buffer underrun");
                        events |= COMEDI_CB_OVERFLOW | COMEDI_CB_ERROR;
                }
@@ -2726,7 +2758,7 @@ static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subde
 
                                comedi_buf_get(async, &datum);
                                pci230_ao_write_fifo(dev, datum,
-                                       CR_CHAN(cmd->chanlist[i]));
+                                                    CR_CHAN(cmd->chanlist[i]));
                        }
                }
                events |= COMEDI_CB_EOS | COMEDI_CB_BLOCK;
@@ -2737,10 +2769,11 @@ static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subde
                                 * to FIFO.  Set FIFO interrupt trigger level
                                 * to 'empty'. */
                                devpriv->daccon = (devpriv->daccon
-                                       & ~PCI230P2_DAC_INT_FIFO_MASK)
-                                       | PCI230P2_DAC_INT_FIFO_EMPTY;
+                                                  &
+                                                  ~PCI230P2_DAC_INT_FIFO_MASK)
+                                   | PCI230P2_DAC_INT_FIFO_EMPTY;
                                outw(devpriv->daccon,
-                                       dev->iobase + PCI230_DACCON);
+                                    dev->iobase + PCI230_DACCON);
                        }
                }
                /* Check if FIFO underrun occurred while writing to FIFO. */
@@ -2751,7 +2784,7 @@ static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subde
                }
        }
        if ((events & (COMEDI_CB_EOA | COMEDI_CB_ERROR | COMEDI_CB_OVERFLOW))
-               != 0) {
+           != 0) {
                /* Stopping AO due to completion or error. */
                pci230_ao_stop(dev, s);
                running = 0;
@@ -2762,14 +2795,15 @@ static int pci230_handle_ao_fifo(struct comedi_device * dev, struct comedi_subde
        return running;
 }
 
-static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_handle_ai(struct comedi_device *dev,
+                            struct comedi_subdevice *s)
 {
        unsigned int events = 0;
        unsigned int status_fifo;
        unsigned int i;
        unsigned int todo;
        unsigned int fifoamount;
-       comedi_async *async = s->async;
+       struct comedi_async *async = s->async;
        unsigned int scanlen = async->cmd.scan_end_arg;
 
        /* Determine number of samples to read. */
@@ -2778,11 +2812,11 @@ static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice
        } else if (devpriv->ai_scan_count == 0) {
                todo = 0;
        } else if ((devpriv->ai_scan_count > PCI230_ADC_FIFOLEVEL_HALFFULL)
-               || (scanlen > PCI230_ADC_FIFOLEVEL_HALFFULL)) {
+                  || (scanlen > PCI230_ADC_FIFOLEVEL_HALFFULL)) {
                todo = PCI230_ADC_FIFOLEVEL_HALFFULL;
        } else {
                todo = (devpriv->ai_scan_count * scanlen)
-                       - devpriv->ai_scan_pos;
+                   - devpriv->ai_scan_pos;
                if (todo > PCI230_ADC_FIFOLEVEL_HALFFULL) {
                        todo = PCI230_ADC_FIFOLEVEL_HALFFULL;
                }
@@ -2815,7 +2849,7 @@ static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice
                                if (devpriv->hwver > 0) {
                                        /* Read PCI230+/260+ ADC FIFO level. */
                                        fifoamount = inw(dev->iobase
-                                               + PCI230P_ADCFFLEV);
+                                                        + PCI230P_ADCFFLEV);
                                        if (fifoamount == 0) {
                                                /* Shouldn't happen. */
                                                break;
@@ -2852,7 +2886,7 @@ static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice
        async->events |= events;
 
        if ((async->events & (COMEDI_CB_EOA | COMEDI_CB_ERROR |
-                               COMEDI_CB_OVERFLOW)) != 0) {
+                             COMEDI_CB_OVERFLOW)) != 0) {
                /* disable hardware conversions */
                pci230_ai_stop(dev, s);
        } else {
@@ -2861,16 +2895,17 @@ static void pci230_handle_ai(struct comedi_device * dev, struct comedi_subdevice
        }
 }
 
-static void pci230_ao_stop(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_ao_stop(struct comedi_device *dev,
+                          struct comedi_subdevice *s)
 {
        unsigned long irqflags;
        unsigned char intsrc;
        int started;
-       comedi_cmd *cmd;
+       struct comedi_cmd *cmd;
 
-       comedi_spin_lock_irqsave(&devpriv->ao_stop_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->ao_stop_spinlock, irqflags);
        started = test_and_clear_bit(AO_CMD_STARTED, &devpriv->state);
-       comedi_spin_unlock_irqrestore(&devpriv->ao_stop_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->ao_stop_spinlock, irqflags);
        if (!started) {
                return;
        }
@@ -2891,46 +2926,48 @@ static void pci230_ao_stop(struct comedi_device * dev, struct comedi_subdevice *
        }
        /* Disable interrupt and wait for interrupt routine to finish running
         * unless we are called from the interrupt routine. */
-       comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        devpriv->int_en &= ~intsrc;
        while (devpriv->intr_running && devpriv->intr_cpuid != THISCPU) {
-               comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
-               comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+               spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+               spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        }
        if (devpriv->ier != devpriv->int_en) {
                devpriv->ier = devpriv->int_en;
                outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
        }
-       comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
 
        if (devpriv->hwver >= 2) {
                /* Using DAC FIFO.  Reset FIFO, clear underrun error,
                 * disable FIFO. */
                devpriv->daccon &= PCI230_DAC_OR_MASK;
                outw(devpriv->daccon | PCI230P2_DAC_FIFO_RESET
-                       | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR,
-                       dev->iobase + PCI230_DACCON);
+                    | PCI230P2_DAC_FIFO_UNDERRUN_CLEAR,
+                    dev->iobase + PCI230_DACCON);
        }
 
        /* Release resources. */
        put_all_resources(dev, OWNER_AOCMD);
 }
 
-static int pci230_ao_cancel(struct comedi_device * dev, struct comedi_subdevice * s)
+static int pci230_ao_cancel(struct comedi_device *dev,
+                           struct comedi_subdevice *s)
 {
        pci230_ao_stop(dev, s);
        return 0;
 }
 
-static void pci230_ai_stop(struct comedi_device * dev, struct comedi_subdevice * s)
+static void pci230_ai_stop(struct comedi_device *dev,
+                          struct comedi_subdevice *s)
 {
        unsigned long irqflags;
-       comedi_cmd *cmd;
+       struct comedi_cmd *cmd;
        int started;
 
-       comedi_spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->ai_stop_spinlock, irqflags);
        started = test_and_clear_bit(AI_CMD_STARTED, &devpriv->state);
-       comedi_spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->ai_stop_spinlock, irqflags);
        if (!started) {
                return;
        }
@@ -2945,32 +2982,34 @@ static void pci230_ai_stop(struct comedi_device * dev, struct comedi_subdevice *
                pci230_cancel_ct(dev, 0);
        }
 
-       comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+       spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        /* Disable ADC interrupt and wait for interrupt routine to finish
         * running unless we are called from the interrupt routine. */
        devpriv->int_en &= ~PCI230_INT_ADC;
        while (devpriv->intr_running && devpriv->intr_cpuid != THISCPU) {
-               comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
-               comedi_spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
+               spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+               spin_lock_irqsave(&devpriv->isr_spinlock, irqflags);
        }
        if (devpriv->ier != devpriv->int_en) {
                devpriv->ier = devpriv->int_en;
                outb(devpriv->ier, devpriv->iobase1 + PCI230_INT_SCE);
        }
-       comedi_spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
+       spin_unlock_irqrestore(&devpriv->isr_spinlock, irqflags);
 
        /* Reset FIFO, disable FIFO and set start conversion source to none.
         * Keep se/diff and bip/uni settings */
        devpriv->adccon = (devpriv->adccon & (PCI230_ADC_IR_MASK
-                       | PCI230_ADC_IM_MASK)) | PCI230_ADC_TRIG_NONE;
+                                             | PCI230_ADC_IM_MASK)) |
+           PCI230_ADC_TRIG_NONE;
        outw(devpriv->adccon | PCI230_ADC_FIFO_RESET,
-               dev->iobase + PCI230_ADCCON);
+            dev->iobase + PCI230_ADCCON);
 
        /* Release resources. */
        put_all_resources(dev, OWNER_AICMD);
 }
 
-static int pci230_ai_cancel(struct comedi_device * dev, struct comedi_subdevice * s)
+static int pci230_ai_cancel(struct comedi_device *dev,
+                           struct comedi_subdevice *s)
 {
        pci230_ai_stop(dev, s);
        return 0;