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.

Sign up for GitHub

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

Jump to bottom

[microTVM] Add Nucleo stm32l4r5zi board to zephyr #8386

Merged
merged 6 commits into from
Jul 3, 2021
Merged

[microTVM] Add Nucleo stm32l4r5zi board to zephyr #8386

merged 6 commits into from
Jul 3, 2021

Conversation

mehrdadh
Copy link
Member

@mehrdadh mehrdadh commented Jul 1, 2021 •
edited
Loading

This PR:

  • adds stm32l4r5zi as another target to micro targets
  • adds stm32l4r5zi config files for zephyr test and zephyr AOT test

cc @areusch @gromero

@mehrdadh
Copy link
Member Author

mehrdadh commented Jul 2, 2021

test_zephyr and test_zephyr_aot has passed on stm32l4r5zi board on my local machine.

leandron
leandron reviewed Jul 2, 2021
View reviewed changes
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM overall. Just a few nits, trying to keep the code organised.

python/tvm/micro/contrib/zephyr.py Outdated Show resolved Hide resolved
python/tvm/target/target.py Outdated Show resolved Hide resolved
tests/micro/zephyr/conftest.py Outdated Show resolved Hide resolved
@mehrdadh
Copy link
Member Author

mehrdadh commented Jul 2, 2021

@leandron thanks for the review. I changed the order.

leandron
leandron approved these changes Jul 2, 2021
View reviewed changes
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @mehrdadh

@leandron leandron merged commit 7e3f068 into apache:main Jul 3, 2021
@mehrdadh mehrdadh deleted the nucleo_stm32l4r5zi branch July 7, 2021 16:15
ylc pushed a commit to ylc/tvm that referenced this pull request Sep 29, 2021
@mehrdadh @ylc
[microTVM] Add Nucleo stm32l4r5zi board to zephyr (apache#8386)
afcee06 
* add stm32l4r5zi_nucleo

* add parameter for test qemu

* file type check

* fix test

* change order

* revert
@junrushao junrushao mentioned this pull request Nov 1, 2021
Closed
zxy844288792 pushed a commit to zxy844288792/tvm that referenced this pull request Mar 4, 2022
@mehrdadh @ylc
[microTVM] Add Nucleo stm32l4r5zi board to zephyr (apache#8386)
a06207a 
* add stm32l4r5zi_nucleo

* add parameter for test qemu

* file type check

* fix test

* change order

* revert
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@areusch areusch areusch left review comments

@leandron leandron leandron approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mehrdadh @leandron @areusch