openamp sample integration/app #7416
Merged
Conversation
This file contains hidden or 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
dbkinder
previously requested changes
May 9, 2018
| @@ -0,0 +1,61 @@ | |||
| .. _lpcxpresso54114_m0: | |||
|
|
|||
| NXP LPCXPRESSO54114 | |||
There was a problem hiding this comment.
might-should add M0 or maybe (M0 core) to this title. Without it, it has the same title as the board doc we've already got.
| as dropping characters, have been observed with older versions of the firmware. | ||
|
|
||
| The code for the secondary core is linked into the primary core binary file. | ||
| Startup code copies the secondary core's code into an appropriate location |
There was a problem hiding this comment.
Might be work clarifying (assuming I've got this right):
Startup code for the primary core copies the secondary core's code into an
appropriate location in RAM and starts the secondary core's execution.
Closed
|
@dbkinder can you review this now from a doc perspective. |
|
This is the first step towards a fuller solution for #3065. We expect in the future to update the openamp implementation both for improved integration with Zephyr, as well as reduce code footprint from an improved openamp library. |
Origin: https://github.com/OpenAMP/libmetal Status: 606c31438025b9fb1515dace1c642d5835d8d33c [v2018.04] When we import libmetal we removed the tests/ and examples/ dir to reduce the amount of code imported. Purpose: HAL abstraction layer used by open-amp Description: Libmetal provides common user APIs to access devices, handle device interrupts and request memory across the following operating environments: * Linux user space (based on UIO and VFIO support in the kernel) * RTOS (with and without virtual memory) * Bare-metal environments Dependencies: Depends on Zephyr itself as it utilizes Zephyr's APIs to provide an abstraction to open-amp. URL: https://github.com/OpenAMP/libmetal commit: 606c31438025b9fb1515dace1c642d5835d8d33c Maintained-by: External License: BSD-3-Clause License Link: https://github.com/OpenAMP/libmetal/blob/master/LICENSE.md Signed-off-by: Kumar Gala <[email protected]>
Origin: https://github.com/OpenAMP/open-amp Status: de361adee09cd31793c60218a0ec49bc307a7410 [v2018.04] When we import open-amp we removed the apps dir to reduce the amount of code imported. Purpose: IPC layer that implements rpmsg communication between cores. Description: This repository is the home for the Open Asymmetric Multi Processing (OpenAMP) framework project. The OpenAMP framework provides software components that enable development of software applications for Asymmetric Multiprocessing (AMP) systems. The framework provides the following key capabilities. * Provides Life Cycle Management, and Inter Processor Communication capabilities for management of remote compute resources and their associated software contexts. * Provides a stand alone library usable with RTOS and Baremetal software environments * Compatibility with upstream Linux remoteproc and rpmsg components * Following AMP configurations supported: a. Linux master/Generic(Baremetal) remote b. Generic(Baremetal) master/Linux remote * Proxy infrastructure and supplied demos showcase ability of proxy on master to handle printf, scanf, open, close, read, write calls from Bare metal based remote contexts. Dependencies: libmetal (https://github.com/OpenAMP/libmetal) - provides HAL layer between OpenAMP and RTOS or OS environment. URL: https://github.com/OpenAMP/open-amp/ commit: de361adee09cd31793c60218a0ec49bc307a7410 Maintained-by: External License: BSD-3-Clause BSD-2-Clause Signed-off-by: Kumar Gala <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #7416 +/- ##
==========================================
- Coverage 55.01% 55.01% -0.01%
==========================================
Files 483 483
Lines 53950 53950
Branches 10495 10495
==========================================
- Hits 29680 29679 -1
- Misses 19984 19985 +1
Partials 4286 4286
Continue to review full report at Codecov.
|
SebastianBoe
previously requested changes
May 18, 2018
This commit adds a sample application using OpenAMP for remote procedure calls on the LPCXpresso54114. It is adapted from the RPMsg-Lite sample application added in PR zephyrproject-rtos#5960, and uses the IPM driver to provide interprocessor interrupts. Signed-off-by: Kristian Klomsten Skordal <[email protected]> Signed-off-by: Kumar Gala <[email protected]>
MaureenHelm
force-pushed
the
openamp-sample
branch
from
5ab35bd to
6cc6c83
Compare
|
Fixed the checkpatch error and all of the warnings except for: |
MaureenHelm
dismissed
SebastianBoe’s stale review
Filed blocking issue #7673 to fix before 1.12 release
MaureenHelm
approved these changes
May 19, 2018
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.
Example of using open-amp.
This example docs need some cleaning up, but this is based on #7415 and #7161