-
Couldn't load subscription status.
- Fork 12
Create aarch32-cpu and aarch32-rt crates. #78
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
base: main
Are you sure you want to change the base?
Conversation
|
This got big, and Armv5TE stuff is broken in weird ways, so I'm going to pull out parts of this as separate PRs. Current list of weirdness:
|
|
Needs to be rebased. I can also perform some testing on the Zynq7000 |
Replaces cortex-r-rt, cortex-a-rt and cortex-ar.
) Bigger stacks, so the examples work in release mode ) Formatting ) Remove gic feature
They have no atomics to switch tests to use portable-atomic. Also give the examples more stack space to match mps3-an536 tests. IFAR is also not available, so skip it in the prefetch abort test.
416e1d6 to
d1d0bcc
Compare
|
Phew, all tests pass on ARMv4T (PXA25) and ARMv5TE (ARM926) in QEMU. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about the double precision context functions? (
cortex-ar/cortex-a-rt/src/lib.rs
Line 595 in b7d60b9
| macro_rules! save_context { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are no EABIHF targets for Armv4T and Armv5TE so I didn’t check.
| readme = "README.md" | ||
| repository = "https://github.com/rust-embedded/cortex-r.git" | ||
| rust-version = "1.83" | ||
| version = "0.2.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it make sense to set this to v0.1.0? It is a merge of 2 run-time crates with a different name, so maybe a the crate with a new name starting at version v0.2.1 might be confusing? I tripped over this trying to include it with version v0.1.0 (former cortex-a-rt version)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point
|
I tested some of the Zynq7000 examples, and they work without issues :) Those did not include floating code I think. Might add that to test the double precision support. |
Replaces cortex-r-rt, cortex-a-rt and cortex-ar.