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

drivers: dac: Introduce gd32 driver #41152

Merged
merged 4 commits into from
Dec 16, 2021

Conversation

cameled
Copy link
Contributor

@cameled cameled commented Dec 14, 2021

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

@nandojve nandojve changed the title Introduce gd32 dac driver drivers: dac: Introduce gd32 driver Dec 14, 2021
@nandojve
Copy link
Member

@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.

@nandojve nandojve mentioned this pull request Dec 14, 2021
55 tasks
@cameled
Copy link
Contributor Author

cameled commented Dec 14, 2021

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.

drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/Kconfig.gd32 Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
@cameled
Copy link
Contributor Author

cameled commented Dec 14, 2021

@gmarull I will fix others issue in tomorrow.

drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
Copy link
Member

@gmarull gmarull left a comment

Choose a reason for hiding this comment

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

LGTM, some suggestions

drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
dts/bindings/dac/gd,gd32-dac.yaml Outdated Show resolved Hide resolved
Add DAC for gd32f4xx and gd32f350xx.

Signed-off-by: HaiLong Yang <cameledyang@pm.me>
Copy link
Member

@martinjaeger martinjaeger left a 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!

samples/drivers/dac/README.rst Outdated Show resolved Hide resolved
samples/drivers/dac/README.rst Outdated Show resolved Hide resolved
drivers/dac/dac_gd32.c Outdated Show resolved Hide resolved
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>
Copy link
Member

@nandojve nandojve left a comment

Choose a reason for hiding this comment

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

LGTM!

@nandojve nandojve added this to the v3.0.0 milestone Dec 15, 2021
@carlescufi carlescufi merged commit ff9d967 into zephyrproject-rtos:main Dec 16, 2021
@cameled cameled deleted the gd32-dac branch December 16, 2021 09:17
@str4t0m str4t0m added the platform: GD32 GigaDevice label Aug 26, 2022
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.

9 participants