Skip to content
This repository was archived by the owner on Dec 18, 2025. It is now read-only.

Initial Cortex R-profile support.#945

Open
alfedotov wants to merge 2 commits intoARM-software:developfrom
alfedotov:Core-R
Open

Initial Cortex R-profile support.#945
alfedotov wants to merge 2 commits intoARM-software:developfrom
alfedotov:Core-R

Conversation

@alfedotov
Copy link
Copy Markdown

About year ago I had an attemp to upsream these changes. And still I see demand on such support.

@alfedotov alfedotov mentioned this pull request Jun 19, 2020
@tomoveu
Copy link
Copy Markdown

tomoveu commented Jun 6, 2022

+1 this looks like a great addition, considering it has no merge conflicts with master.

This would solve #314 Could someone please review this PR, @llefaucheur ?

@christophe0606
Copy link
Copy Markdown
Contributor

@JonatanAntoni This is a CMSIS-Core contribution. Independent from CMSIS-DSP

@tomoveu
Copy link
Copy Markdown

tomoveu commented Jun 20, 2022

@JonatanAntoni This is a CMSIS-Core contribution. Independent from CMSIS-DSP

Could you please elaborate what this means? @christophe0606

Do we have to submit these changes to a different repository or in a different manner?

@JonatanAntoni
Copy link
Copy Markdown
Member

@tomoveu,

I guess Christophe is just wondering why I was asking him to review code in "my" area.
Your PR is totally fine here. I am just unable to confirm the code is correct. I am looking for some Cortex-R experts who can give the PR a review.

Thanks,
Jonatan

@bernardfitch
Copy link
Copy Markdown

+1 I too would like to see basic Cortex-R support like this in CMSIS-Core. I would like to encourage @christophe0606 and @llefaucheur to please review and pull these changes (if deemed appropriate) :)

If there is anything that I can do to help please let me know

(I am currently working on an R52-based chip design and continually building/testing code - coming from the M4 world it made me sad that there is no Cortex-R CMSIS code, having to revert to assembly for all but the most primitive of startup code seems like a backward step).

Bernard ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants