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

[µTVM]: Zephyr: Add mps2_an521 board to the CI #7914

Merged
merged 1 commit into from
May 3, 2021

Conversation

gromero
Copy link
Contributor

@gromero gromero commented Apr 24, 2021

Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
This will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current test (x86) to ease reading about which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero gustavo.romero@linaro.org

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@gromero gromero force-pushed the add-mps2-to-ci-for-rev branch 2 times, most recently from bba2b12 to 16f34f1 Compare May 1, 2021 02:09
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
@gromero gromero force-pushed the add-mps2-to-ci-for-rev branch from 16f34f1 to 284c2a8 Compare May 1, 2021 13:14
@gromero
Copy link
Contributor Author

gromero commented May 2, 2021

@areusch @mehrdadh Finally CI is happy :) Could I get reviews for that change please? fyi @tom-gall @leandron @u99127

@areusch areusch merged commit 01252d0 into apache:main May 3, 2021
@areusch
Copy link
Contributor

areusch commented May 3, 2021

thanks @gromero, the PR is now merged!

@gromero
Copy link
Contributor Author

gromero commented May 3, 2021

thanks @gromero, the PR is now merged!

@areusch Cool. Thanks a lot Andrew!

umangyadav pushed a commit to umangyadav/tvm that referenced this pull request May 5, 2021
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request May 11, 2021
Since Arm reference board mps2_an521 is now added as a test platform to
test µTVM with Zephyr and that test platform runs by default emulated,
plus Zephyr docker images were updated to use Zephyr v2.5-branch, add
the mps2_an521 board as a platform to be automatically used by the CI.

That change will allow testing µTVM on top of Zephyr running on a
Cortex-m33 MCU. Currently only a x86 VM is used for that kind of test.
Hence it will help ensure that there is no regression on Arm-based MCUs.

That commit also adds explicitly the parameter --microtvm-platforms=host
to the current x86 test for ease of reading on which test platforms are
triggered in the CI ('host' is the default platform when that parameter
is ommited, so nothing changes for tests on the x86 VM).

Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
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