Add support for AD4052 ADC Family #2642

gastmaier · 2024-11-01T18:12:36Z

PR Description

Add:

driver support for AD4052/AD4058/AD4050/AD4056 (16-bit/12-bit SAR ADC SPI)
devicetree binding

Optional:

sample devicetree for coraz7s
kconfig imply driver

Features:

Single shot using gpio as ADC CNV
Buffer readings using AXI PWM ADC CNV and ADC GP1 Data Ready as AXI SPI Engine Offload Trigger
Monitor mode with ADC GP0 as interrupt trigger (either IIO threshold direction event)
- User should do register access to get event details.
Functional modes via oversampling config: Sample Mode (0/1) or Burst Averaging Mode (>1)
- Burst Avg enables configuring averaging filter length value (AVG_CONFIG.AVG_WIN_LEN)
DEVICE_CONFIG.POWER_MODE : put device on sleep at pm_runtime ops.
TIMER_CONFIG.FS_BURST_AUTO : configure burst avg enable and trigger mode sample rates via channel sampling frequency config.

Overall, the highlight of this device is the monitor capability, leveraged by the autonomous trigger mode and exposed as IIO Event.

Linux driver doc:

HDL: analogdevicesinc/hdl#1504
Datasheet: www.analog.com/ad4052
Tested on Coraz7s

PR Type

Bug fix (a change that fixes an issue)
New feature (a change that adds new functionality)
Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

I have conducted a self-review of my own code changes
I have tested the changes on the relevant hardware
I have updated the documentation outside this repo accordingly (if there is the case)

gastmaier · 2024-11-04T16:40:27Z

Forced pushed to:

Add support to TIMER_CONFIG.FS_BURST_AUTO
Only write GP during probe.
Simplify IRQ Handler, leave to the user to clear the irq, because:
- Entering config mode causes the irq to unset.
- Re-entering monitor mode sets the irq back if still outside the threshold range, causing new handler calls.
- "Ignoring next interrupt" would generate obnoxious logic.

Forced pushed (2) to:

Invert fs_burst_auto array entries, default all devices to 2MSPS (Yes, this value is -EINVAL for ad4056/ad4058, but is the datasheet reset value)

Forced pushed (3) to:

Clean-up unused enums.
Rename ad405x to ad4052 in the dts

nunojsa

Here it goes first round of review...

I think this one can be upstreamed without the offload bits

Documentation/devicetree/bindings/iio/adc/adi,ad4052.yaml

drivers/iio/adc/ad4052.c

gastmaier · 2024-11-12T00:15:30Z

Change log v0 -> v1

Review changes:

Move functional modes from devicetree entry to different OVERSAMPLING_RATIO values, affecting RAW read and buffered.
Documentation YAML fixes, removal of functional mode.
Header on alphabetical order.
Use field_prep
Remove forward declaration, reorder methods instead.
Fix cache line aligment.
Use devm runtime enable, remove manual remove.
Use dev_err_probe
General Documentation/.yaml fixes
Make all regmap_bulk dma safe.
Remove/simplify switch cases where not needed.
Instead of using of_node, use spi->irq directly
Use regmap update bits.
Use fsleep
Use oversampling iio property (value 0 is invalid, 1 put on Sample Mode, > 1 Burst Averaging mode)

New:

Add regmap access tables

Design changes:

Under IIO Event thrshold enabled (device Monitor Mode), return device access busy for every other access, simplifying logic by not requiring exiting and re-entering mode and having an obscure "monitoring downtime" during access.
Use oversampling iio prop instead of devicetree entry to switch functional modes.

Documentation/devicetree/bindings/iio/adc/adi,ad4052.yaml

nunojsa · 2024-11-12T14:14:12Z

drivers/iio/adc/Kconfig

+	tristate "Analog Devices AD4052 Driver"
+	depends on SPI
+	depends on PWM
+	depends on GPIOLIB


I'm also not sure if gpio is something you should depend on or just select it. I think the single shot reading could be something useful to have even more to make this driver more upstreamable without offload support (we can still some questions about it though).

nunojsa · 2024-11-12T14:14:47Z

drivers/iio/adc/Kconfig

