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

zephyr: Enable support for RISCV #194

Merged
merged 1 commit into from
Jan 4, 2022
Merged

zephyr: Enable support for RISCV #194

merged 1 commit into from
Jan 4, 2022

Conversation

carlocaione
Copy link
Collaborator

Make libmetal usable also for RISCV targets.

Signed-off-by: Carlo Caione ccaione@baylibre.com

Copy link
Contributor

@arnopo arnopo left a comment

Choose a reason for hiding this comment

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

minor clean-up else LGTM

lib/system/zephyr/riscv/CMakeLists.txt Outdated Show resolved Hide resolved
@arnopo arnopo added this to the Release V2022.04 milestone Dec 22, 2021
Make libmetal usable also for RISCV targets.

Signed-off-by: Carlo Caione <ccaione@baylibre.com>
Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

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

Looks good to go.

@edmooring
Copy link
Contributor

Can we add at least a build test to the CI?

@arnopo
Copy link
Contributor

arnopo commented Jan 4, 2022

Can we add at least a build test to the CI?

That's a good point! at least we could add build test with RISC compiler in CI.
@carlocaione
I suppose that everything is available to build for RISCV in Zephyr environment
what should be update in Zephyr build sequence to build for RISCV?

@carlocaione
Copy link
Collaborator Author

what should be update in Zephyr build sequence to build for RISCV?

Only one line.

For RISCV64:

cmake . -DWITH_ZEPHYR=on -DBOARD=qemu_riscv64 -DWITH_TESTS=on -Bbuild-zephyr

for RISCV32:

cmake . -DWITH_ZEPHYR=on -DBOARD=qemu_riscv32 -DWITH_TESTS=on -Bbuild-zephyr

@arnopo
Copy link
Contributor

arnopo commented Jan 4, 2022

what should be update in Zephyr build sequence to build for RISCV?

Only one line.

For RISCV64:

cmake . -DWITH_ZEPHYR=on -DBOARD=qemu_riscv64 -DWITH_TESTS=on -Bbuild-zephyr

for RISCV32:

cmake . -DWITH_ZEPHYR=on -DBOARD=qemu_riscv32 -DWITH_TESTS=on -Bbuild-zephyr

Thanks!
I sent a related pull request: #195

@arnopo arnopo merged commit c6ae2c6 into OpenAMP:main Jan 4, 2022
@carlocaione carlocaione deleted the riscv branch January 4, 2022 11:19
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.

3 participants