Skip to content

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

Merged
merged 1 commit into from
Feb 27, 2021
Merged

Mips target #493

merged 1 commit into from
Feb 27, 2021

Conversation

kiffie
Copy link
Contributor

@kiffie kiffie commented Feb 22, 2021

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

@kiffie kiffie requested a review from a team as a code owner February 22, 2021 20:48
@rust-highfive
Copy link

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.

@rust-highfive rust-highfive added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-tools labels Feb 22, 2021
@burrbull
Copy link
Member

Add test if it is possible and update changelog, please.

@Emilgardis
Copy link
Member

if possible, add some svd2rust-regress targets as well, not necessary but appreciated.

@kiffie
Copy link
Contributor Author

kiffie commented Feb 23, 2021

svd2rust-regresstarget added.

CHANGELOG.md updated

Reviewing all the code motivated me to move the core access stuff to a new crate mips_mcu to separate it from the startup routine and the ISR tables. I changed my svd2rust patch accordingly.

@burrbull
Copy link
Member

@kiffie
Copy link
Contributor Author

kiffie commented Feb 23, 2021

@burrbull
Copy link
Member

Update this line and squash commits:

VENDOR: [rustfmt, Atmel, Freescale, Fujitsu, Holtek, Nordic, Nuvoton, NXP, RISC-V, SiliconLabs, Spansion, STMicro, Toshiba]

@burrbull
Copy link
Member

Looks good to me

@kiffie
Copy link
Contributor Author

kiffie commented Feb 25, 2021

Looks good to me

Yes, the tests passed. In the meantime, I created PACs that include Peripherals::take().

Would be great if you could merge it.

@burrbull
Copy link
Member

I still wait you squash commits.

@kiffie
Copy link
Contributor Author

kiffie commented Feb 26, 2021

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.

@burrbull
Copy link
Member

bors r+

@bors bors bot merged commit e94fc7b into rust-embedded:master Feb 27, 2021
@kiffie
Copy link
Contributor Author

kiffie commented Feb 27, 2021

Cool. @burrbull, thanks a lot for your help and patience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants