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

drivers: caam: update return type of caam_sm_free() #6533

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

sahilnxp
Copy link
Contributor

@sahilnxp sahilnxp commented Dec 7, 2023

Update return type of caam_sm_free() from TEE_Result to enum caam_status.

@sahilnxp
Copy link
Contributor Author

sahilnxp commented Dec 7, 2023

Sorry, There is some error in this patch formatting I am correcting it.

@sahilnxp
Copy link
Contributor Author

sahilnxp commented Dec 7, 2023

Sorry, There is some error in this patch formatting I am correcting it.

Done.

@jforissier
Copy link
Contributor

Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>

@clementfaure
Copy link
Contributor

Acked-by: Clement Faure <clement.faure@nxp.com>

Update return type of caam_sm_free() from TEE_Result
to enum caam_status.

Signed-off-by: Sahil Malhotra <sahil.malhotra@nxp.com>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Clement Faure <clement.faure@nxp.com>
@sahilnxp
Copy link
Contributor Author

sahilnxp commented Dec 7, 2023

Tags applied.

@jforissier jforissier merged commit dc6563d into OP-TEE:master Dec 8, 2023
8 checks passed
@sahilnxp sahilnxp deleted the patch-1 branch December 9, 2023 06:04
smk-embedded pushed a commit to phytec/meta-phytec that referenced this pull request Jan 19, 2024
Fix build issue on master branch by adding a fix from master
branch.

See OP-TEE/optee_os#6533

Signed-off-by: Teresa Remmet <t.remmet@phytec.de>
Signed-off-by: Stefan Müller-Klieser <s.mueller-klieser@phytec.de>
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