OpenAMP / open-amp

The main OpenAMP library implementing RPMSG, Virtio, and Remoteproc for RTOS etc
https://www.openampproject.org/
Other
706 stars 288 forks source link

Support for baremetal R5 in TI K3 family devices #503

Closed kwillis01 closed 5 months ago

kwillis01 commented 1 year ago

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.

tnmysh commented 1 year ago

@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 commented 1 year ago

@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 commented 1 year ago

@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.

github-actions[bot] commented 11 months ago

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

glneo commented 5 months ago

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.