E523 drivers: adc: add support for ad4692 by RaduSabau1 · Pull Request #2913 · analogdevicesinc/no-OS · GitHub
[go: up one dir, main page]

Skip to content

drivers: adc: add support for ad4692#2913

Open
RaduSabau1 wants to merge 5 commits intomainfrom
staging/ad4692
Open

drivers: adc: add support for ad4692#2913
RaduSabau1 wants to merge 5 commits intomainfrom
staging/ad4692

Conversation

@RaduSabau1
Copy link
Collaborator

Pull Request Description

The AD4691/AD4692 are compact, high accuracy, 16-channel, 16-
bit successive approximation register (SAR) analog-to-digital con-
verters (ADCs) optimized for high-density multichannel precision
data acquisition solutions. The AD4691/AD4692 combine precision
performance with Easy Drive features and flexible digital processing
to ensure compatibility with space-constrained analog front-end
(AFE) designs and low-power digital hosts.

This PR contains commits for :

  • ad4692 no-OS driver
  • ad4692 no-OS IIO driver
  • ad4692 no-OS driver documentation
  • ad4692_ardz no-OS project for EVAL-AD4692-ARDZ board
  • ad4692_ardz no-OS project documentation

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

dbogdan
dbogdan previously approved these changes Mar 2, 2026
Copy link
Contributor
@amiclaus amiclaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some comments on my side.

}

init_param->conv_param->period_ns = MICRO;
case AD4692_CNV_CLOCK: /* fallthrough */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's no fallthrough here. only abofer.

if (ret)
pr_info("Error!\n");
return ret;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: new line at eof

static int ad4692_iio_update_channels(void *desc, uint32_t mask)
{
struct ad4692_iio_desc *iio_ad4692;
uint32_t reg_val;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: not used.

@RaduSabau1 RaduSabau1 marked this pull request as draft March 3, 2026 12:06
@RaduSabau1
Copy link
Collaborator Author

These drivers have to be updated due to revision change.

@RaduSabau1 RaduSabau1 marked this pull request as ready for review March 5, 2026 18:07
@RaduSabau1
Copy link
Collaborator Author

V2:

  • Updated driver, IIO driver and project for new revision compatibility.

Copy link
Contributor
@amiclaus amiclaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this ready for review?

@RaduSabau1
Copy link
Collaborator Author

is this ready for review?

This PR is ready for review.

@RaduSabau1 RaduSabau1 force-pushed the staging/ad4692 branch 2 times, most recently from 96deec6 to edccadb Compare March 10, 2026 16:16
Add initial header and source file for AD4692 no-OS driver.

Signed-off-by: Radu Sabau <radu.sabau@analog.com>
Add initial header and source files for AD4692 IIO driver.

Signed-off-by: Radu Sabau <radu.sabau@analog.com>
Add initial .rst documentation file for AD4692 driver.

Signed-off-by: Radu Sabau <radu.sabau@analog.com>
Add initial project files for EVAL-AD4692-ARDZ board.

Signed-off-by: Radu Sabau <radu.sabau@analog.com>
Add initial .rst documentation file for AD4692-ARDZ project.

Signed-off-by: Radu Sabau <radu.sabau@analog.com>
@RaduSabau1
Copy link
Collaborator Author

V3:

  • Addressed unresolved comments from V1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0