-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
STM32 DAC Driver - Add support to DAC3 #4255
Conversation
|
||
/* Register the DAC driver at "/dev/dac0" */ | ||
|
||
ret = dac_register("/dev/dac0", g_dac5); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What will happen if CONFIG_STM32_DAC1CH1 and CONFIG_STM32_DAC3CH2 be defined? I think it will fail to register the second "/dev/dac0". I think you need to avoid it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right. I will change the register path of DAC3CH2 to "/dev/dac5".
Thanks
COMP example was changed to use DAC3CH2 signal on COMP INM input.
6a3ae78
to
d7cee36
Compare
There is any update about this PR? Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Summary
Add support to DAC3 for STM32G4 devices. The nucleo-g431kb:comp configuration was changed to use DAC3CH2 signal on
COMP2 inverting input. The voltage on inverting input can be changed using dac tool.
Impact
Testing
Tested on nucleo-g431kb board.