mbox series

[v2,0/5] iio: adc: ad7606: add support for AD760{7,8,9} parts

Message ID 20241025095939.271811-1-aardelean@baylibre.com (mailing list archive)
Headers show
Series iio: adc: ad7606: add support for AD760{7,8,9} parts | expand

Message

Alexandru Ardelean Oct. 25, 2024, 9:59 a.m. UTC
This change-set adds support for AD7607, AD7608 and AD7609.
These parts are simpler parts for the AD7606x family. They support only
HW-only mode like AD7605, but they support oversampling like the other
AD7606x parts.

AD7607 has a 14-bit resolution.
AD7608 and AD7609 are 18-bit resolution.
AD7607 & AD7608 supports +/-5V and +/-10V ranges.
AD7609 supports +/-10V & +/-20V ranges.

The oversampling settings are the same.
Because of AD7607, the scales had to be reworked (again), but this time
doing away with the allocation at runtime for the scale-available-show
values. This time, the full IIO_VAL_INT_PLUS_MICRO values are stored
statically. AD7607 supports a scale of 1.220703, which is the first and
only (so far) scale that is above 1.

Changelog v1 -> v2:
* https://lore.kernel.org/linux-iio/20241021130221.1469099-1-aardelean@baylibre.com/
* Dropped patch 'iio: adc: ad7606: fix issue/quirk with find_closest() for oversampling'
  - Will focus on fixing 'find_closest()' instead
* Patch 'iio: adc: ad7606: use realbits for sign-extending in scan_direct'
  - Applied David's suggestions to check only for > 16 bits
* Patch 'iio: adc: ad7606: rework scale-available to be static'
  - Applied David's sugestions
    - Rename variable to 'scale_avail_uv'
    - Use 'MICRO' definition for '1000000'
* Patch ' dt-bindings: iio: adc: adi,ad7606: document AD760{7,8,9} parts'
  - Added Connor's Acked-by tag

Alexandru Ardelean (5):
  iio: adc: ad7606: fix/persist oversampling_ratio setting
  iio: adc: ad7606: use realbits for sign-extending in scan_direct
  iio: adc: ad7606: rework scale-available to be static
  dt-bindings: iio: adc: adi,ad7606: document AD760{7,8,9} parts
  iio: adc: ad7606: add support for AD760{7,8,9} parts

 .../bindings/iio/adc/adi,ad7606.yaml          |   9 +
 drivers/iio/adc/ad7606.c                      | 221 +++++++++++++-----
 drivers/iio/adc/ad7606.h                      |   9 +-
 drivers/iio/adc/ad7606_par.c                  |   6 +
 drivers/iio/adc/ad7606_spi.c                  |  42 ++++
 5 files changed, 220 insertions(+), 67 deletions(-)

Comments

David Lechner Oct. 25, 2024, 3:13 p.m. UTC | #1
On 10/25/24 4:59 AM, Alexandru Ardelean wrote:
> This change-set adds support for AD7607, AD7608 and AD7609.
> These parts are simpler parts for the AD7606x family. They support only
> HW-only mode like AD7605, but they support oversampling like the other
> AD7606x parts.
> 
Reviewed-by: David Lechner <dlechner@baylibre.com>
Jonathan Cameron Oct. 26, 2024, 10:46 a.m. UTC | #2
On Fri, 25 Oct 2024 10:13:23 -0500
David Lechner <dlechner@baylibre.com> wrote:

> On 10/25/24 4:59 AM, Alexandru Ardelean wrote:
> > This change-set adds support for AD7607, AD7608 and AD7609.
> > These parts are simpler parts for the AD7606x family. They support only
> > HW-only mode like AD7605, but they support oversampling like the other
> > AD7606x parts.
> >   
> Reviewed-by: David Lechner <dlechner@baylibre.com>
> 

Applied to the togreg branch of iio.git and pushed out as testing initially for
all the normal reasons.

Thanks,

Jonathan