Skip to content

Conversation

@linguini1
Copy link
Contributor

Summary

Since the ADS1115 has a relatively slow conversion rate, this additional ioctl command makes it possible to trigger a conversion before reading the reading the conversion result, allowing the user to perform other computation in between instead of waiting for the conversion time to complete. It improves sampling time.

Impact

Users can parallelize sampling of the ADS1115 ADC to avoid delays from sleeping while waiting for the conversion time duration.

Testing

Tested on a rocketry system with 4 ADS1115s in the sensor array. This change allowed parallezation to speed up sampling from 10Hz to 15Hz per channel per ADC.

@bskdany performed this testing, as did I.

@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Area: Drivers Drivers issues Size: M The size of the change in this PR is medium labels Jul 11, 2025
@linguini1 linguini1 force-pushed the ads1115_conversion_trigger branch from 06b7c9f to 2dc341d Compare July 11, 2025 19:59
@linguini1
Copy link
Contributor Author

I don't believe the build failure is a result of these changes.

bskdany and others added 2 commits July 12, 2025 14:39
Since the ADS1115 has a relative slow conversion rate, this additional
ioctl command makes it possible to trigger a conversion before reading
the reading the conversion result, allowing the user to perform other
computation in between instead of waiting for the conversion time to
complete. It improves sampling time.

Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
Updates the ADS1115 documentation with information about the two new
commands for triggering conversion and reading the result.

Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
@linguini1 linguini1 force-pushed the ads1115_conversion_trigger branch from 2dc341d to a9267c0 Compare July 12, 2025 18:40
@github-actions github-actions bot added the Size: L The size of the change in this PR is large label Jul 12, 2025
@linguini1 linguini1 requested a review from cederom July 13, 2025 13:17
@linguini1 linguini1 merged commit 953ab4e into apache:master Jul 14, 2025
52 of 76 checks passed
@linguini1 linguini1 deleted the ads1115_conversion_trigger branch July 14, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Documentation Improvements or additions to documentation Area: Drivers Drivers issues Size: L The size of the change in this PR is large Size: M The size of the change in this PR is medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants