Staging: comedi: pcmcia irq fixes
[safe/jmp/linux-2.6] / drivers / staging / comedi / drivers / ni_at_a2150.c
index 8a13e86..dd75dfb 100644 (file)
@@ -64,6 +64,7 @@ TRIG_WAKE_EOS
 
 */
 
+#include <linux/interrupt.h>
 #include "../comedidev.h"
 
 #include <linux/ioport.h>
@@ -73,10 +74,10 @@ TRIG_WAKE_EOS
 #include "comedi_fc.h"
 
 #define A2150_SIZE           28
-#define A2150_DMA_BUFFER_SIZE  0xff00  // size in bytes of dma buffer
+#define A2150_DMA_BUFFER_SIZE  0xff00  /*  size in bytes of dma buffer */
 
-//#define A2150_DEBUG   // enable debugging code
-#undef A2150_DEBUG             // disable debugging code
+/* #define A2150_DEBUG     enable debugging code */
+#undef A2150_DEBUG             /*  disable debugging code */
 
 /* Registers and bits */
 #define CONFIG_REG             0x0
@@ -85,110 +86,112 @@ TRIG_WAKE_EOS
 #define   CLOCK_SELECT_BITS(x)         (((x) & 0x3) << 3)
 #define   CLOCK_DIVISOR_BITS(x)                (((x) & 0x3) << 5)
 #define   CLOCK_MASK           (0xf << 3)
-#define   ENABLE0_BIT          0x80    // enable (don't internally ground) channels 0 and 1
-#define   ENABLE1_BIT          0x100   // enable (don't internally ground) channels 2 and 3
-#define   AC0_BIT              0x200   // ac couple channels 0,1
-#define   AC1_BIT              0x400   // ac couple channels 2,3
-#define   APD_BIT              0x800   // analog power down
-#define   DPD_BIT              0x1000  // digital power down
-#define TRIGGER_REG            0x2     // trigger config register
+#define   ENABLE0_BIT          0x80    /*  enable (don't internally ground) channels 0 and 1 */
+#define   ENABLE1_BIT          0x100   /*  enable (don't internally ground) channels 2 and 3 */
+#define   AC0_BIT              0x200   /*  ac couple channels 0,1 */
+#define   AC1_BIT              0x400   /*  ac couple channels 2,3 */
+#define   APD_BIT              0x800   /*  analog power down */
+#define   DPD_BIT              0x1000  /*  digital power down */
+#define TRIGGER_REG            0x2     /*  trigger config register */
 #define   POST_TRIGGER_BITS            0x2
 #define   DELAY_TRIGGER_BITS           0x3
-#define   HW_TRIG_EN           0x10    // enable hardware trigger
-#define FIFO_START_REG         0x6     // software start aquistion trigger
-#define FIFO_RESET_REG         0x8     // clears fifo + fifo flags
-#define FIFO_DATA_REG          0xa     // read data
-#define DMA_TC_CLEAR_REG               0xe     // clear dma terminal count interrupt
-#define STATUS_REG             0x12    // read only
-#define   FNE_BIT              0x1     // fifo not empty
-#define   OVFL_BIT             0x8     // fifo overflow
-#define   EDAQ_BIT             0x10    // end of aquisition interrupt
-#define   DCAL_BIT             0x20    // offset calibration in progress
-#define   INTR_BIT             0x40    // interrupt has occured
-#define   DMA_TC_BIT           0x80    // dma terminal count interrupt has occured
+#define   HW_TRIG_EN           0x10    /*  enable hardware trigger */
+#define FIFO_START_REG         0x6     /*  software start aquistion trigger */
+#define FIFO_RESET_REG         0x8     /*  clears fifo + fifo flags */
+#define FIFO_DATA_REG          0xa     /*  read data */
+#define DMA_TC_CLEAR_REG               0xe     /*  clear dma terminal count interrupt */
+#define STATUS_REG             0x12    /*  read only */
+#define   FNE_BIT              0x1     /*  fifo not empty */
+#define   OVFL_BIT             0x8     /*  fifo overflow */
+#define   EDAQ_BIT             0x10    /*  end of aquisition interrupt */
+#define   DCAL_BIT             0x20    /*  offset calibration in progress */
+#define   INTR_BIT             0x40    /*  interrupt has occured */
+#define   DMA_TC_BIT           0x80    /*  dma terminal count interrupt has occured */
 #define   ID_BITS(x)   (((x) >> 8) & 0x3)
