Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for AD7191 #2655

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add support for AD7191 #2655

wants to merge 4 commits into from

Conversation

romandariana
Copy link
Contributor

@romandariana romandariana commented Nov 21, 2024

PR Description

AD7191 is a chip from the AD719X family, with similar functionalities. Since this chip is entirely pin-programmable with no registers, I wrote a separate driver from ad7192.c.
Sigma delta adc drivers use the set_mode() callback function which contains a register write to make sure the chip select line is asserted before waiting for an interrupt. Devices such as AD7191 and AD7780 have no registers to write to, so I added a helper function ad_sd_assert_cs() for these use cases.

  • Add helper function for CS assertion in ad_sigma_delta
  • Add AD7191 documentation and driver

PR Type

  • New feature (a change that adds new functionality)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have tested the changes on the relevant hardware

@romandariana romandariana changed the title Add AD7191 driver Add support for AD7191 Nov 21, 2024
@romandariana romandariana marked this pull request as ready for review December 12, 2024 16:17
@romandariana romandariana changed the title Add support for AD7191 Add support for AD7191 + ad_sigma helper function Dec 13, 2024
@romandariana romandariana marked this pull request as draft December 20, 2024 13:03
Some sigma-delta ADCs, such as AD7191 and AD7780, have no registers and
start conversion when CS is asserted. Add helper function to support
this use case by allowing devices to assert CS without performing
register operations.

This function can be used by drivers through their set_mode callback.

Signed-off-by: Alisa-Dariana Roman <[email protected]>
Automatically imply AD7191.

Signed-off-by: Alisa-Dariana Roman <[email protected]>
AD7191 is a pin-programmable, ultralow noise 24-bit sigma-delta ADC
designed for precision bridge sensor measurements. It features two
differential analog input channels, selectable output rates,
programmable gain, internal temperature sensor and simultaneous
50Hz/60Hz rejection.

Signed-off-by: Alisa-Dariana Roman <[email protected]>
AD7191 is a pin-programmable, ultralow noise 24-bit sigma-delta ADC
designed for precision bridge sensor measurements. It features two
differential analog input channels, selectable output rates,
programmable gain, internal temperature sensor and simultaneous
50Hz/60Hz rejection.

Signed-off-by: Alisa-Dariana Roman <[email protected]>
@romandariana romandariana changed the title Add support for AD7191 + ad_sigma helper function Add support for AD7191 Dec 20, 2024
@romandariana romandariana marked this pull request as ready for review December 20, 2024 13:57
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.

1 participant