staging:iio:impedance-analyser make use of iio_sw_buffer_preenable
authorJonathan Cameron <jic23@kernel.org>
Sat, 21 Apr 2012 09:09:43 +0000 (10:09 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 24 Apr 2012 18:23:38 +0000 (11:23 -0700)
This avoids some code duplication by using the generic form
in a non performance critical place.

Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/iio/impedance-analyzer/ad5933.c

index 93e5a716f9e125b719394a7c36e8b81799ba5fe0..8a0485e697470d4de4bc3dac31788e9e0c7ff522 100644 (file)
@@ -569,19 +569,14 @@ static const struct iio_info ad5933_info = {
 static int ad5933_ring_preenable(struct iio_dev *indio_dev)
 {
        struct ad5933_state *st = iio_priv(indio_dev);
-       size_t d_size;
        int ret;
 
        if (bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength))
                return -EINVAL;
 
-       d_size = bitmap_weight(indio_dev->active_scan_mask,
-                              indio_dev->masklength) *
-                ad5933_channels[1].scan_type.storagebits / 8;
-
-       if (indio_dev->buffer->access->set_bytes_per_datum)
-               indio_dev->buffer->access->
-                       set_bytes_per_datum(indio_dev->buffer, d_size);
+       ret = iio_sw_buffer_preenable(indio_dev);
+       if (ret < 0)
+               return ret;
 
        ret = ad5933_reset(st);
        if (ret < 0)