-#define IRQ_DMA_CNTRL_REG              0x12    // write only
-#define   DMA_CHAN_BITS(x)             ((x) & 0x7)     // sets dma channel
-#define   DMA_EN_BIT           0x8     // enables dma
-#define   IRQ_LVL_BITS(x)              (((x) & 0xf) << 4)      // sets irq level
-#define   FIFO_INTR_EN_BIT             0x100   // enable fifo interrupts
-#define   FIFO_INTR_FHF_BIT            0x200   // interrupt fifo half full
-#define   DMA_INTR_EN_BIT              0x800   // enable interrupt on dma terminal count
-#define   DMA_DEM_EN_BIT       0x1000  // enables demand mode dma
+#define IRQ_DMA_CNTRL_REG              0x12    /*  write only */
+#define   DMA_CHAN_BITS(x)             ((x) & 0x7)     /*  sets dma channel */
+#define   DMA_EN_BIT           0x8     /*  enables dma */
+#define   IRQ_LVL_BITS(x)              (((x) & 0xf) << 4)      /*  sets irq level */
+#define   FIFO_INTR_EN_BIT             0x100   /*  enable fifo interrupts */
+#define   FIFO_INTR_FHF_BIT            0x200   /*  interrupt fifo half full */
+#define   DMA_INTR_EN_BIT              0x800   /*  enable interrupt on dma terminal count */
+#define   DMA_DEM_EN_BIT       0x1000  /*  enables demand mode dma */
 #define I8253_BASE_REG         0x14
 #define I8253_MODE_REG         0x17
-#define   HW_COUNT_DISABLE             0x30    // disable hardware counting of conversions
+#define   HW_COUNT_DISABLE             0x30    /*  disable hardware counting of conversions */
 
-typedef struct a2150_board_struct {
+struct a2150_board {
        const char *name;
-       int clock[4];           // master clock periods, in nanoseconds
-       int num_clocks;         // number of available master clock speeds
-       int ai_speed;           // maximum conversion rate in nanoseconds
-} a2150_board;
+       int clock[4];           /*  master clock periods, in nanoseconds */
+       int num_clocks;         /*  number of available master clock speeds */
+       int ai_speed;           /*  maximum conversion rate in nanoseconds */
+};
 
-//analog input range
+/* analog input range */
 static const struct comedi_lrange range_a2150 = {
        1,
        {
-                       RANGE(-2.828, 2.828),
-               }
+        RANGE(-2.828, 2.828),
+        }
 };
 
-// enum must match board indices
+/* enum must match board indices */
 enum { a2150_c, a2150_s };
-static const a2150_board a2150_boards[] = {
+static const struct a2150_board a2150_boards[] = {
        {
-             name:     "at-a2150c",
-             clock:    {31250, 22676, 20833, 19531},
-             num_clocks:4,
-             ai_speed:19531,
-               },
+        .name = "at-a2150c",
+        .clock = {31250, 22676, 20833, 19531},
+        .num_clocks = 4,
+        .ai_speed = 19531,
+        },
        {
-             name:     "at-a2150s",
-             clock:    {62500, 50000, 41667, 0},
-             num_clocks:3,
-             ai_speed:41667,
-               },
+        .name = "at-a2150s",
+        .clock = {62500, 50000, 41667, 0},
+        .num_clocks = 3,
+        .ai_speed = 41667,
+        },
 };
 
 /*
  * Useful for shorthand access to the particular board structure
  */
-#define thisboard ((const a2150_board *)dev->board_ptr)
+#define thisboard ((const struct a2150_board *)dev->board_ptr)
+
+struct a2150_private {
 
-typedef struct {
        volatile unsigned int count;    /* number of data points left to be taken */
-       unsigned int dma;       // dma channel
-       s16 *dma_buffer;        // dma buffer
-       unsigned int dma_transfer_size; // size in bytes of dma transfers
-       int irq_dma_bits;       // irq/dma register bits
-       int config_bits;        // config register bits
-} a2150_private;
+       unsigned int dma;       /*  dma channel */
+       s16 *dma_buffer;        /*  dma buffer */
+       unsigned int dma_transfer_size; /*  size in bytes of dma transfers */
+       int irq_dma_bits;       /*  irq/dma register bits */
+       int config_bits;        /*  config register bits */
+};
 
-#define devpriv ((a2150_private *)dev->private)
+#define devpriv ((struct a2150_private *)dev->private)
 
-static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it);
-static int a2150_detach(struct comedi_device * dev);
-static int a2150_cancel(struct comedi_device * dev, struct comedi_subdevice * s);
+static int a2150_attach(struct comedi_device *dev, struct comedi_devconfig *it);
+static int a2150_detach(struct comedi_device *dev);
+static int a2150_cancel(struct comedi_device *dev, struct comedi_subdevice *s);
 
 static struct comedi_driver driver_a2150 = {
-      driver_name:"ni_at_a2150",
-      module:THIS_MODULE,
-      attach:a2150_attach,
-      detach:a2150_detach,
+       .driver_name = "ni_at_a2150",
+       .module = THIS_MODULE,
+       .attach = a2150_attach,
+       .detach = a2150_detach,
 };
 
-static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG);
-static int a2150_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       struct comedi_cmd * cmd);
-static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s);
-static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data);
-static int a2150_get_timing(struct comedi_device * dev, unsigned int *period,
-       int flags);
-static int a2150_probe(struct comedi_device * dev);
-static int a2150_set_chanlist(struct comedi_device * dev, unsigned int start_channel,
-       unsigned int num_channels);
+static irqreturn_t a2150_interrupt(int irq, void *d);
+static int a2150_ai_cmdtest(struct comedi_device *dev,
+                           struct comedi_subdevice *s, struct comedi_cmd *cmd);
+static int a2150_ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s);
+static int a2150_ai_rinsn(struct comedi_device *dev, struct comedi_subdevice *s,
+                         struct comedi_insn *insn, unsigned int *data);
+static int a2150_get_timing(struct comedi_device *dev, unsigned int *period,
+                           int flags);
+static int a2150_probe(struct comedi_device *dev);
+static int a2150_set_chanlist(struct comedi_device *dev,
+                             unsigned int start_channel,
+                             unsigned int num_channels);
 /*
  * A convenient macro that defines init_module() and cleanup_module(),
  * as necessary.
@@ -197,17 +200,17 @@ COMEDI_INITCLEANUP(driver_a2150);
 
 #ifdef A2150_DEBUG
 
-static void ni_dump_regs(struct comedi_device * dev)
+static void ni_dump_regs(struct comedi_device *dev)
 {
-       rt_printk("config bits 0x%x\n", devpriv->config_bits);
-       rt_printk("irq dma bits 0x%x\n", devpriv->irq_dma_bits);
-       rt_printk("status bits 0x%x\n", inw(dev->iobase + STATUS_REG));
+       printk("config bits 0x%x\n", devpriv->config_bits);
+       printk("irq dma bits 0x%x\n", devpriv->irq_dma_bits);
+       printk("status bits 0x%x\n", inw(dev->iobase + STATUS_REG));
 }
 
 #endif
 
 /* interrupt service routine */
-static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
+static irqreturn_t a2150_interrupt(int irq, void *d)
 {
        int i;
        int status;
@@ -224,7 +227,7 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
                comedi_error(dev, "premature interrupt");
                return IRQ_HANDLED;
        }
-       // initialize async here to make sure s is not NULL
+       /*  initialize async here to make sure s is not NULL */
        async = s->async;
        async->events = 0;
        cmd = &async->cmd;
@@ -256,7 +259,7 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
         * count and address get set correctly */
        clear_dma_ff(devpriv->dma);
 
-       // figure out how many points to read
+       /*  figure out how many points to read */
        max_points = devpriv->dma_transfer_size / sample_size;
        /* residue is the number of points left to be done on the dma
         * transfer.  It should always be zero at this point unless
@@ -267,7 +270,7 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
        if (devpriv->count < num_points && cmd->stop_src == TRIG_COUNT)
                num_points = devpriv->count;
 
-       // figure out how many points will be stored next time
+       /*  figure out how many points will be stored next time */
        leftover = 0;
        if (cmd->stop_src == TRIG_NONE) {
                leftover = devpriv->dma_transfer_size / sample_size;
@@ -286,7 +289,7 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
        for (i = 0; i < num_points; i++) {
                /* write data point to comedi buffer */
                dpnt = devpriv->dma_buffer[i];
-               // convert from 2's complement to unsigned coding
+               /*  convert from 2's complement to unsigned coding */
                dpnt ^= 0x8000;
                cfc_write_to_buffer(s, dpnt);
                if (cmd->stop_src == TRIG_COUNT) {
@@ -297,7 +300,7 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
                        }
                }
        }
-       // re-enable  dma
+       /*  re-enable  dma */
        if (leftover) {
                set_dma_addr(devpriv->dma, virt_to_bus(devpriv->dma_buffer));
                set_dma_count(devpriv->dma, leftover * sample_size);
@@ -315,14 +318,14 @@ static irqreturn_t a2150_interrupt(int irq, void *d PT_REGS_ARG)
        return IRQ_HANDLED;
 }
 
-// probes board type, returns offset
-static int a2150_probe(struct comedi_device * dev)
+/* probes board type, returns offset */
+static int a2150_probe(struct comedi_device *dev)
 {
        int status = inw(dev->iobase + STATUS_REG);
        return ID_BITS(status);
 }
 
-static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
+static int a2150_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 {
        struct comedi_subdevice *s;
        unsigned long iobase = it->options[0];
@@ -332,7 +335,7 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
        int i;
 
        printk("comedi%d: %s: io 0x%lx", dev->minor, driver_a2150.driver_name,
-               iobase);
+              iobase);
        if (irq) {
                printk(", irq %u", irq);
        } else {
@@ -346,7 +349,7 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
        printk("\n");
 
        /* allocate and initialize dev->private */
-       if (alloc_private(dev, sizeof(a2150_private)) < 0)
+       if (alloc_private(dev, sizeof(struct a2150_private)) < 0)
                return -ENOMEM;
 
        if (iobase == 0) {
@@ -363,12 +366,12 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
 
        /* grab our IRQ */
        if (irq) {
-               // check that irq is supported
+               /*  check that irq is supported */
                if (irq < 3 || irq == 8 || irq == 13 || irq > 15) {
                        printk(" invalid irq line %u\n", irq);
                        return -EINVAL;
                }
-               if (comedi_request_irq(irq, a2150_interrupt, 0,
+               if (request_irq(irq, a2150_interrupt, 0,
                                driver_a2150.driver_name, dev)) {
                        printk("unable to allocate irq %u\n", irq);
                        return -EINVAL;
@@ -376,7 +379,7 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
                devpriv->irq_dma_bits |= IRQ_LVL_BITS(irq);
                dev->irq = irq;
        }
-       // initialize dma
+       /*  initialize dma */
        if (dma) {
                if (dma == 4 || dma > 7) {
                        printk(" invalid dma channel %u\n", dma);
@@ -388,7 +391,7 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
                }
                devpriv->dma = dma;
                devpriv->dma_buffer =
-                       kmalloc(A2150_DMA_BUFFER_SIZE, GFP_KERNEL | GFP_DMA);
+                   kmalloc(A2150_DMA_BUFFER_SIZE, GFP_KERNEL | GFP_DMA);
                if (devpriv->dma_buffer == NULL)
                        return -ENOMEM;
 
@@ -422,23 +425,24 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
         * prevent hardware count from stopping aquisition */
        outw(HW_COUNT_DISABLE, dev->iobase + I8253_MODE_REG);
 
-       // set card's irq and dma levels
+       /*  set card's irq and dma levels */
        outw(devpriv->irq_dma_bits, dev->iobase + IRQ_DMA_CNTRL_REG);
 
-       // reset and sync adc clock circuitry
+       /*  reset and sync adc clock circuitry */
        outw_p(DPD_BIT | APD_BIT, dev->iobase + CONFIG_REG);
        outw_p(DPD_BIT, dev->iobase + CONFIG_REG);
-       // initialize configuration register
+       /*  initialize configuration register */
        devpriv->config_bits = 0;
        outw(devpriv->config_bits, dev->iobase + CONFIG_REG);
-       // wait until offset calibration is done, then enable analog inputs
+       /*  wait until offset calibration is done, then enable analog inputs */
        for (i = 0; i < timeout; i++) {
                if ((DCAL_BIT & inw(dev->iobase + STATUS_REG)) == 0)
                        break;
-               comedi_udelay(1000);
+               udelay(1000);
        }
        if (i == timeout) {
-               printk(" timed out waiting for offset calibration to complete\n");
+               printk
+                   (" timed out waiting for offset calibration to complete\n");
                return -ETIME;
        }
        devpriv->config_bits |= ENABLE0_BIT | ENABLE1_BIT;
@@ -447,19 +451,19 @@ static int a2150_attach(struct comedi_device * dev, comedi_devconfig * it)
        return 0;
 };
 
-static int a2150_detach(struct comedi_device * dev)
+static int a2150_detach(struct comedi_device *dev)
 {
        printk("comedi%d: %s: remove\n", dev->minor, driver_a2150.driver_name);
 
        /* only free stuff if it has been allocated by _attach */
        if (dev->iobase) {
-               // put board in power-down mode
+               /*  put board in power-down mode */
                outw(APD_BIT | DPD_BIT, dev->iobase + CONFIG_REG);
                release_region(dev->iobase, A2150_SIZE);
        }
 
        if (dev->irq)
-               comedi_free_irq(dev->irq, dev);
+               free_irq(dev->irq, dev);
        if (devpriv) {
                if (devpriv->dma)
                        free_dma(devpriv->dma);
@@ -470,23 +474,23 @@ static int a2150_detach(struct comedi_device * dev)
        return 0;
 };
 
-static int a2150_cancel(struct comedi_device * dev, struct comedi_subdevice * s)
+static int a2150_cancel(struct comedi_device *dev, struct comedi_subdevice *s)
 {
-       // disable dma on card
+       /*  disable dma on card */
        devpriv->irq_dma_bits &= ~DMA_INTR_EN_BIT & ~DMA_EN_BIT;
        outw(devpriv->irq_dma_bits, dev->iobase + IRQ_DMA_CNTRL_REG);
 
-       // disable computer's dma
+       /*  disable computer's dma */
        disable_dma(devpriv->dma);
 
-       // clear fifo and reset triggering circuitry
+       /*  clear fifo and reset triggering circuitry */
        outw(0, dev->iobase + FIFO_RESET_REG);
 
        return 0;
 }
 
-static int a2150_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice * s,
-       struct comedi_cmd * cmd)
+static int a2150_ai_cmdtest(struct comedi_device *dev,
+                           struct comedi_subdevice *s, struct comedi_cmd *cmd)
 {
        int err = 0;
        int tmp;
@@ -580,31 +584,30 @@ static int a2150_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        if (err)
                return 4;
 
-       // check channel/gain list against card's limitations
+       /*  check channel/gain list against card's limitations */
        if (cmd->chanlist) {
                startChan = CR_CHAN(cmd->chanlist[0]);
                for (i = 1; i < cmd->chanlist_len; i++) {
                        if (CR_CHAN(cmd->chanlist[i]) != (startChan + i)) {
                                comedi_error(dev,
-                                       "entries in chanlist must be consecutive channels, counting upwards\n");
+                                            "entries in chanlist must be consecutive channels, counting upwards\n");
                                err++;
                        }
                }
                if (cmd->chanlist_len == 2 && CR_CHAN(cmd->chanlist[0]) == 1) {
                        comedi_error(dev,
-                               "length 2 chanlist must be channels 0,1 or channels 2,3");
+                                    "length 2 chanlist must be channels 0,1 or channels 2,3");
                        err++;
                }
                if (cmd->chanlist_len == 3) {
                        comedi_error(dev,
-                               "chanlist must have 1,2 or 4 channels");
+                                    "chanlist must have 1,2 or 4 channels");
                        err++;
                }
                if (CR_AREF(cmd->chanlist[0]) != CR_AREF(cmd->chanlist[1]) ||
-                       CR_AREF(cmd->chanlist[2]) != CR_AREF(cmd->chanlist[3]))
-               {
+                   CR_AREF(cmd->chanlist[2]) != CR_AREF(cmd->chanlist[3])) {
                        comedi_error(dev,
-                               "channels 0/1 and 2/3 must have the same analog reference");
+                                    "channels 0/1 and 2/3 must have the same analog reference");
                        err++;
                }
        }
@@ -615,7 +618,7 @@ static int a2150_ai_cmdtest(struct comedi_device * dev, struct comedi_subdevice
        return 0;
 }
 
-static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
+static int a2150_ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 {
        struct comedi_async *async = s->async;
        struct comedi_cmd *cmd = &async->cmd;
@@ -625,23 +628,23 @@ static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
 
        if (!dev->irq || !devpriv->dma) {
                comedi_error(dev,
-                       " irq and dma required, cannot do hardware conversions");
+                            " irq and dma required, cannot do hardware conversions");
                return -1;
        }
        if (cmd->flags & TRIG_RT) {
                comedi_error(dev,
-                       " dma incompatible with hard real-time interrupt (TRIG_RT), aborting");
+                            " dma incompatible with hard real-time interrupt (TRIG_RT), aborting");
                return -1;
        }
-       // clear fifo and reset triggering circuitry
+       /*  clear fifo and reset triggering circuitry */
        outw(0, dev->iobase + FIFO_RESET_REG);
 
        /* setup chanlist */
        if (a2150_set_chanlist(dev, CR_CHAN(cmd->chanlist[0]),
-                       cmd->chanlist_len) < 0)
+                              cmd->chanlist_len) < 0)
                return -1;
 
