diff options
| author | Alexandru Ardelean <alexandru.ardelean@analog.com> | 2019-09-17 19:10:23 +0300 |
|---|---|---|
| committer | Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2019-10-05 15:45:03 +0300 |
| commit | ea14163d29130649e43bfcad12ef6ef9a956fbe3 (patch) | |
| tree | 50963f4d11885a566f2be1ad6c9923ac4840545a | |
| parent | 8033997a8dafbad1aae437bd25f0d7307fb084be (diff) | |
| download | linux-ea14163d29130649e43bfcad12ef6ef9a956fbe3.tar.xz | |
iio: gyro: adis16130: remove mlock usage
The use of indio_dev's mlock is discouraged. The driver already defines
it's own `bus_lock` in `adis16130_spi_read()`, so using the mlock is
redundant.
The parts supported by this chip are obsoleted anyway, so for now we just
remove mlock as part of a general cleanup, until the driver gets removed.
Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
| -rw-r--r-- | drivers/iio/gyro/adis16130.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/iio/gyro/adis16130.c b/drivers/iio/gyro/adis16130.c index de3f66f89496..79e63c8a2ea8 100644 --- a/drivers/iio/gyro/adis16130.c +++ b/drivers/iio/gyro/adis16130.c @@ -76,9 +76,7 @@ static int adis16130_read_raw(struct iio_dev *indio_dev, switch (mask) { case IIO_CHAN_INFO_RAW: /* Take the iio_dev status lock */ - mutex_lock(&indio_dev->mlock); ret = adis16130_spi_read(indio_dev, chan->address, &temp); - mutex_unlock(&indio_dev->mlock); if (ret) return ret; *val = temp; |