linux.git/drivers/staging/iio, branch v5.13

staging: iio: cdc: ad7746: avoid overwrite of num_channels

2021-05-13T15:50:08+00:00

AD7745 devices don't have the CIN2 pins and therefore can't handle related
channels. Forcing the number of AD7746 channels may lead to enabling more
channels than what the hardware actually supports.
Avoid num_channels being overwritten after first assignment.

Signed-off-by: Lucas Stankus 
Fixes: 83e416f458d53 ("staging: iio: adc: Replace, rewrite ad7745 from scratch.")
Signed-off-by: Jonathan Cameron 
Cc:

iio:cdc:ad7150: Move driver out of staging.

2021-04-07T07:36:37+00:00

This capacitance to digital converter (CDC) driver is compliant with
the IIO ABI.  Note, not all features supported (e.g. window event modes)
but the driver should be in a useful functional state.

The cleanup was done against QEMU emulation of the device rather than
actual hardware.   Whilst this was a bit of an experiment, it made it
easy to confirm that the driver remained in a consistent working state
through the various refactors.  If it worked in the first place, it
should still be working after this cleanup.

Given some IIO drivers require expensive hardware setups, (not particularly
true with this one) the use of QEMU may provide a viable way forward
for providing testing during code changes where previously we'd had
to rely on sharp eyes and crossed fingers.

Note, no explicit MAINTAINERS entry as it will be covered by the
generic catch-alls for ADI and IIO drivers which are sufficient.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-25-jic23@kernel.org

staging:iio:cdc:ad7150: Add copyright notice given substantial changes.

2021-04-07T07:36:37+00:00

It seems to me that the changes made to get this ready to move out of
staging are substantial enough to warant a copyright notice addition.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-23-jic23@kernel.org

staging:iio:cdc:ad7150: Add of_match_table

2021-04-07T07:36:37+00:00

Rather than using the fallback path in the i2c subsystem and hoping
for no clashes across vendors, lets put in an explicit table for
matching.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Reviewed-by: Barry Song 
Link: https://lore.kernel.org/r/20210314181511.531414-21-jic23@kernel.org

staging:iio:cdc:ad7150: Really basic regulator support.

2021-04-07T07:36:36+00:00

Given DT docs will include regulators, lets just turn them on and
off with driver probe() and remove().

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-20-jic23@kernel.org

staging:iio:cdc:ad7150: Add scale and offset to info_mask_shared_by_type

2021-04-07T07:36:36+00:00

The datasheet provides these two values on the assumption they are applied
to unshift raw value.  Hence shift both the offset and scale by 4
to compensate.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-19-jic23@kernel.org

staging:iio:cdc:ad7150: Shift the _raw readings by 4 bits.

2021-04-07T07:36:36+00:00

Every other register related to raw value on the datasheet is
described as correpsonding to the 12MSB of the actual
data registers + the bottom 4 bits are 0.  So lets treat this
as what it actually is, which is a 12 bit value.
Note that we will have to be a little careful to compensate for
the offset and scale values.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-18-jic23@kernel.org

staging:iio:cdc:ad7150: Drop unnecessary block comments.

2021-04-07T07:36:36+00:00

These have a habit of not getting updated with driver reorganizations
and don't add much info so drop them.

Also fix a minor comment syntax issue.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Barry Song 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-17-jic23@kernel.org

staging:iio:cdc:ad7150: Tidy up local variable positioning.

2021-04-07T07:36:36+00:00

Where there is no other basis on which to order declarations
let us prefer reverse xmas tree.  Also reduce scope where
sensible.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Barry Song 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-16-jic23@kernel.org

staging:iio:cdc:ad7150: Reorganize headers.

2021-04-07T07:36:36+00:00

Whilst not important, it's nice to have the general headers in
alphabetical order.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Barry Song 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210314181511.531414-15-jic23@kernel.org