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

xtest: remove CFG_SECSTOR_TA_MGMT_PTA dependency #768

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

etienne-lms
Copy link
Contributor

Makes xtest tool more flexible regarding OP-TEE configuration of CFG_SECSTOR_TA_MGMT_PTA.

Embed TA install interface in xtest even if devkit configuration says the secure storage TA management PTA service is not available. If the service is not found (secstor TA management PTA) print an explicit error message instead.

This change does not change xtest regression suite behavior and will still fails if one uses --install-ta option when the effective embedded TEE does not provide this service. This change adds an explicit error message when so.

Copy link
Contributor

@jenswi-linaro jenswi-linaro left a comment

Choose a reason for hiding this comment

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

Acked-by: Jens Wiklander <[email protected]>

Makes xtest tool more flexible regarding OP-TEE configuration
of CFG_SECSTOR_TA_MGMT_PTA.

Embed TA install interface in xtest even if devkit configuration
says the secure storage TA management PTA service is not available.
If the service is not found (secstor TA management PTA) print an
explicit error message instead.

This change does not change xtest regression suite behavior and will
still fails if one uses --install-ta option when the effective embedded
TEE does not provide this service. This change adds an explicit error
message when so.

Signed-off-by: Etienne Carriere <[email protected]>
Acked-by: Jens Wiklander <[email protected]>
@jforissier jforissier merged commit 44ec72b into OP-TEE:master Jan 9, 2025
1 check passed
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