]> git.baikalelectronics.ru Git - kernel.git/commit
staging: iio: adis16060: Remove iio_dev mlock and refactor code
authorsimran singhal <singhalsimran0@gmail.com>
Wed, 22 Mar 2017 16:20:43 +0000 (21:50 +0530)
committerJonathan Cameron <jic23@kernel.org>
Thu, 23 Mar 2017 19:43:03 +0000 (19:43 +0000)
commitbe63d954be9d0ab41deccd88b0512b8b12a0d05d
treea8b2f6ccaad4fca57035c6c604d65c67186e1a6c
parent71666a84d1b1e32bf58b787f394c1dbcc8d1b86c
staging: iio: adis16060: Remove iio_dev mlock and refactor code

The IIO subsystem is redefining iio_dev->mlock to be used by
the IIO core only for protecting device operating mode changes.
ie. Changes between INDIO_DIRECT_MODE, INDIO_BUFFER_* modes.

In this driver, mlock was being used to protect hardware state changes.

In the driver, buf_lock protects both the adis16060_spi_write() and
adis16060_spi_read() functions and both are always called in
pair. First write, then read. Refactor the code to have
one single function adis16060_spi_write_than_read() protected by
the buf_lock.  This removes the need for additional locking via
mlock, so this locking is removed.

Signed-off-by: simran singhal <singhalsimran0@gmail.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/gyro/adis16060_core.c