-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
drivers: dac: Introduce gd32 driver #41152
Conversation
@cameled , Thank you for your PR. I would ask you to always add PR description to make community aware about what is your work. That is the place were you should describe, in few words, what people can expect, pros and cons (when applicable) etc. This way you get more attention. |
done. |
@gmarull I will fix others issue in tomorrow. |
7b63d9f
to
583ca44
Compare
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, some suggestions
Add DAC for gd32f4xx and gd32f350xx. Signed-off-by: HaiLong Yang <cameledyang@pm.me>
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.
Thanks for this addition. Only got some nitpicking comments, otherwise looks good!
Add DAC driver for gd32 series SoCs. Signed-off-by: HaiLong Yang <cameledyang@pm.me>
Add DAC0 pinctrl info to gd32f450i_eval board. Signed-off-by: HaiLong Yang <cameledyang@pm.me>
DAC sample support gd32f450i_eval board. Signed-off-by: HaiLong Yang <cameledyang@pm.me>
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!
Introduce GD32 DAC driver in here.
From GD32 MCU Selection Guide, we can find most gd32 SoCs have 2 DAC output, but some like gd23f350 have only one DAC output. This PR add support for both 2 DAC and 1 DAC output.
Also add GD32450i-EVAL board to dac driver sample.
More GD32 support progress? #38657