-
Notifications
You must be signed in to change notification settings - Fork 171
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
Virtio exp #213
Virtio exp #213
Conversation
Signed-off-by: Vincent Guittot <[email protected]> Signed-off-by: Dan Milea <[email protected]>
Signed-off-by: Dan Milea <[email protected]>
Signed-off-by: Dan Milea <[email protected]>
Signed-off-by: Dan Milea <[email protected]>
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.
@@ -0,0 +1,5 @@ | |||
collect (PROJECT_LIB_HEADERS sys.h) |
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.
#208 probably makes this patch obsolete
I can move the virtio_exp
branch on top of it
@@ -31,7 +31,7 @@ extern "C" { | |||
} | |||
|
|||
#ifndef METAL_MAX_DEVICE_REGIONS | |||
#define METAL_MAX_DEVICE_REGIONS 1 | |||
#define METAL_MAX_DEVICE_REGIONS 2 |
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.
This should be defined in zephyr. in prj.conf or in Cmakelists.txt
for instance: https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/subsys/ipc/openamp_rsc_table/CMakeLists.txt#L15
@@ -0,0 +1,14 @@ | |||
# |
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.
This one is OK for the virtio_exp branch to avoid to fork the zephyrproject-rtoss/open-amp and zephyrproject-rtos/libmetal repos.
Please, could you add the information in commit subject
@@ -0,0 +1 @@ | |||
aarch64 |
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.
should also be addressed in #208
virtio-mmio framework for libmetal / open-amp