staging:iio: Remove naming via IDR's where no longer necessary under new abi.
authorJonathan Cameron <jic23@cam.ac.uk>
Tue, 4 May 2010 13:43:08 +0000 (14:43 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 11 May 2010 18:36:06 +0000 (11:36 -0700)
Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/iio/accel/lis3l02dq_ring.c
drivers/staging/iio/accel/sca3000_core.c
drivers/staging/iio/adc/max1363_ring.c
drivers/staging/iio/industrialio-ring.c
drivers/staging/iio/ring_generic.h

index bba4b0925aceee4ce373d1379be207504c82f1d2..7617da81871c687b6680954eaa03e81415643483 100644 (file)
@@ -581,7 +581,7 @@ error_iio_sw_rb_free:
 
 int lis3l02dq_initialize_ring(struct iio_ring_buffer *ring)
 {
-       return iio_ring_buffer_register(ring);
+       return iio_ring_buffer_register(ring, 0);
 }
 
 void lis3l02dq_uninitialize_ring(struct iio_ring_buffer *ring)
index 45e47776d913f2be6ebe104898f70cbc77bf5af8..9485b132f6dd4325ee6590d0f71955b8095edd59 100644 (file)
@@ -1338,7 +1338,7 @@ static int __devinit __sca3000_probe(struct spi_device *spi,
        if (ret < 0)
                goto error_free_dev;
        regdone = 1;
-       ret = iio_ring_buffer_register(st->indio_dev->ring);
+       ret = iio_ring_buffer_register(st->indio_dev->ring, 0);
        if (ret < 0)
                goto error_unregister_dev;
        if (spi->irq && gpio_is_valid(irq_to_gpio(spi->irq)) > 0) {
index 6003f7e10a6e49ffbb88f7c0b9761a5916e8376d..f1e37f2cf4ebdb10a1358ac21959a35c3c7224dc 100644 (file)
@@ -255,5 +255,5 @@ void max1363_uninitialize_ring(struct iio_ring_buffer *ring)
 
 int max1363_initialize_ring(struct iio_ring_buffer *ring)
 {
-       return iio_ring_buffer_register(ring);
+       return iio_ring_buffer_register(ring, 0);
 };
index 0f19bd1b57603bae93b8c2a474881f4102518f31..fc27d22a7ab104c21f7affa84a2aebc8147e8dfa 100644 (file)
 #include <linux/poll.h>
 #include <linux/module.h>
 #include <linux/cdev.h>
-#include <linux/idr.h>
 #include <linux/slab.h>
 
 #include "iio.h"
 #include "ring_generic.h"
 
-/* IDR for ring buffer identifier */
-static DEFINE_IDR(iio_ring_idr);
-/* IDR for ring event identifier */
-static DEFINE_IDR(iio_ring_event_idr);
-/* IDR for ring access identifier */
-static DEFINE_IDR(iio_ring_access_idr);
-
 int iio_push_ring_event(struct iio_ring_buffer *ring_buf,
                       int event_code,
                       s64 timestamp)
@@ -158,11 +150,8 @@ __iio_request_ring_buffer_event_chrdev(struct iio_ring_buffer *buf,
                                       struct device *dev)
 {
        int ret;
-       ret = iio_get_new_idr_val(&iio_ring_event_idr);
-       if (ret < 0)
-               goto error_ret;
-       else
-               buf->ev_int.id = ret;
+
+       buf->ev_int.id = id;
 
        snprintf(buf->ev_int._name, sizeof(buf->ev_int._name),
                 "%s:event%d",
@@ -173,11 +162,9 @@ __iio_request_ring_buffer_event_chrdev(struct iio_ring_buffer *buf,
                               owner,
                               dev);
        if (ret)
-               goto error_free_id;
+               goto error_ret;
        return 0;
 
-error_free_id:
-       iio_free_idr_val(&iio_ring_event_idr, buf->ev_int.id);
 error_ret:
        return ret;
 }
@@ -186,7 +173,6 @@ static inline void
 __iio_free_ring_buffer_event_chrdev(struct iio_ring_buffer *buf)
 {
        iio_free_ev_int(&(buf->ev_int));
-       iio_free_idr_val(&iio_ring_event_idr, buf->ev_int.id);
 }
 
 static void iio_ring_access_release(struct device *dev)
@@ -222,18 +208,16 @@ __iio_request_ring_buffer_access_chrdev(struct iio_ring_buffer *buf,
        }
        buf->access_dev.devt = MKDEV(MAJOR(iio_devt), minor);
 
-       ret = iio_get_new_idr_val(&iio_ring_access_idr);
-       if (ret < 0)
-               goto error_device_put;
-       else
-               buf->access_id = ret;
+
+       buf->access_id = id;
+
        dev_set_name(&buf->access_dev, "%s:access%d",
                     dev_name(&buf->dev),
                     buf->access_id);
        ret = device_add(&buf->access_dev);
        if (ret < 0) {
                printk(KERN_ERR "failed to add the ring access dev\n");
-               goto error_free_idr;
+               goto error_device_put;
        }
 
        cdev_init(&buf->access_handler.chrdev, &iio_ring_fileops);
@@ -245,10 +229,9 @@ __iio_request_ring_buffer_access_chrdev(struct iio_ring_buffer *buf,
                goto error_device_unregister;
        }
        return 0;
+
 error_device_unregister:
        device_unregister(&buf->access_dev);
-error_free_idr:
-       iio_free_idr_val(&iio_ring_access_idr, buf->access_id);
 error_device_put:
        put_device(&buf->access_dev);
 
@@ -257,7 +240,6 @@ error_device_put:
 
 static void __iio_free_ring_buffer_access_chrdev(struct iio_ring_buffer *buf)
 {
-       iio_free_idr_val(&iio_ring_access_idr, buf->access_id);
        device_unregister(&buf->access_dev);
 }
 
@@ -274,21 +256,18 @@ void iio_ring_buffer_init(struct iio_ring_buffer *ring,
 }
 EXPORT_SYMBOL(iio_ring_buffer_init);
 
-int iio_ring_buffer_register(struct iio_ring_buffer *ring)
+int iio_ring_buffer_register(struct iio_ring_buffer *ring, int id)
 {
        int ret;
-       ret = iio_get_new_idr_val(&iio_ring_idr);
-       if (ret < 0)
-               goto error_ret;
-       else
-               ring->id = ret;
+
+       ring->id = id;
 
        dev_set_name(&ring->dev, "%s:buffer%d",
                     dev_name(ring->dev.parent),
                     ring->id);
        ret = device_add(&ring->dev);
        if (ret)
-               goto error_free_id;
+               goto error_ret;
 
        ret = __iio_request_ring_buffer_event_chrdev(ring,
                                                     0,
@@ -309,8 +288,6 @@ error_free_ring_buffer_event_chrdev:
        __iio_free_ring_buffer_event_chrdev(ring);
 error_remove_device:
        device_del(&ring->dev);
-error_free_id:
-       iio_free_idr_val(&iio_ring_idr, ring->id);
 error_ret:
        return ret;
 }
@@ -321,7 +298,6 @@ void iio_ring_buffer_unregister(struct iio_ring_buffer *ring)
        __iio_free_ring_buffer_access_chrdev(ring);
        __iio_free_ring_buffer_event_chrdev(ring);
        device_del(&ring->dev);
-       iio_free_idr_val(&iio_ring_idr, ring->id);
 }
 EXPORT_SYMBOL(iio_ring_buffer_unregister);
 
index d74b5aea0112383804ba9a77ad94a2686d345951..0e443757b029ab6a928aa07fa8a60aba8a863445 100644 (file)
@@ -259,7 +259,7 @@ static inline void iio_put_ring_buffer(struct iio_ring_buffer *ring)
        container_of(d, struct iio_ring_buffer, dev)
 #define access_dev_to_iio_ring_buffer(d)                       \
        container_of(d, struct iio_ring_buffer, access_dev)
-int iio_ring_buffer_register(struct iio_ring_buffer *ring);
+int iio_ring_buffer_register(struct iio_ring_buffer *ring, int id);
 void iio_ring_buffer_unregister(struct iio_ring_buffer *ring);
 
 ssize_t iio_read_ring_length(struct device *dev,