linux.git/drivers/staging/iio, branch v6.0

staging: iio: ad2s1210: remove redundant assignment to variable negative

2022-04-28T18:22:56+00:00

Variable negative is being assigned a value that is never read, it is
being re-assigned later. The assignment is redundant and can be removed.

Cleans up clang scan build warning:
drivers/staging/iio/resolver/ad2s1210.c:502:3: warning: Value stored
to 'negative' is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King 
Link: https://lore.kernel.org/r/20220418134603.81336-1-colin.i.king@gmail.com
Signed-off-by: Jonathan Cameron

iio: Replace strtobool() with kstrtobool()

2022-04-28T18:22:54+00:00

strtobool() is deprecated and just a wrapper around kstrtobool().Replace
it with kstrtobool() so the deprecated function can be removed eventually.

Suggested-by: Andy Shevchenko 
Signed-off-by: Lars-Peter Clausen 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20220409105812.2113895-1-lars@metafoo.de
Signed-off-by: Jonathan Cameron

iio: core: Simplify the registration of kfifo buffers

2022-04-10T15:25:46+00:00

Among all the users of the kfifo buffers, no one uses the
INDIO_BUFFER_HARDWARE mode. So let's take this as a general rule and
simplify a little bit the internals - overall the documentation - by
eliminating unused specific cases. Use the INDIO_BUFFER_SOFTWARE mode by
default with kfifo buffers, which will basically mimic what all the "non
direct" modes do.

Cc: Benson Leung 
Cc: Guenter Roeck 
Cc: Jyoti Bhayana 
Cc: Jean-Baptiste Maneyrol 
Cc: Lorenzo Bianconi 
Cc: Michael Hennerich 
Cc: Greg Kroah-Hartman 
Signed-off-by: Miquel Raynal 
Link: https://lore.kernel.org/r/20220207143840.707510-13-miquel.raynal@bootlin.com
Signed-off-by: Jonathan Cameron

iio:adc:ad7280a: Move out of staging

2022-02-21T19:33:05+00:00

This is a rather unusual device (in IIO anyway).  However, it has
a near to standard userspace ABI.

Note the work to move this out of staging was done against a minimal
QEMU model, which doesn't model all the features of the device.
I have no intention to upstream the QEMU model as it was developed
just to enable this driver cleanup.

https://github.com/jic23/qemu/tree/ad7280a-hacks

Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-21-jic23@kernel.org

staging:iio:adc:ad7280a: Use more conservative delays to allow 105C operation.

2022-02-21T19:33:05+00:00

The datasheet provides timings for operating this device at up to 105
degrees centigrade. Adopt these more conservative timings.

Suggested-by: Marcelo Schmitt 
Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-20-jic23@kernel.org

staging:iio:adc:ad7280a: Remove shift from cb_mask state cache.

2022-02-21T19:33:05+00:00

Making the local storage of the Cell Balance mask a simple
bitmap and then shifting it only at time of register write simplifies
several code paths.

Suggested-by: Marcelo Schmitt 
Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-19-jic23@kernel.org

staging:iio:adc:ad7280a: Drop buggy support for early termination of AUX alert.

2022-02-21T19:33:05+00:00

This functionality is intended to allow for a few temperature
sensors to be missing (and hence not worth reading) on the final
device in a chain.  The ones removed are 3 and 5 (unlike for
the ADC channels where it is 4 and 5).

The datasheet includes a foot note 3 to Table 12 that makes this complex
to support.

"(3) To remove AUX5 or AUX5 and AUX3 from the alert detection, conversions
on three auxiliary ADC input channels only must be selected in the
control register."

This mode has never been supported by the driver.

As this support would be complex to add and the rework is being done
against a QEMU model developed for the purposes of verifying nothing
is broken, it is better to drop this support for now.

Reported-by: Marcelo Schmitt 
Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-16-jic23@kernel.org

staging:iio:adc:ad7280a: Use device properties to replace platform data.

2022-02-21T19:33:05+00:00

Convert all the device specific info that was previously in platform data
over to generic firmware query interfaces.

dt-bindings to follow shortly.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-15-jic23@kernel.org

staging:iio:adc:ad7280a: Use a local dev pointer to avoid &spi->dev

2022-02-21T19:33:05+00:00

We use this a few times already and it is about to get worse, so
introduce a local variable to simplify things.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-14-jic23@kernel.org

staging:iio:ad7280a: Reflect optionality of irq in ABI

2022-02-21T19:33:05+00:00

Given the irq is optional, let us remove the interfaces related to
events when it is not present.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Marcelo Schmitt 
Link: https://lore.kernel.org/r/20220206190328.333093-13-jic23@kernel.org