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

Support for baremetal R5 in TI K3 family devices #503

Closed
wants to merge 3 commits into from

Conversation

kwillis01
Copy link

Added support for RPMsg on the baremetal R5 in the TI SK-AM64. This can be extended to other devices in the TI K3 family by changing addresses in the linker and platform_info files. A mailbox driver was added to kick RPMsg for these devices.

Kendall Willis added 2 commits August 8, 2023 16:45
Enabled ASM language in project to allow for board specific
functions and interrupts in the TI K3 family devices.

Signed-off-by: Kendall Willis <kendallwillis@ti.com>
Adding baremetal support for the R5 to use RPMsg for the devices
in the TI K3 family, specifically for the SK-AM64. Users should
be able to replace memory addresses for the R5 in the
linker_remote.ld file and platform_info.h files to make RPMsg
compatible with the R5 in their chosen board. Cmake file was
added to aid in building for the R5 in TI K3 family devices.
Mailbox driver added to allow RPMsg to work with the TI K3 family
devices. Other API files added to aid in interrupts and other
board specific functions. IPI and no IPI is supported in the
platform_info files to allow mailbox to kick RPMsg.

Signed-off-by: Kendall Willis <kendallwillis@ti.com>
@tnmysh
Copy link
Collaborator

tnmysh commented Aug 10, 2023

@kwillis01 Thank you for the patches.

We have planned to move demos and application development to openamp-system-reference repository. As this is new development, I suggest to create pull reuqest in that repository here:
https://github.com/OpenAMP/openamp-system-reference

The good location in that repository would be:
examples/baremetal/TI/ or examples/baremetal/platforms/TI/ something like this.

The existing apps will be moved there soon as well.

@wmamills @arnopo @edmooring for reviews.

@kwillis01
Copy link
Author

@tnmysh Okay I will do that. What should I do with the ti_k3_r5.cmake and CMakeLists.txt I edited when moving to the system reference? Should I just add in the ti_k3_r5.cmake and add a note to change the CMakeLists.txt line?

@tnmysh
Copy link
Collaborator

tnmysh commented Aug 11, 2023

@tnmysh Okay I will do that. What should I do with the ti_k3_r5.cmake and CMakeLists.txt I edited when moving to the system reference? Should I just add in the ti_k3_r5.cmake and add a note to change the CMakeLists.txt line?

I did not get the question. We will link ope-amp library statically with baremetal code. So, I don't think you should modify CMakeLists.txt at all.

….cmake

Allow users to run build cmake without having to specify where libmetal
is if it is in the same directory as OpenAMP. Included LIBMETAL_PATH
variable if the libmetal directory is not in the same directory as
OpenAMP.

Signed-off-by: Kendall Willis <kendallwillis@ti.com>
@github-actions
Copy link

This pull request has been marked as a stale pull request because it has been open (more than) 45 days with no activity.

@github-actions github-actions bot added the Stale label Oct 14, 2023
@github-actions github-actions bot closed this Oct 22, 2023
@arnopo arnopo reopened this Oct 25, 2023
@glneo
Copy link
Contributor

glneo commented May 5, 2024

I'll re-open this PR over on the openamp-system-reference repo once the current demos from here and libmetal are moved over. This can be closed.

@github-actions github-actions bot removed the Stale label May 6, 2024
@arnopo arnopo closed this May 6, 2024
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.

4 participants