Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Integrate numbered RAL instances #3

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

Integrate numbered RAL instances #3

wants to merge 5 commits into from

Conversation

mciantyre
Copy link
Member

The PR integrates the numbered RAL instances into the CCM API. Now that RAL Instances are generic over a typenum, we need to change the Instance implementations in the ral module. We now implement the Instance trait on the RegisterBlock, which is a dereference away from a RAL Instance.

See imxrt-rs/imxrt-ral#4 for more information.

@mciantyre
Implement Instance on RAL RegisterBlocks
3143c25
@mciantyre
Target WIP RAL crate
baee7bf
@mciantyre
Manual require deref coercion for Handle methods
dccfcd8 
Not sure why this is required here, but not on the clock methods.
@mciantyre
Revert "Manual require deref coercion for Handle methods"
77e7e4b 
This reverts commit 72eedff613eab6eee36cd57e2c4cac8b68bb33c5.
@mciantyre
Manually derefence RAL instances for Handle calls
de0bdab 
I'm not sure why this is necessary. It seems that we can support
deref coercion for the clock types, but not with the handle. This
is a minor inconvenience for RAL users, but it keeps the CCM API
consistent and usable for other purposes.
@mciantyre
Copy link
Member Author

WIP; depends on the RAL implementation.

Consider merging #2 before this PR. #2 adds a new Instance implementation for the DCDC. We should include an update for that implementation as part of this branch.

@mciantyre mciantyre mentioned this pull request Dec 26, 2020
Closed
10 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mciantyre