Skip to content

Add clock option for CH32V003 #66

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

Closed
wants to merge 2 commits into from

Conversation

DeqingSun
Copy link
Contributor

I tried the Arduino Repo on a customized board and the systick was 50% speed while the UART run in correct speed. After some troubleshooting I realized the clock was set to external 48M, which fails and return to backup internal clock.

This PR allow user to choose clock source. That is useful for projects without external oscillator.

@maxint-rd
Copy link
Contributor

Upvote for this - I'm all in favor for this and suggested the identical change in issue #27. Thank you for submitting this as a PR. (I was hoping WCH would take my suggestion to make the change and include other MCU's that I don't own and couldn't test...)

@DeqingSun
Copy link
Contributor Author

Upvote for this - I'm all in favor for this and suggested the identical change in issue #27. Thank you for submitting this as a PR. (I was hoping WCH would take my suggestion to make the change and include other MCU's that I don't own and couldn't test...)

Did not see your issue since there was so many. Hoping a PR can make the work easier on WCH's side.

@maxint-rd
Copy link
Contributor

Another upvote. Tested this PR successfully in Arduino IDE v2.3.2.

Note that after manually copying the modified files to their proper location, the local cache of the Arduino IDE needs to be deleted to see the menu changed. See also this Arduino IDE issue.

Copy link
Contributor

@maxint-rd maxint-rd left a comment

Choose a reason for hiding this comment

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

Works fine. Excellent addition needed by everyone using bare chips.

@TianpeiLee
Copy link
Collaborator

Thank you for your contribution. I may need to use your method for unified modification to adapt to all current chips.

@DeqingSun
Copy link
Contributor Author

Thank you for your contribution. I may need to use your method for unified modification to adapt to all current chips.

Please go ahead. Look forward to see this feature in release.

@TianpeiLee
Copy link
Collaborator

thanks again

@TianpeiLee TianpeiLee closed this May 30, 2024
@maxint-rd maxint-rd mentioned this pull request May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants