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

H2: Add initial ADC support with example #565

Merged
merged 2 commits into from
May 31, 2023

Conversation

JurajSadel
Copy link
Contributor

@JurajSadel JurajSadel commented May 30, 2023

We already have analog gpios and other stuff, so this just adds an example and export in prelude.
There is no calibration yet, we should wait for the calibration PR(#555) to be merged and we can address it afterward.
H2 calibration is currently not implemented in ESP-IDF (IDF-6216)

@JurajSadel JurajSadel marked this pull request as ready for review May 30, 2023 10:59
@bjoernQ
Copy link
Contributor

bjoernQ commented May 30, 2023

Did some very simple tests and it seems to work fine for me

Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

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

LGTM

@JurajSadel JurajSadel requested a review from jessebraham May 30, 2023 11:09
@JurajSadel JurajSadel force-pushed the feature/esp32h2-adc branch from dd283a9 to e7ba5c2 Compare May 31, 2023 12:55
@jessebraham jessebraham merged commit b1630db into esp-rs:main May 31, 2023
MabezDev pushed a commit to MabezDev/esp-hal that referenced this pull request Jun 1, 2023
* H2: Add initial ADC support with example

* Add changelog
SergioGasquez pushed a commit to SergioGasquez/esp-hal that referenced this pull request Jun 9, 2023
* H2: Add initial ADC support with example

* Add changelog
@JurajSadel JurajSadel deleted the feature/esp32h2-adc branch April 10, 2024 15:29
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.

3 participants