Commit dc6ac050 authored by Linus Walleij's avatar Linus Walleij Committed by Jonathan Cameron

iio: accel: kxsd9: Drop the buffer lock

The RX/TX buffers are gone so drop the lock (it should have been
in the transport struct anyway).
Tested-by: default avatarJonathan Cameron <jic23@kernel.org>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent a483ab79
...@@ -41,13 +41,10 @@ ...@@ -41,13 +41,10 @@
/** /**
* struct kxsd9_state - device related storage * struct kxsd9_state - device related storage
* @transport: transport for the KXSD9 * @map: regmap to the device
* @buf_lock: protect the rx and tx buffers. */
* @us: spi device
**/
struct kxsd9_state { struct kxsd9_state {
struct regmap *map; struct regmap *map;
struct mutex buf_lock;
}; };
#define KXSD9_SCALE_2G "0.011978" #define KXSD9_SCALE_2G "0.011978"
...@@ -73,7 +70,6 @@ static int kxsd9_write_scale(struct iio_dev *indio_dev, int micro) ...@@ -73,7 +70,6 @@ static int kxsd9_write_scale(struct iio_dev *indio_dev, int micro)
if (!foundit) if (!foundit)
return -EINVAL; return -EINVAL;
mutex_lock(&st->buf_lock);
ret = regmap_read(st->map, ret = regmap_read(st->map,
KXSD9_REG_CTRL_C, KXSD9_REG_CTRL_C,
&val); &val);
...@@ -83,7 +79,6 @@ static int kxsd9_write_scale(struct iio_dev *indio_dev, int micro) ...@@ -83,7 +79,6 @@ static int kxsd9_write_scale(struct iio_dev *indio_dev, int micro)
KXSD9_REG_CTRL_C, KXSD9_REG_CTRL_C,
(val & ~KXSD9_FS_MASK) | i); (val & ~KXSD9_FS_MASK) | i);
error_ret: error_ret:
mutex_unlock(&st->buf_lock);
return ret; return ret;
} }
...@@ -93,15 +88,11 @@ static int kxsd9_read(struct iio_dev *indio_dev, u8 address) ...@@ -93,15 +88,11 @@ static int kxsd9_read(struct iio_dev *indio_dev, u8 address)
struct kxsd9_state *st = iio_priv(indio_dev); struct kxsd9_state *st = iio_priv(indio_dev);
__be16 raw_val; __be16 raw_val;
mutex_lock(&st->buf_lock);
ret = regmap_bulk_read(st->map, address, &raw_val, sizeof(raw_val)); ret = regmap_bulk_read(st->map, address, &raw_val, sizeof(raw_val));
if (ret) if (ret)
goto out_fail_read; return ret;
/* Only 12 bits are valid */ /* Only 12 bits are valid */
ret = be16_to_cpu(raw_val) & 0xfff0; return be16_to_cpu(raw_val) & 0xfff0;
out_fail_read:
mutex_unlock(&st->buf_lock);
return ret;
} }
static IIO_CONST_ATTR(accel_scale_available, static IIO_CONST_ATTR(accel_scale_available,
...@@ -220,7 +211,6 @@ int kxsd9_common_probe(struct device *parent, ...@@ -220,7 +211,6 @@ int kxsd9_common_probe(struct device *parent,
st = iio_priv(indio_dev); st = iio_priv(indio_dev);
st->map = map; st->map = map;
mutex_init(&st->buf_lock);
indio_dev->channels = kxsd9_channels; indio_dev->channels = kxsd9_channels;
indio_dev->num_channels = ARRAY_SIZE(kxsd9_channels); indio_dev->num_channels = ARRAY_SIZE(kxsd9_channels);
indio_dev->name = name; indio_dev->name = name;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment