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 ADC disable #699

Merged
merged 2 commits into from Apr 4, 2022
Merged

Fix ADC disable #699

merged 2 commits into from Apr 4, 2022

Conversation

Sh3Rm4n
Copy link
Member

@Sh3Rm4n Sh3Rm4n commented Feb 9, 2022

ADDIS, as ADEN, is a signal to the ADC peripheral to change it's
state. Both signals have to be "1" to change the state.

In case of ADDIS it will be only reset to "0", after the state has
successfully changed.

The procedure of disabling is the same between stm32f303xc (RM0316
15.3.10), where I've noticed the bug, and stm32h7x3 (RM0443 25.4.9),
for which this code was initially committed, so I assume this affects
every device using the adc_v3.yaml.

`ADDIS`, as `ADEN` is a signal to the ADC peripheral to change it's
state. Both signals have to be "1" to change the state.

In case of `ADDIS` it will be only reset to "0", after the state has
successfully changed.

The procedure of disabling is the same between `stm32f303xc` (RM0316
15.3.10), where I've noticed the bug, and `stm32h7x3` (RM0443 25.4.9),
for which this code was initially committed, so I assume this affects
every device using the `adc_v3.yaml`.
@github-actions
Copy link

github-actions bot commented Feb 9, 2022

Memory map comparison

@richardeoin
Copy link
Member

This is an error in the enumerations, thanks for highlighting it. We actually have better enumerations for these bits in CR in adc_l0.yaml (among others) already, so I'm going to add those on top also

@richardeoin richardeoin added the bug label Apr 4, 2022
@github-actions
Copy link

github-actions bot commented Apr 4, 2022

Memory map comparison

Copy link
Member

@richardeoin richardeoin left a comment

Choose a reason for hiding this comment

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

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 4, 2022

Build succeeded:

@bors bors bot merged commit 13c1502 into stm32-rs:master Apr 4, 2022
@Sh3Rm4n Sh3Rm4n deleted the fix-adc-disable branch April 4, 2022 19:47
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.

None yet

3 participants