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

optee-utee: make panic_handler optional in no-std by adding "no_panic_handler" feature flag #146

Closed
wants to merge 1 commit into from

Conversation

ivila
Copy link
Contributor

@ivila ivila commented Oct 25, 2024

This PR is going to provide a feature flag no_panic_handler, which allow to disable the default panic_handler optee_utee provided, and let the TA developer develop their own one.
Relative issue is here: #145

@DemesneGH
Copy link
Contributor

@ivila Thanks for your contribution! Since the main branch is the current developing branch, could you rebase on the main branch and recreate a PR? thanks!

@DemesneGH
Copy link
Contributor

Sorry for a pending PR for main, I have merged that so it's good to be rebased on now

@ivila
Copy link
Contributor Author

ivila commented Oct 25, 2024

Sorry for a pending PR for main, I have merged that so it's good to be rebased on now

You can do that first, I am kind of busy now, and I should recreate a valid PR about 15:00 today.

@b49020
Copy link
Contributor

b49020 commented Oct 25, 2024

Care to update commit header and description as follows?

no-std: optee-utee: make panic_handler optional

Make panic_handler optional for no-std TAs by adding "no_panic_handler"
feature flag. It allows no-std TAs to have a custom panic handler.

@ivila
Copy link
Contributor Author

ivila commented Oct 25, 2024

PR close due to using the wrong branch (should use main branch instead of no-std)
The recreate one is #147

@ivila ivila closed this Oct 25, 2024
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.

3 participants