-       // setup ac/dc coupling
+       /*  setup ac/dc coupling */
        if (CR_AREF(cmd->chanlist[0]) == AREF_OTHER)
                devpriv->config_bits |= AC0_BIT;
        else
@@ -651,33 +654,33 @@ static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
        else
                devpriv->config_bits &= ~AC1_BIT;
 
-       // setup timing
+       /*  setup timing */
        a2150_get_timing(dev, &cmd->scan_begin_arg, cmd->flags);
 
-       // send timing, channel, config bits
+       /*  send timing, channel, config bits */
        outw(devpriv->config_bits, dev->iobase + CONFIG_REG);
 
-       // initialize number of samples remaining
+       /*  initialize number of samples remaining */
        devpriv->count = cmd->stop_arg * cmd->chanlist_len;
 
-       // enable computer's dma
+       /*  enable computer's dma */
        lock_flags = claim_dma_lock();
        disable_dma(devpriv->dma);
        /* clear flip-flop to make sure 2-byte registers for
         * count and address get set correctly */
        clear_dma_ff(devpriv->dma);
        set_dma_addr(devpriv->dma, virt_to_bus(devpriv->dma_buffer));
-       // set size of transfer to fill in 1/3 second
+       /*  set size of transfer to fill in 1/3 second */
 #define ONE_THIRD_SECOND 333333333
        devpriv->dma_transfer_size =
-               sizeof(devpriv->dma_buffer[0]) * cmd->chanlist_len *
-               ONE_THIRD_SECOND / cmd->scan_begin_arg;
+           sizeof(devpriv->dma_buffer[0]) * cmd->chanlist_len *
+           ONE_THIRD_SECOND / cmd->scan_begin_arg;
        if (devpriv->dma_transfer_size > A2150_DMA_BUFFER_SIZE)
                devpriv->dma_transfer_size = A2150_DMA_BUFFER_SIZE;
        if (devpriv->dma_transfer_size < sizeof(devpriv->dma_buffer[0]))
                devpriv->dma_transfer_size = sizeof(devpriv->dma_buffer[0]);
        devpriv->dma_transfer_size -=
-               devpriv->dma_transfer_size % sizeof(devpriv->dma_buffer[0]);
+           devpriv->dma_transfer_size % sizeof(devpriv->dma_buffer[0]);
        set_dma_count(devpriv->dma, devpriv->dma_transfer_size);
        enable_dma(devpriv->dma);
        release_dma_lock(lock_flags);
@@ -686,36 +689,36 @@ static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
         * one spurious interrupt that has been happening */
        outw(0x00, dev->iobase + DMA_TC_CLEAR_REG);
 
-       // enable dma on card
+       /*  enable dma on card */
        devpriv->irq_dma_bits |= DMA_INTR_EN_BIT | DMA_EN_BIT;
        outw(devpriv->irq_dma_bits, dev->iobase + IRQ_DMA_CNTRL_REG);
 
-       // may need to wait 72 sampling periods if timing was changed
+       /*  may need to wait 72 sampling periods if timing was changed */
        i8254_load(dev->iobase + I8253_BASE_REG, 0, 2, 72, 0);
 
-       // setup start triggering
+       /*  setup start triggering */
        trigger_bits = 0;
-       // decide if we need to wait 72 periods for valid data
+       /*  decide if we need to wait 72 periods for valid data */
        if (cmd->start_src == TRIG_NOW &&
-               (old_config_bits & CLOCK_MASK) !=
-               (devpriv->config_bits & CLOCK_MASK)) {
-               // set trigger source to delay trigger
+           (old_config_bits & CLOCK_MASK) !=
+           (devpriv->config_bits & CLOCK_MASK)) {
+               /*  set trigger source to delay trigger */
                trigger_bits |= DELAY_TRIGGER_BITS;
        } else {
-               // otherwise no delay
+               /*  otherwise no delay */
                trigger_bits |= POST_TRIGGER_BITS;
        }
-       // enable external hardware trigger
+       /*  enable external hardware trigger */
        if (cmd->start_src == TRIG_EXT) {
                trigger_bits |= HW_TRIG_EN;
        } else if (cmd->start_src == TRIG_OTHER) {
-               // XXX add support for level/slope start trigger using TRIG_OTHER
+               /*  XXX add support for level/slope start trigger using TRIG_OTHER */
                comedi_error(dev, "you shouldn't see this?");
        }
-       // send trigger config bits
+       /*  send trigger config bits */
        outw(trigger_bits, dev->iobase + TRIGGER_REG);
 
-       // start aquisition for soft trigger
+       /*  start aquisition for soft trigger */
        if (cmd->start_src == TRIG_NOW) {
                outw(0, dev->iobase + FIFO_START_REG);
        }
@@ -726,35 +729,35 @@ static int a2150_ai_cmd(struct comedi_device * dev, struct comedi_subdevice * s)
        return 0;
 }
 
