staging: comedi: pass subdevice to comedi_buf_write_alloc()
authorIan Abbott <abbotti@mev.co.uk>
Tue, 6 May 2014 12:12:04 +0000 (13:12 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 May 2014 12:25:48 +0000 (21:25 +0900)
Change the parameters of `comedi_buf_write_alloc()` to pass a pointer to
the comedi subdevice instead of a pointer to the "async" structure
belonging to the subdevice.

The main aim at the moment is to replace all the `struct comedi_async *`
parameters with `struct comedi_subdevice *` parameters in the comedi
driver API.

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/comedi_buf.c
drivers/staging/comedi/comedi_fops.c
drivers/staging/comedi/comedidev.h
drivers/staging/comedi/drivers/addi-data/hwdrv_apci3200.c
drivers/staging/comedi/drivers/comedi_fc.c
drivers/staging/comedi/drivers/mite.c
drivers/staging/comedi/drivers/ni_mio_common.c
drivers/staging/comedi/drivers/ni_pcidio.c
drivers/staging/comedi/drivers/ni_tiocmd.c

index 199ebb48974c3d9d5f0994d44279455e17423729..6a655008655fb3640e8a07db9b3731740a2f9867 100644 (file)
@@ -269,10 +269,10 @@ static unsigned int __comedi_buf_write_alloc(struct comedi_async *async,
 }
 
 /* allocates chunk for the writer from free buffer space */
-unsigned int comedi_buf_write_alloc(struct comedi_async *async,
+unsigned int comedi_buf_write_alloc(struct comedi_subdevice *s,
                                    unsigned int nbytes)
 {
-       return __comedi_buf_write_alloc(async, nbytes, 0);
+       return __comedi_buf_write_alloc(s->async, nbytes, 0);
 }
 EXPORT_SYMBOL_GPL(comedi_buf_write_alloc);
 
index 0affd1f96660945efcac2dd9994919373af87206..58c1f6e18d27917a2a2831c3857b2193587152c9 100644 (file)
@@ -1004,7 +1004,7 @@ static int do_bufinfo_ioctl(struct comedi_device *dev,
 
        if (bi.bytes_written && (s->subdev_flags & SDF_CMD_WRITE)) {
                bi.bytes_written =
-                   comedi_buf_write_alloc(async, bi.bytes_written);
+                   comedi_buf_write_alloc(s, bi.bytes_written);
                comedi_buf_write_free(async, bi.bytes_written);
        }
 
@@ -2038,7 +2038,7 @@ static unsigned int comedi_poll(struct file *file, poll_table *wait)
                unsigned int bps = bytes_per_sample(s);
 
                poll_wait(file, &s->async->wait_head, wait);
-               comedi_buf_write_alloc(s->async, s->async->prealloc_bufsz);
+               comedi_buf_write_alloc(s, s->async->prealloc_bufsz);
                if (!s->busy || !comedi_is_subdevice_running(s) ||
                    comedi_buf_write_n_allocated(s->async) >= bps)
                        mask |= POLLOUT | POLLWRNORM;
@@ -2136,7 +2136,7 @@ static ssize_t comedi_write(struct file *file, const char __user *buf,
                m = n;
                if (async->buf_write_ptr + m > async->prealloc_bufsz)
                        m = async->prealloc_bufsz - async->buf_write_ptr;
-               comedi_buf_write_alloc(async, async->prealloc_bufsz);
+               comedi_buf_write_alloc(s, async->prealloc_bufsz);
                if (m > comedi_buf_write_n_allocated(async))
                        m = comedi_buf_write_n_allocated(async);
                if (m < n)
index 0a68006e2c21757eb636ddf4f270e2eb3bf2d94f..b47cc969d5ad70649bb8fd5b75f253b621a92f15 100644 (file)
@@ -336,7 +336,7 @@ static inline unsigned int bytes_per_sample(const struct comedi_subdevice *subd)
  */
 int comedi_set_hw_dev(struct comedi_device *dev, struct device *hw_dev);
 
-unsigned int comedi_buf_write_alloc(struct comedi_async *, unsigned int);
+unsigned int comedi_buf_write_alloc(struct comedi_subdevice *s, unsigned int n);
 unsigned int comedi_buf_write_free(struct comedi_async *, unsigned int);
 
 unsigned int comedi_buf_read_n_available(struct comedi_async *);
index 51095bebc08726c17d16c31cda394068bc85898e..246220abf9a9f43e0d1353c963bca107eac1aab3 100644 (file)
@@ -2702,8 +2702,7 @@ static int i_APCI3200_InterruptHandleEos(struct comedi_device *dev)
                        s->async->events |= COMEDI_CB_EOS;
 
                        /*  Test if enougth memory is available and allocate it for 7 values */
-                       /* n = comedi_buf_write_alloc(s->async, 7*sizeof(unsigned int)); */
-                       n = comedi_buf_write_alloc(s->async,
+                       n = comedi_buf_write_alloc(s,
                                (7 + 12) * sizeof(unsigned int));
 
                        /*  If not enough memory available, event is set to Comedi Buffer Error */
index 5eff9d1cd072db06dfe0e08ecf8a3c51ca63f422..21cf34e7dc780e390898b7584335857bbdc79de8 100644 (file)
@@ -67,7 +67,7 @@ unsigned int cfc_write_array_to_buffer(struct comedi_subdevice *s,
        if (num_bytes == 0)
                return 0;
 
-       retval = comedi_buf_write_alloc(async, num_bytes);
+       retval = comedi_buf_write_alloc(s, num_bytes);
        if (retval != num_bytes) {
                dev_warn(s->device->class_dev, "buffer overrun\n");
                async->events |= COMEDI_CB_OVERFLOW;
index 1a572c83f996c71f5a9a0c7a712e0378d52f5a52..a5999f52ae20f731373debc6a5edc3ae02e15a47 100644 (file)
@@ -533,7 +533,7 @@ int mite_sync_input_dma(struct mite_channel *mite_chan,
 
        old_alloc_count = async->buf_write_alloc_count;
        /* write alloc as much as we can */
-       comedi_buf_write_alloc(async, async->prealloc_bufsz);
+       comedi_buf_write_alloc(s, async->prealloc_bufsz);
 
        nbytes = mite_bytes_written_to_memory_lb(mite_chan);
        if ((int)(mite_bytes_written_to_memory_ub(mite_chan) -
index d4e99af6de717275630b5212500b7c77ea05baf3..d8415fe2188b8b44e572377428fea50d08ff90e3 100644 (file)
@@ -1505,7 +1505,7 @@ static int ni_ai_setup_MITE_dma(struct comedi_device *dev)
 /* printk("comedi_debug: using mite channel %i for ai.\n", devpriv->ai_mite_chan->channel); */
 
        /* write alloc the entire buffer */
-       comedi_buf_write_alloc(s->async, s->async->prealloc_bufsz);
+       comedi_buf_write_alloc(s, s->async->prealloc_bufsz);
 
        spin_lock_irqsave(&devpriv->mite_channel_lock, flags);
        if (devpriv->ai_mite_chan == NULL) {
index 6dd10044a02338c7453444ecd6a2c916dd424cc6..d0ad82a3b4b9b44c3d968bc896d7017f77eeb033 100644 (file)
@@ -759,7 +759,7 @@ static int setup_mite_dma(struct comedi_device *dev, struct comedi_subdevice *s)
                return retval;
 
        /* write alloc the entire buffer */
-       comedi_buf_write_alloc(s->async, s->async->prealloc_bufsz);
+       comedi_buf_write_alloc(s, s->async->prealloc_bufsz);
 
        spin_lock_irqsave(&devpriv->mite_channel_lock, flags);
        if (devpriv->di_mite_chan) {
index 7a163fc31f5d12bbb1dbde09429bb6c7c21acd7d..9cc77c0b2f0ff5ccd5a3ce2b3bc1521e60aefb23 100644 (file)
@@ -125,7 +125,7 @@ static int ni_tio_input_cmd(struct comedi_subdevice *s)
        int retval = 0;
 
        /* write alloc the entire buffer */
-       comedi_buf_write_alloc(async, async->prealloc_bufsz);
+       comedi_buf_write_alloc(s, async->prealloc_bufsz);
        counter->mite_chan->dir = COMEDI_INPUT;
        switch (counter_dev->variant) {
        case ni_gpct_variant_m_series: