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

fix(PeriphDrivers): Added update to force scale select sample to occur with ADC and ADC RefSelect for REVA API #779

Merged
merged 8 commits into from
Oct 24, 2023

Conversation

EricB-ADI
Copy link
Contributor

@EricB-ADI EricB-ADI commented Oct 24, 2023

Description

For chips with an ADC which allows for external voltage division, the scale select is only sampled after the channel select is switched to a new channel. So when using channel one, the external scale will not take effect since 0 is the default channel and is not switched even if configured. This fix forcefully toggles the channel selected anytime the ExtScale is set.

Also added is the API for ADC RefSelect for REVA

@EricB-ADI
Copy link
Contributor Author

/clang-format-run

@github-actions github-actions bot added MAX32570 Related to the MAX32570 (ME13) MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32665 Related to the MAX32665 (ME14) MAX32680 Related to the MAX32680 (ME20) MAX78000 Related to the MAX78000 (AI85) labels Oct 24, 2023
@EricB-ADI EricB-ADI changed the title fix(PeriphDrivers): Added update to force scale select sample to occur with ADC fix(PeriphDrivers): Added update to force scale select sample to occur with ADC and ADC RefSelect for REVA API Oct 24, 2023
@EricB-ADI
Copy link
Contributor Author

/clang-format-run

@github-actions github-actions bot added the Register Change This issue or pull request involves a change to the MSDK registers. label Oct 24, 2023
@EricB-ADI
Copy link
Contributor Author

/generate-register-files

Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

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

Looks good, just need to add mxc_adc_ref_t to MAX32650 and MAX32665 top-level adc.h headers to fix the build errors

@Jake-Carter
Copy link
Contributor

Bot updated headers :/

c8b648e

@EricB-ADI
Copy link
Contributor Author

/generate-register-files

@EricB-ADI EricB-ADI merged commit ed99a4c into main Oct 24, 2023
7 of 8 checks passed
@EricB-ADI EricB-ADI deleted the fix/adc-scale-sel branch October 25, 2023 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32570 Related to the MAX32570 (ME13) MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32665 Related to the MAX32665 (ME14) MAX32680 Related to the MAX32680 (ME20) MAX78000 Related to the MAX78000 (AI85) Register Change This issue or pull request involves a change to the MSDK registers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants