Skip to content

DAC fixes #1326

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

Merged
merged 2 commits into from
Mar 16, 2021
Merged

DAC fixes #1326

merged 2 commits into from
Mar 16, 2021

Conversation

ABOSTM
Copy link
Contributor

@ABOSTM ABOSTM commented Mar 15, 2021

Summary

  • DAC: STM32F3 dac may have output switch instead of output buffer
    For STM32F3 serie, depending on instance/channel used,
    output buffer may be replaced by output switch
    Both being exclusive, they can be both set in HAL structure.

  • DAC: remove HAL_DAC_DeInit() to avoid other channel deconfiguration

Fixes #1324

ABOSTM added 2 commits March 15, 2021 19:02
For STM32F3 serie, depending on instance/channel used,
output buffer may be replaced by output switch
Both being exclusive, they can be both set in HAL structure.

Fixes stm32duino#1324

Signed-off-by: Alexandre Bourdiol <[email protected]>
@fpistm fpistm self-requested a review March 15, 2021 18:36
@fpistm fpistm added the fix 🩹 Bug fix label Mar 15, 2021
@fpistm fpistm added this to the 2.0.0 milestone Mar 15, 2021
Copy link
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@kiiv-cz kiiv-cz left a comment

Choose a reason for hiding this comment

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

Just tested, all three channels working now.

@fpistm fpistm merged commit 01a1aa3 into stm32duino:master Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 🩹 Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DAC1 channel 2 and DAC2 channel 1 are not working on STM32F303
3 participants