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

Generate RAL for imxrt1180 #51

Merged
merged 2 commits into from
Nov 11, 2024
Merged

Conversation

nilfit
Copy link
Contributor

@nilfit nilfit commented Oct 30, 2024

The common.yaml SVD patches do not apply in this case, as some previously common peripherals are not present in the RT1180, (SNVS, IOMUXC_GPR).

The common SVD patches do not apply here.
@nilfit
Copy link
Contributor Author

nilfit commented Oct 31, 2024

I had forgotten to support the rt feature. This is now fixed.

Copy link
Member

@mciantyre mciantyre left a comment

Choose a reason for hiding this comment

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

I'm not doing a deep review to make sure the patches / generated code is correct. We'll find those as we use them. I'd like to get the 1189 covered by CI, so I'm adding those jobs in a separate commit.

I'm planning a imxr-ral 0.6 with updates throughout the imxrt-rs ecosystem. Right now, I intend to keep these 1189 contributions out of the first 0.6.0 release. I'd like to give folks a chance to improve and break 1189 RAL items as we work through driver development. Once there's some 1189 drivers ready to go, we can release the 1189 in a 0.6 patch release.

This isn't a firm policy; let me know if you have a reason to immediately include the 1189 in the next breaking release.

@mciantyre mciantyre merged commit b6d1bc4 into imxrt-rs:master Nov 11, 2024
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.

2 participants