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

Feature/document and streamline rcc interface #241

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zheylmun
Copy link

Added documentation based on review of RCC section of reference manual
Simplified existence tokens and ensure that they can't be created outside of the HAL.

Interface Question:
From reading the reference manual, it's not clear to me that applying a clock configuration "freezes" it.

  • If the RCC config is mutable, then maybe the freeze terminology and statement that the existence of a Clocks struct indicates that the clocks cannot be further modified should be reconsidered?

  • If the RCC config is fixed somehow once it's applied, maybe it might make sense to remove the clock_src function from the Config implementation, as you only get a config by specifying a src to begin with.

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.

1 participant