-
Notifications
You must be signed in to change notification settings - Fork 17
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
Open-AMP update to SHA 5fce292fd447 #17
Open-AMP update to SHA 5fce292fd447 #17
Conversation
@carlocaione @arnopo please review |
In this PR I've taken also this commit : OpenAMP/open-amp@e63d07d and I get the following warnings:
when compiling openamp_rsc_table sample for nxp_adsp_imx8m: Not quite sure how to fix this. |
Probably @arnopo must take a look here but it looks a name clash to me. Maybe we can rename the zephyr define? |
The issue is in open-amp lib. the VRING_ALIGNMENT is a system definition that should be common to the main and the coprocessor. |
I see that a PR was created: OpenAMP/open-amp#505 I'll wait for this PR, from open-amp repo, to be merge and then I'll update mine. |
@iuliana-prodan |
Yes, I've tested it and everything works perfectly.
@carlocaione what do you think about this? |
At least we should wait for that fix to be merged, then we can update this commit with the new SHA. |
Synchronize code with sha1 5fce292fd447c2396731efeb7852ab7d84454aa1 Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
Use WITH_DCACHE option for all cache operations: vrings, buffers and resource table. The other options will be deprecated. Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
d975406
to
e58b256
Compare
Updated open-amp to 5fce292fd447c2396731efeb7852ab7d84454aa1 from https://github.com/OpenAMP/open-amp/commits/main |
@carlocaione |
@iuliana-prodan after the freeze is over please push a Zephyr commit to update the manifest SHA |
|
Open-AMP update to SHA 5fce292fd447c2396731efeb7852ab7d84454aa1 - https://github.com/OpenAMP/open-amp/commits/main