-
Notifications
You must be signed in to change notification settings - Fork 156
Mips target #493
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
Mips target #493
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @therealprof (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
Add test if it is possible and update changelog, please. |
if possible, add some svd2rust-regress targets as well, not necessary but appreciated. |
Reviewing all the code motivated me to move the core access stuff to a new crate |
Here also, please: |
done. |
Update this line and squash commits: svd2rust/.github/workflows/ci.yml Line 17 in 6bfc843
|
Looks good to me |
Yes, the tests passed. In the meantime, I created PACs that include Would be great if you could merge it. |
I still wait you squash commits. |
I didn't realize that there was still an open point because I was not familiar with "squashing" commits. Now everything should be in one single commit. |
bors r+ |
Cool. @burrbull, thanks a lot for your help and patience. |
r? @therealprof
Hi all,
Willing to add some basic support for MIPS cores? This patch enables the Peripheral Singleton. I use svd2rust to create PACs for PIC32MX microcontroller.
Best regards,
Stephan