Skip to content

Optee os 3.20+fio testing#1019

Closed
igoropaniuk wants to merge 2 commits intofoundriesio:mainfrom
igoropaniuk:optee-os-3.20+fio-testing
Closed

Optee os 3.20+fio testing#1019
igoropaniuk wants to merge 2 commits intofoundriesio:mainfrom
igoropaniuk:optee-os-3.20+fio-testing

Conversation

@igoropaniuk
Copy link
Copy Markdown
Contributor

No description provided.

ricardosalveti and others added 2 commits January 26, 2023 22:19
Update OP-TEE to 3.20.

Signed-off-by: Ricardo Salveti <[email protected]>
Signed-off-by: Igor Opaniuk <[email protected]>
Update plug-and-trust to the latest release.

Relevant changes:
- b4fbe94 scp03 challenge: do not call OP-TEE
- 13497ec se050x: policy: do not modify requested policy
- 67cac4f ecdsa: op-tee signature handling
- 6068e92 ecdsa: support P-521 curve

Signed-off-by: Igor Opaniuk <[email protected]>
CROSS_COMPILE_ta_${OPTEE_ARCH}=${HOST_PREFIX} \
CFG_WERROR=y DEBUG=0 CFG_OPTEE_REVISION_EXTRA=+fio \
CFG_TEE_CORE_LOG_LEVEL=2 CFG_TEE_TA_LOG_LEVEL=2 \
CFG_ENABLE_EMBEDDED_TESTS=y \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good for testing but we need to remember to drop before merging the final pr.

Copy link
Copy Markdown
Contributor

@angolini angolini left a comment

Choose a reason for hiding this comment

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

Assuming the drop of 3.18 is intentional, the PR LGTM

Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti
Copy link
Copy Markdown
Member

Added the se050 fallback and a bump to the latest from our repo at #1059 (taking this pr as reference). Will close this one in favor of the new one.

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.

5 participants