Skip to content

Really fix AD4630-20/AD4632-20 data capture#3250

Merged
nunojsa merged 1 commit intoanalogdevicesinc:mainfrom
machschmitt:main-ad4630-20-data-fix
Apr 14, 2026
Merged

Really fix AD4630-20/AD4632-20 data capture#3250
nunojsa merged 1 commit intoanalogdevicesinc:mainfrom
machschmitt:main-ad4630-20-data-fix

Conversation

@machschmitt
Copy link
Copy Markdown
Contributor

PR Description

It happens that #3241 missed updating the data width to 20-bit.
This time, I got confirmation from an application engineer that data capture runs as expected. Finally fix the data capture on main xD

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 compiled my changes, including the documentation
  • An ADI product applications engineer tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

Using a larger data width made SPI transfers run 4 additional bits which
caused the capture data to be shifted by 4 bits. Use the correct data width
for 20-bit precision ADCs, making SPI transfers run 20 SCLK pulses per
transfer which in turn will make the controller gather only the expected
data bits to the sample buffer.

Fixes: 299dc60 ("iio: adc: ad4630: Add support for AD4630-20 and AD4632-20")
Signed-off-by: Marcelo Schmitt <[email protected]>
@nunojsa nunojsa merged commit dfb6a9f into analogdevicesinc:main Apr 14, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants