iio: gyro: adis16130: remove mlock usage
authorAlexandru Ardelean <alexandru.ardelean@analog.com>
Tue, 17 Sep 2019 16:10:23 +0000 (19:10 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 5 Oct 2019 12:45:03 +0000 (13:45 +0100)
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>
drivers/iio/gyro/adis16130.c

index de3f66f..79e63c8 100644 (file)
@@ -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;