Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Move some crypto custom configs to TF-PSA-Crypto #9879

Closed
ronald-cron-arm opened this issue Jan 7, 2025 · 3 comments
Closed

Move some crypto custom configs to TF-PSA-Crypto #9879

ronald-cron-arm opened this issue Jan 7, 2025 · 3 comments
Assignees
@Harry-Ramsey
Labels
component-crypto Crypto primitives and low-level interfaces enhancement size-s Estimated task size: small (~2d)

Comments

@ronald-cron-arm
Copy link
Contributor

ronald-cron-arm commented Jan 7, 2025 •
edited
Loading

Suggested enhancement

Move the crypto custom configs that are not used in TLS context to TF-PSA-Crypto configs directory.

Configs to move:

  • configs/crypto-config-symmetric-only.h
  • configs/crypto-config-ccm-aes-sha256.h
  • configs/ext/crypto_config_profile_medium.h together with the README.txt (to be adapted).

The git history of the files should be kept as when we move files to the framework (see Mbed-TLS/mbedtls-docs#145)

Justification

Mbed TLS needs this because of the repo split.
@ronald-cron-arm ronald-cron-arm added enhancement component-crypto Crypto primitives and low-level interfaces size-s Estimated task size: small (~2d) labels Jan 7, 2025
@ronald-cron-arm ronald-cron-arm moved this to TF-PSA-Crypto all.sh components in Mbed TLS Epics Jan 7, 2025
@ronald-cron-arm ronald-cron-arm mentioned this issue Jan 7, 2025
Open
@ronald-cron-arm ronald-cron-arm moved this from TF-PSA-Crypto all.sh components to TF-PSA-Crypto all.sh 2 - misc in Mbed TLS Epics Jan 7, 2025
@Harry-Ramsey Harry-Ramsey self-assigned this Jan 13, 2025
@Harry-Ramsey Harry-Ramsey mentioned this issue Jan 14, 2025
Closed
3 tasks
@Harry-Ramsey
Copy link
Contributor

Sorry, but should it be crypto-config-symmetric-only instead of config-symmetric-only?

@Harry-Ramsey Harry-Ramsey mentioned this issue Jan 23, 2025
Merged
5 tasks
@ronald-cron-arm
Copy link
Contributor Author

ronald-cron-arm commented Jan 24, 2025 •
edited
Loading

Sorry, but should it be crypto-config-symmetric-only instead of config-symmetric-only?

Yes that's right. I've fixed that.

@ronald-cron-arm
Copy link
Contributor Author

Done in #9908

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@Harry-Ramsey Harry-Ramsey

Labels
component-crypto Crypto primitives and low-level interfaces enhancement size-s Estimated task size: small (~2d)
Projects
Mbed TLS Epics
Status: TF-PSA-Crypto all.sh 2 - misc
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move Configs To TF-PSA-Crypto Harry-Ramsey/TF-PSA-Crypto
2 participants
@ronald-cron-arm @Harry-Ramsey