feat(Other): Enable building zephyr for risc-v targets #1051
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Configures the zephyr build system to use the appropriate cmsis system startup and peripheral driver sources based on the target core architecture. The mxc lock driver is not implemented for risc-v and emits a build warning (which twister promotes to an error), therefore the zephyr build system is configured to build it only for arm. Before this patch, we could only build zephyr for arm targets, but now we can build zephyr for arm or risc-v targets. Signed-off-by: Maureen Helm <[email protected]>
ozersa
approved these changes
Jun 24, 2024
|
@Jake-Carter can you take a look? |
Jake-Carter
reviewed
Jul 1, 2024
There was a problem hiding this comment.
MAX78000 and MAX78002 also have RISC-V cores and need similar modifications.
Also, the RISC-V core needs to be booted up by the ARM core, and by default the RISC-V JTAG is not enabled. In the MSDK we have a system for dynamically linking RISC-V application on top of some basic start-up code that runs in the ARM core (see RV_ARM_Loader and the implementation here).
The implementation isn't really straightforward and involves parsing the .map file for a custom symbol added in the linker script. Is there a similar mechanism planned for Zephyr?
|
On RISC-V MXC_Lock implementation - we've had this in the TODO for a while. Let me know if we need to bump this up in priority. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Configures the zephyr build system to use the appropriate cmsis system startup and peripheral driver sources based on the target core architecture. The mxc lock driver is not implemented for risc-v and emits a build warning (which twister promotes to an error), therefore the zephyr build system is configured to build it only for arm.
Before this patch, we could only build zephyr for arm targets, but now we can build zephyr for arm or risc-v targets.
Checklist Before Requesting Review