+	tristate "Analog Devices AD4052 Driver"
+	depends on SPI
+	depends on PWM
+	depends on GPIOLIB


Also one not on the commit message. Style is iio: adc: .... Drop the drivers:

drivers/iio/adc/ad4052.c

nunojsa · 2024-11-12T15:04:53Z

drivers/iio/adc/ad4052.c

+	if (val == 0) {
+		st->mode = AD4052_SAMPLE_MODE;
+	} else {
+		st->mode = AD4052_BURST_AVERAGING_MODE;


I would double check other examples of oversampling. To me this is a 1 or 0 thing. Either it's enabled it not. Your mode variable also just seems to have two states here so the way you're handling val raises some questions.

The device supports from 2 to 4096 in powers of 2 samples for avg mode.
Decided to reserve 0 and 1 to disable the feature (return to sample mode)

gastmaier · 2024-11-17T22:16:46Z

Change log v1 -> v2

Review changes:

Include all peripheral properties
Drop cnv-gpio description
Updated dt-binding commit description
Replace d32 array by d16/d32 union (requires ignoring misleading checkpatch warning).
Remove MODULE_IMPORT_NS(IIO_AD4052);
Add error handling where missing
remove {} in single statement ifs, coherent style
Use multiple scan masks for changing precision due to oversampling
Move all if (st->mode == AD4052_MONITOR_MODE) inside the IIO mutex claim.
Drop switches where there is only one channel/event type
Remove unused AD4052_AVERAGING_MODE (would require a timed trigger) and AD4052_CONFIG_MODE (implicit mode outside any reading)
ad4052_set_avg_filter pass st directly.
Use different variables instead of the array for the chip_info

Changes:

Use GPIO interrupt instead of PS IRQ
All GPIOs are optional, does not require GPIO on KConfig. The user shall bring his own external trigger or short the CS and CNV, for example.
Move PWM sampling rate to IIO_ATTR
Move ext fs_burst_auto to channel sampling freq
Use Data Ready assertions for single shot readings, timeout after 1 second

Extra force push to resolve merge conflict

gastmaier · 2024-11-27T17:19:53Z

Change log v2 -> v3

Add "0" (disabled) to oversampling avail list
Update iic controller dts address

nunojsa · 2024-11-29T13:28:55Z

Change log v2 -> v3

This is not forgotten... I'll try to check it Monday

gastmaier · 2024-12-03T17:23:23Z

Force pushed to resolve merge conflicts due to main rebase.
(minor drivers/iio/adc/Kconfig drivers/iio/adc/Makefile entries )
Then checked with git diff public/main..@

gastmaier · 2024-12-16T15:25:34Z

@nunojsa is this version good enough to consider finished?
Should we try to upstream without the offload+pwm parts?
I have a similar driver for another family that uses triggered buffer, but the data rate is much slower + no adjustable buffer sample rate.

nunojsa · 2024-12-17T12:44:19Z

@gastmaier,

Sorry for the slow approach on this one...

Should we try to upstream without the offload+pwm parts?

Definitely... But I would even say to maybe wait a few more weeks. We can be close to land spi offload support upstream.

is this version good enough to consider finished?

I'll take a look. Note that CI is failing - maybe due to conflicts with 6.6 merge

Documentation/devicetree/bindings/iio/adc/adi,ad4052.yaml

drivers/iio/adc/ad4052.c

nunojsa · 2024-12-17T13:04:44Z

drivers/iio/adc/ad4052.c

+	/* Single sample read should be used only for oversampling and
+	 * sampling frequency pairs that take less than 1 sec.
+	 */
+	if (st->gp1_irq.enabled) {


can we do raw reads without the IRQ?

Without oversampling enabled the raw read may be slower than the gpio CNV to spi transfer,
allowing to do raw reads without the IRQ.
But I will remove this option/use case.
Note: CNV and GP0 (threshold) are still optional.
Note 2: Since CNV is optional, if cnv not present I will yield an empty spi transfer to trigger a conversion (considering the user had shorten CNV to CS)

nunojsa · 2024-12-17T13:05:38Z

drivers/iio/adc/ad4052.c

+		return PTR_ERR(gpio);
+
+	if (gpio) {
+		irq = gpiod_to_irq(gpio);


Directly use devm_request_threaded_irq()... i see one the IRQs is for completion but what's the purpose of the other one? We need some comments here I guess

One is for the threshold IIO event (ad4052_irq_handler_thresh).
The other is drdy for raw read (completion).

I didn't get the comment to directly use devm_request_threaded_irq.
Do you want me to change from a soft interrupt from the GPIO (like ad9081,c) to a hard interrupt (like ad7280a.c)?

nunojsa · 2024-12-17T13:08:00Z

drivers/iio/adc/ad4052.c

+	} else {
+		val = ilog2(val);
+		st->mode = AD4052_BURST_AVERAGING_MODE;
+		ret = regmap_write(st->regmap, AD4052_REG_AVG_CONFIG, val - 1);


We should still make sure val is valid right?

It already bounds the value at AD4052_CHECK_OVERSAMPLING.

gastmaier · 2024-12-17T22:09:42Z

V2 -> V3

Rebase
Make gp1-gpios required
If the user shorten CS with CNV and do not provide cnv-gpio, do empty spi transfer to trigger cnv on raw read.
Fix hackish devm_irq_free, disable/enable instead
Invert order in postdisable for pwm and irq_enable (related to last bulletpoint)
Tune max fsclk frequency

git range-diff 85e9f9a4dc7f85804292d1a0a67c60967282ae53~..204d667060e1f805ce8be96e13bc438d67193b46 f09a86296a9367306b1c653e54e1cb37295923ea~..10a85eb73314419f7a5836ae064f51fa097f6101

Tested on HW at fsclk 31.25MHz (could not achieve datasheet 62.5MHz).

TODO: Solve buffer/sample_freq hack . Wait upstream offload?

gastmaier · 2025-02-10T09:56:47Z

V4:

Use axi-pwmgen backport PWM waveform, axi-pwmgen backport #2640
Rebase

In practice:

git checkout -b main public/main
# Pick ad4052-pwm with #2640 backport
git cherry-pick 752ba29b6c6a7bd5d49960e2cc9f7ff20aa82c17~..8c423d9fe643ac41e5b4f0f7180764bd8ffdcd42
# ad4052-pwm..V4
git range-diff 752ba29b6c6a7bd5d49960e2cc9f7ff20aa82c17~..8c423d9fe643ac41e5b4f0f7180764bd8ffdcd42 841115c3aa35dd0a0e30edf6ccc7c66e7dbc0ec6~..02e0f9d12314d289eaffb80f666c621417c355df
# V3..V4
git range-diff f09a86296a9367306b1c653e54e1cb37295923ea~..10a85eb73314419f7a5836ae064f51fa097f6101 841115c3aa35dd0a0e30edf6ccc7c66e7dbc0ec6~..02e0f9d12314d289eaffb80f666c621417c355df

Sorry for forgetting to track #2640, this could have been done earlier.

Note: waiting hw test to push V5, that resolves the hackish:

This is hackish... I would likely just add the sampling frequency as a device attr as typical. Otherwise, we would need to extend devm_iio_dmaengine_buffer_setup() to get a **buffer_attrs parameter.

By using samp_freq for the pwm freq (like pulsar.c), and moving the sample_rate device register to an ext_info attribute.

gastmaier · 2025-02-17T11:27:31Z

V5

Resolve buffer attribute "hack" by setting:
- IIO_SAMP_FREQ for pwm frequency
- IIO_ENUM for device internal sample rate
License date update
Set irq return methods as static
gpiod_set_value->gpiod_set_value_cansleep
spi_sync_transfer->spi_sync

V4..V5

git diff 6eb21ea97b0b8cf65b2e5f4ecfe50b32ca28ff71..7c5b1047280e9780c1d038d555421905a195b325

Wait V6 due to #2713?

nunojsa · 2025-02-19T11:09:54Z

Wait V6 due to #2713?

This was merged... Sorry for missing this one for a long time. Ping me when you have a new version for review

Add dt-bindings for AD4052 family, devices AD4050/AD4052/AD4056/AD4058, low-power with monitor capabilities SAR ADCs. Contain selectable oversampling and sample rate, the latter for both oversampling and monitor mode. The monitor capability is exposed as an IIO threshold either direction event. Signed-off-by: Jorge Marques <[email protected]>

The AD4052 CNV pin is driven by a GPIO for single shot readings and by a PWM for buffer readings. The functional-mode entry allows to set Sample Mode (0) or Burst Averaging Mode (1). During runtime, it is possible to enter Trigger Mode through IIO Events. Signed-off-by: Jorge Marques <[email protected]>

This adds a new page to document how to use the ad4052 ADC driver. Signed-off-by: Jorge Marques <[email protected]>

gastmaier · 2025-02-21T20:27:26Z

V6

Use back ported offload support from Spi offload backport #2713 with offload being a optional feature.
Add docs page

I decided to not fallback to a triggered buffer to not have to mux the RDY signal for triggered buffer and raw read.

# V5..V6
git range-diff 841115c3aa35dd0a0e30edf6ccc7c66e7dbc0ec6~..bc5f2b396e08d6f7315bd4b4d6132d33b5a2aaf3 ae94ce3bf80e03c29c9a04b5246a601477de7684~..a4295b79b52a52c8e77b043eedef133321daa82d

nunojsa · 2025-02-24T10:10:50Z

I'll wait until you fix the build issues in CI

The AD4052/AD4058/AD4050/AD4056 are versatile, 16-bit/12-bit, successive approximation register (SAR) analog-to-digital converter (ADC) that enables low-power, high-density data acquisition solutions without sacrificing precision. This ADC offers a unique balance of performance and power efficiency, plus innovative features for seamlessly switching between high-resolution and low-power modes tailored to the immediate needs of the system. The AD4052/AD4058/AD4050/AD4056 are ideal for battery-powered, compact data acquisition and edge sensing applications. Signed-off-by: Jorge Marques <[email protected]>

Add entry for the AD4052 ADC family driver. Signed-off-by: Jorge Marques <[email protected]>

gastmaier · 2025-02-24T12:17:37Z

@nunojsa Sorry, fixed the warnings.
Bindings Check Script is failing because of the bumped setuptools version required by dt-shema 2025.02
https://github.com/devicetree-org/dt-schema/blob/v2025.02/pyproject.toml#L2

pkg_resources.VersionConflict: (setuptools 59.6.0 (/usr/lib/python3/dist-packages), Requirement.parse('setuptools>=61'))

gastmaier requested review from nunojsa, mhennerich, dbogdan, btogorean, chegbeli, buha, amiclaus, rbolboac, danmois, CiprianRegus and machschmitt as code owners November 1, 2024 18:12

gastmaier force-pushed the staging/ad4052 branch 3 times, most recently from ce15fe8 to 86e6d7c Compare November 4, 2024 16:33

gastmaier force-pushed the staging/ad4052 branch 3 times, most recently from 1f5b827 to 9611c42 Compare November 6, 2024 19:03

nunojsa reviewed Nov 8, 2024

View reviewed changes

This comment was marked as duplicate.

Sign in to view

gastmaier force-pushed the staging/ad4052 branch 2 times, most recently from 4b81da9 to 494d89f Compare November 12, 2024 00:15

nunojsa reviewed Nov 12, 2024

View reviewed changes

gastmaier force-pushed the staging/ad4052 branch from 494d89f to 03cfbc9 Compare November 17, 2024 22:16

gastmaier force-pushed the staging/ad4052 branch from 03cfbc9 to a7eca14 Compare November 17, 2024 22:24

gastmaier requested a review from nunojsa November 25, 2024 13:20

gastmaier force-pushed the staging/ad4052 branch 2 times, most recently from eb23140 to 0506d4c Compare November 27, 2024 17:18

gastmaier force-pushed the staging/ad4052 branch from 0506d4c to b6ab861 Compare November 27, 2024 17:24

gastmaier force-pushed the staging/ad4052 branch from b6ab861 to 4b0f9d7 Compare December 3, 2024 17:22

gastmaier force-pushed the staging/ad4052 branch from 4b0f9d7 to 204d667 Compare December 4, 2024 00:29

nunojsa reviewed Dec 17, 2024

View reviewed changes

gastmaier force-pushed the staging/ad4052 branch from 204d667 to 10a85eb Compare December 17, 2024 22:09

gastmaier force-pushed the staging/ad4052 branch from 10a85eb to 02e0f9d Compare February 10, 2025 09:56

gastmaier force-pushed the staging/ad4052 branch from 02e0f9d to bc5f2b3 Compare February 17, 2025 11:27

gastmaier force-pushed the staging/ad4052 branch from bc5f2b3 to f41923e Compare February 21, 2025 20:13

gastmaier added 2 commits February 21, 2025 21:23

docs: iio: new docs for ad4052 driver

df8c108

This adds a new page to document how to use the ad4052 ADC driver. Signed-off-by: Jorge Marques <[email protected]>

gastmaier force-pushed the staging/ad4052 branch from f41923e to a4295b7 Compare February 21, 2025 20:24

gastmaier requested a review from nunojsa February 24, 2025 07:51

gastmaier force-pushed the staging/ad4052 branch from a4295b7 to a824f15 Compare February 24, 2025 10:41

gastmaier added 2 commits February 24, 2025 11:42

Kconfig.adi: imply AD4052

ce4ae13

Add entry for the AD4052 ADC family driver. Signed-off-by: Jorge Marques <[email protected]>

gastmaier force-pushed the staging/ad4052 branch from a824f15 to ce4ae13 Compare February 24, 2025 10:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add support for AD4052 ADC Family #2642

Add support for AD4052 ADC Family #2642

gastmaier commented Nov 1, 2024 •

edited

Loading

gastmaier commented Nov 4, 2024 •

edited

Loading

nunojsa left a comment

This comment was marked as duplicate.

gastmaier commented Nov 12, 2024 •

edited

Loading

nunojsa Nov 12, 2024

nunojsa Nov 12, 2024

nunojsa Nov 12, 2024

gastmaier Nov 17, 2024

gastmaier commented Nov 17, 2024 •

edited

Loading

gastmaier commented Nov 27, 2024

nunojsa commented Nov 29, 2024

gastmaier commented Dec 3, 2024

gastmaier commented Dec 16, 2024 •

edited

Loading

nunojsa commented Dec 17, 2024

nunojsa Dec 17, 2024

gastmaier Dec 17, 2024

nunojsa Dec 17, 2024

gastmaier Dec 17, 2024

nunojsa Dec 17, 2024

gastmaier Dec 17, 2024

gastmaier commented Dec 17, 2024 •

edited

Loading

gastmaier commented Feb 10, 2025 •

edited

Loading

gastmaier commented Feb 17, 2025 •

edited

Loading

nunojsa commented Feb 19, 2025

gastmaier commented Feb 21, 2025

nunojsa commented Feb 24, 2025

gastmaier commented Feb 24, 2025

Add support for AD4052 ADC Family #2642

Are you sure you want to change the base?

Add support for AD4052 ADC Family #2642

Conversation

gastmaier commented Nov 1, 2024 • edited Loading

PR Description

PR Type

PR Checklist

gastmaier commented Nov 4, 2024 • edited Loading

nunojsa left a comment

Choose a reason for hiding this comment

This comment was marked as duplicate.

gastmaier commented Nov 12, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gastmaier commented Nov 17, 2024 • edited Loading

gastmaier commented Nov 27, 2024

nunojsa commented Nov 29, 2024

gastmaier commented Dec 3, 2024

gastmaier commented Dec 16, 2024 • edited Loading

nunojsa commented Dec 17, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gastmaier commented Dec 17, 2024 • edited Loading

gastmaier commented Feb 10, 2025 • edited Loading

gastmaier commented Feb 17, 2025 • edited Loading

nunojsa commented Feb 19, 2025

gastmaier commented Feb 21, 2025

nunojsa commented Feb 24, 2025

gastmaier commented Feb 24, 2025

gastmaier commented Nov 1, 2024 •

edited

Loading

gastmaier commented Nov 4, 2024 •

edited

Loading

gastmaier commented Nov 12, 2024 •

edited

Loading

gastmaier commented Nov 17, 2024 •

edited

Loading

gastmaier commented Dec 16, 2024 •

edited

Loading

gastmaier commented Dec 17, 2024 •

edited

Loading

gastmaier commented Feb 10, 2025 •

edited

Loading

gastmaier commented Feb 17, 2025 •

edited

Loading