-static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice * s,
-       comedi_insn * insn, unsigned int * data)
+static int a2150_ai_rinsn(struct comedi_device *dev, struct comedi_subdevice *s,
+                         struct comedi_insn *insn, unsigned int *data)
 {
        unsigned int i, n;
        static const int timeout = 100000;
        static const int filter_delay = 36;
 
-       // clear fifo and reset triggering circuitry
+       /*  clear fifo and reset triggering circuitry */
        outw(0, dev->iobase + FIFO_RESET_REG);
 
        /* setup chanlist */
        if (a2150_set_chanlist(dev, CR_CHAN(insn->chanspec), 1) < 0)
                return -1;
 
-       // set dc coupling
+       /*  set dc coupling */
        devpriv->config_bits &= ~AC0_BIT;
        devpriv->config_bits &= ~AC1_BIT;
 
-       // send timing, channel, config bits
+       /*  send timing, channel, config bits */
        outw(devpriv->config_bits, dev->iobase + CONFIG_REG);
 
-       // disable dma on card
+       /*  disable dma on card */
        devpriv->irq_dma_bits &= ~DMA_INTR_EN_BIT & ~DMA_EN_BIT;
        outw(devpriv->irq_dma_bits, dev->iobase + IRQ_DMA_CNTRL_REG);
 
-       // setup start triggering
+       /*  setup start triggering */
        outw(0, dev->iobase + TRIGGER_REG);
 
-       // start aquisition for soft trigger
+       /*  start aquisition for soft trigger */
        outw(0, dev->iobase + FIFO_START_REG);
 
        /* there is a 35.6 sample delay for data to get through the antialias filter */
@@ -762,7 +765,7 @@ static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
                for (i = 0; i < timeout; i++) {
                        if (inw(dev->iobase + STATUS_REG) & FNE_BIT)
                                break;
-                       comedi_udelay(1);
+                       udelay(1);
                }
                if (i == timeout) {
                        comedi_error(dev, "timeout");
@@ -771,12 +774,12 @@ static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
                inw(dev->iobase + FIFO_DATA_REG);
        }
 
-       // read data
+       /*  read data */
        for (n = 0; n < insn->n; n++) {
                for (i = 0; i < timeout; i++) {
                        if (inw(dev->iobase + STATUS_REG) & FNE_BIT)
                                break;
-                       comedi_udelay(1);
+                       udelay(1);
                }
                if (i == timeout) {
                        comedi_error(dev, "timeout");
@@ -787,12 +790,12 @@ static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
 #endif
                data[n] = inw(dev->iobase + FIFO_DATA_REG);
 #ifdef A2150_DEBUG
-               rt_printk(" data is %i\n", data[n]);
+               printk(" data is %i\n", data[n]);
 #endif
                data[n] ^= 0x8000;
        }
 
-       // clear fifo and reset triggering circuitry
+       /*  clear fifo and reset triggering circuitry */
        outw(0, dev->iobase + FIFO_RESET_REG);
 
        return n;
@@ -800,14 +803,14 @@ static int a2150_ai_rinsn(struct comedi_device * dev, struct comedi_subdevice *
 
 /* sets bits in devpriv->clock_bits to nearest approximation of requested period,
  * adjusts requested period to actual timing. */
-static int a2150_get_timing(struct comedi_device * dev, unsigned int *period,
-       int flags)
+static int a2150_get_timing(struct comedi_device *dev, unsigned int *period,
+                           int flags)
 {
        int lub, glb, temp;
        int lub_divisor_shift, lub_index, glb_divisor_shift, glb_index;
        int i, j;
 
-       // initialize greatest lower and least upper bounds
+       /*  initialize greatest lower and least upper bounds */
        lub_divisor_shift = 3;
        lub_index = 0;
        lub = thisboard->clock[lub_index] * (1 << lub_divisor_shift);
@@ -815,19 +818,19 @@ static int a2150_get_timing(struct comedi_device * dev, unsigned int *period,
        glb_index = thisboard->num_clocks - 1;
        glb = thisboard->clock[glb_index] * (1 << glb_divisor_shift);
 
-       // make sure period is in available range
+       /*  make sure period is in available range */
        if (*period < glb)
                *period = glb;
        if (*period > lub)
                *period = lub;
 
-       // we can multiply period by 1, 2, 4, or 8, using (1 << i)
+       /*  we can multiply period by 1, 2, 4, or 8, using (1 << i) */
        for (i = 0; i < 4; i++) {
-               // there are a maximum of 4 master clocks
+               /*  there are a maximum of 4 master clocks */
                for (j = 0; j < thisboard->num_clocks; j++) {
-                       // temp is the period in nanosec we are evaluating
+                       /*  temp is the period in nanosec we are evaluating */
                        temp = thisboard->clock[j] * (1 << i);
-                       // if it is the best match yet
+                       /*  if it is the best match yet */
                        if (temp < lub && temp >= *period) {
                                lub_divisor_shift = i;
                                lub_index = j;
@@ -844,7 +847,7 @@ static int a2150_get_timing(struct comedi_device * dev, unsigned int *period,
        switch (flags) {
        case TRIG_ROUND_NEAREST:
        default:
-               // if least upper bound is better approximation
+               /*  if least upper bound is better approximation */
                if (lub - *period < *period - glb) {
                        *period = lub;
                } else {
@@ -859,23 +862,24 @@ static int a2150_get_timing(struct comedi_device * dev, unsigned int *period,
                break;
        }
 
-       // set clock bits for config register appropriately
+       /*  set clock bits for config register appropriately */
        devpriv->config_bits &= ~CLOCK_MASK;
        if (*period == lub) {
                devpriv->config_bits |=
-                       CLOCK_SELECT_BITS(lub_index) |
-                       CLOCK_DIVISOR_BITS(lub_divisor_shift);
+                   CLOCK_SELECT_BITS(lub_index) |
+                   CLOCK_DIVISOR_BITS(lub_divisor_shift);
        } else {
                devpriv->config_bits |=
-                       CLOCK_SELECT_BITS(glb_index) |
-                       CLOCK_DIVISOR_BITS(glb_divisor_shift);
+                   CLOCK_SELECT_BITS(glb_index) |
+                   CLOCK_DIVISOR_BITS(glb_divisor_shift);
        }
 
        return 0;
 }
 
-static int a2150_set_chanlist(struct comedi_device * dev, unsigned int start_channel,
-       unsigned int num_channels)
+static int a2150_set_chanlist(struct comedi_device *dev,
+                             unsigned int start_channel,
+                             unsigned int num_channels)
 {
        if (start_channel + num_channels > 4)
                return -1;