linux.git/drivers/staging/iio/resolver, branch v5.1

staging:iio:ad2s90: Move out of staging

2018-12-01T15:40:39+00:00

Move ad2s90 resolver driver out of staging to the main tree.

Signed-off-by: Matheus Tavares 
Signed-off-by: Victor Colombo 
Acked-by: Alexandru Ardelean 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Add comment to device state mutex

2018-12-01T15:40:22+00:00

Fix the checkpatch.pl issue:
"CHECK: struct mutex definition without comment".

Signed-off-by: Victor Colombo 
Signed-off-by: Matheus Tavares 
Acked-by: Alexandru Ardelean 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Replace license text w/ SPDX identifier

2018-12-01T15:38:01+00:00

This patch removes the license boilerplate text at the top of ad2s90.c
and, instead, adds the SPDX GPL-2.0 license identifier, which solves the
checkpatch.pl warning:
"WARNING: Missing or malformed SPDX-License-Identifier tag in line 1".

Signed-off-by: Matheus Tavares 
Acked-by: Alexandru Ardelean 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Add max frequency check at probe

2018-12-01T15:35:33+00:00

This patch adds a max frequency check at the beginning of ad2s90_probe
function so that when it is set to a value above 0.83Mhz, dev_err is
called with an appropriate message and -EINVAL is returned.

The defined limit is 0.83Mhz instead of 2Mhz, which is the chip's max
frequency as specified in the datasheet, because, as also specified in
the datasheet, a 600ns delay is expected between the application of a
logic LO to CS and the application of SCLK. Since the delay is not
implemented in the spi code, to satisfy it, SCLK's period should be at
most 2 * 600ns, so the max frequency should be 1 / (2 * 6e-7), which
gives roughly 830000Hz.

Signed-off-by: Alexandru Ardelean 
Signed-off-by: Matheus Tavares 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Remove spi setup that should be done via dt

2018-12-01T15:35:25+00:00

The ad2s90 driver currently sets some spi settings (max_speed_hz and
mode) at ad2s90_probe. Since the maximum frequency is a required element
in DT binding for spi slave devices and because the spi mode for the
device can be either (0,0) or (1,1), these settings should be handled
via device tree, not in the driver's code. This patch removes them from
the probe function.

Note: The way in which the mentioned spi settings need to be specified
on the ad2s90's node of a device tree will be documented in the future
patch "dt-bindings:iio:resolver: Add docs for ad2s90".

Signed-off-by: Matheus Tavares 
Acked-by: Alexandru Ardelean 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Add device tree support

2018-12-01T15:35:18+00:00

This patch adds device tree support to ad2s90 with standard
device tree id table.

Signed-off-by: Matheus Tavares 
Acked-by: Alexandru Ardelean 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Check channel type at read_raw

2018-11-11T15:29:49+00:00

This patch adds a channel type check at the beginning of the
ad2s90_read_raw function. Since ad2s90 has only one channel, it just
checks if the given channel is the expected one and if not, return
-EINVAL.

Signed-off-by: Matheus Tavares 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Add IIO_CHAN_INFO_SCALE to channel spec and read_raw

2018-11-11T15:29:48+00:00

This patch adds the IIO_CHAN_INFO_SCALE mask to ad2s90_chan and
implements the relative read behavior at ad2s90_read_raw.

Signed-off-by: Victor Colombo 
Signed-off-by: Matheus Tavares 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Move device registration to the end of probe

2018-11-11T15:29:48+00:00

Previously, devm_iio_device_register was being called before the
spi_setup call and the spi_device's max_speed_hz and mode assignments.
This could lead to a race condition since the driver was still being
set up after it was already made ready to use. To fix it, this patch
moves the device registration to the end of ad2s90_probe.

Signed-off-by: Matheus Tavares 
Signed-off-by: Jonathan Cameron

staging:iio:ad2s90: Remove always overwritten assignment

2018-11-11T15:29:48+00:00

This patch removes an initial assignment to the variable ret at probe,
that was always overwritten.

Signed-off-by: Matheus Tavares 
Signed-off-by: Jonathan Cameron