linux-stable.git/drivers/staging/iio, branch linux-4.12.y

staging:iio:resolver:ad2s1210 fix negative IIO_ANGL_VEL read

2017-08-16T20:46:54+00:00

commit 105967ad68d2eb1a041bc041f9cf96af2a653b65 upstream.

gcc-7 points out an older regression:

drivers/staging/iio/resolver/ad2s1210.c: In function 'ad2s1210_read_raw':
drivers/staging/iio/resolver/ad2s1210.c:515:42: error: '<<' in boolean context, did you mean '<' ? [-Werror=int-in-bool-context]

The original code had 'unsigned short' here, but incorrectly got
converted to 'bool'. This reverts the regression and uses a normal
type instead.

Fixes: 29148543c521 ("staging:iio:resolver:ad2s1210 minimal chan spec conversion.")
Signed-off-by: Arnd Bergmann 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Greg Kroah-Hartman

staging: iio: ad7152: Fix deadlock in ad7152_write_raw_samp_freq()

2017-05-28T14:45:46+00:00

ad7152_write_raw_samp_freq() is called by ad7152_write_raw() with
chip->state_lock held. So, there is unavoidable deadlock when
ad7152_write_raw_samp_freq() locks the mutex itself.

The patch removes unneeded locking.

Found by Linux Driver Verification project (linuxtesting.org).

Signed-off-by: Alexey Khoroshilov 
Fixes: 6572389bcc11 ("staging: iio: cdc: ad7152: Implement
IIO_CHAN_INFO_SAMP_FREQ attribute")
Acked-by: Lars-Peter Clausen 
Cc: 
Signed-off-by: Jonathan Cameron

Staging: iio: accel: adis16203: fixed a brace coding style issue

2017-04-14T15:13:34+00:00

Fixed a brace coding style issue.

Signed-off-by: René Hickersberger 
Signed-off-by: Jonathan Cameron

iio:meter:ade7759: Removing use of deprecated macros (S_IRUGO, S_IWUSR)

2017-04-14T14:09:07+00:00

This fixes the coding style issue of using (S_IWUSR | S_IRUGO) in place of
4-digit octal numbers.

Signed-off-by: Chen Guanqiao 
Signed-off-by: Jonathan Cameron

staging: iio: ad5933: Protect DIRECT mode using claim/release helpers

2017-04-14T14:09:06+00:00

This device operates in DIRECT_MODE and BUFFER_HARDWARE mode.
Replace usages of iio_dev->mlock with iio_device_{claim|release}_direct_mode()
helper functions to guarantee DIRECT mode and consequently protect
BUFFER mode too.

Add and use a device private lock to protect against conflicting access of the
state data.
This helps with IIO subsystem 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.

Protect changing of attributes inside ad5933_store(). Attributes
can no longer be changed while in buffered mode.

Remove lock from ad5933_work() because buffer mode should be enabled
when we reach this, and claiming DIRECT mode in all the other places
should protect it.

Signed-off-by: Narcisa Ana Maria Vasile 
Signed-off-by: Jonathan Cameron

iio:meter:ade7759: Removing use of deprecated macros (S_IRUGO, S_IWUSR, S_IXUGO)

2017-04-14T14:09:05+00:00

Removing use of deprecated macros(S_IRUGO, S_IWUSR, S_IXUGO), and replace
with 4 digit octal.

Signed-off-by: Chen Guanqiao 
Signed-off-by: Jonathan Cameron

staging: iio: ad9832: use 4-digit octal permissions

2017-04-02T09:20:48+00:00

This fixes the coding style issue of using S_IWUSR in place of 4-digit
octal numbers.

Issue detected by checkpatch.

Signed-off-by: Guru Das Srinagesh 
Signed-off-by: Jonathan Cameron

iio: gyro: adis16060: Change the name of function.

2017-04-02T08:10:32+00:00

Change the name of function from adis16060_spi_write_than_read()
to adis16060_spi_write_then_read(). change "than" to "then" as
its time depended.

Signed-off-by: simran singhal 
Signed-off-by: Jonathan Cameron

staging: iio: light: tsl2x7x constify attribute_group structures

2017-04-01T10:40:36+00:00

As the event_attrs field of iio_info structures is constant, so these
attribute_group structures can also be declared constant.

File size before:
   text	   data	    bss	    dec	    hex	filename
  15064	   1528	      0	  16592	   40d0
drivers/staging/iio/light/tsl2x7x_core.o

File size after:
   text	   data	    bss	    dec	    hex	filename
  15192	   1400	      0	  16592	   40d0
drivers/staging/iio/light/tsl2x7x_core.o

Signed-off-by: simran singhal 
Signed-off-by: Jonathan Cameron

staging: iio: update locking method during frequency writes

2017-03-30T18:20:38+00:00

The driver needs to insure atomicity during frequency
changes of bus and device. The iiodev->mlock as used
was not doing that. Replace it with the drivers existing
buffer lock and introduce an auxiliary spi_write() that does
not hold the lock.

Signed-off-by: Gargi Sharma 
Signed-off-by: Jonathan Cameron