linux-toradex.git/drivers/iio, branch v3.13-rc1

kfifo API type safety

2013-11-15T00:32:23+00:00

This patch enhances the type safety for the kfifo API.  It is now safe
to put const data into a non const FIFO and the API will now generate a
compiler warning when reading from the fifo where the destination
address is pointing to a const variable.

As a side effect the kfifo_put() does now expect the value of an element
instead a pointer to the element.  This was suggested Russell King.  It
make the handling of the kfifo_put easier since there is no need to
create a helper variable for getting the address of a pointer or to pass
integers of different sizes.

IMHO the API break is okay, since there are currently only six users of
kfifo_put().

The code is also cleaner by kicking out the "if (0)" expressions.

[akpm@linux-foundation.org: coding-style fixes]
Signed-off-by: Stefani Seibold 
Cc: Russell King 
Cc: Hauke Mehrtens 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

tree-wide: use reinit_completion instead of INIT_COMPLETION

2013-11-15T00:32:21+00:00

Use this new function to make code more comprehensible, since we are
reinitialzing the completion, not initializing.

[akpm@linux-foundation.org: linux-next resyncs]
Signed-off-by: Wolfram Sang 
Acked-by: Linus Walleij  (personally at LCE13)
Cc: Ingo Molnar 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

iio: light: vcnl4000: Remove redundant code

2013-10-24T13:48:14+00:00

The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Hence return the function directly.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: dac: mcp4725: Remove redundant code

2013-10-24T13:47:49+00:00

Remove an inconsequential print message and return directly
thereby cleaning up some code.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: dac: max517: Remove redundant variable

2013-10-24T13:47:15+00:00

Remove an inconsequential print message and return directly
thereby eliminating an intermediate variable.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: dac: ad5755: Remove redundant code

2013-10-24T13:46:33+00:00

The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Error messages are already printed by iio_device_register();
hence not needed.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: dac: ad5421: Remove redundant code

2013-10-24T13:46:00+00:00

The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Hence return the function directly. Error messages are already
printed by iio_device_register(); hence not needed.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: adc: twl6030-gpadc: Remove redundant code

2013-10-24T13:45:09+00:00

The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Hence return the function directly.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: accel: kxsd9: Remove redundant variable

2013-10-24T13:43:24+00:00

Return directly thereby eliminating an intermediate variable.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron

iio: core: Add misssing braces

2013-10-24T13:42:44+00:00

Silences the following checkpatch warning:
WARNING: sizeof *iio_attr should be sizeof(*iio_attr)

Signed-off-by: Sachin Kamat 
Signed-off-by: Jonathan Cameron