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

zephyr: Enable support for Xtensa #197

Merged
merged 2 commits into from
Jan 19, 2022
Merged

Conversation

dbaluta
Copy link
Contributor

@dbaluta dbaluta commented Jan 8, 2022

Make libmetal usable also for Xtensa targets.

Signed-off-by: Daniel Baluta daniel.baluta@nxp.com

Make libmetal usable also for Xtensa targets.

Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
@carlocaione
Copy link
Collaborator

Can you also add the build test for Xtensa (see #195)?

@dbaluta
Copy link
Contributor Author

dbaluta commented Jan 9, 2022

@carlocaione will do after #195 is merged so that we can avoid conflicts.

@arnopo
Copy link
Contributor

arnopo commented Jan 10, 2022

@carlocaione will do after #195 is merged so that we can avoid conflicts.

Hi @dbaluta
I would prefer that you add the CI test in your pull request. No issue to merge this one before my PR #195

Thanks,
Arnaud

@dbaluta
Copy link
Contributor Author

dbaluta commented Jan 10, 2022

@arnopo all right, will do that today.

Later edit: done!

This adds CI build test for Xtensa architecture.

Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
Copy link
Contributor

@arnopo arnopo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arnopo arnopo added this to the Release V2022.04 milestone Jan 10, 2022
@carlocaione
Copy link
Collaborator

@arnopo do you think that we can tag a new release after me merge the new support for Xtensa e RISCV?

@arnopo
Copy link
Contributor

arnopo commented Jan 18, 2022

@arnopo do you think that we can tag a new release after me merge the new support for Xtensa e RISCV?

I'm waiting @edmooring approval before integrating both.
Then new release is planned in April 2022. If the need is for Zephyr I propose to back port the PRs in zephyr waiting the new release.

Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go.

@arnopo arnopo merged commit 3ba1f1a into OpenAMP:main Jan 19, 2022
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