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

Add various configuration components #151

Open
wants to merge 25 commits into
base: development
Choose a base branch
from

Conversation

Harry-Ramsey
Copy link
Contributor

@Harry-Ramsey Harry-Ramsey commented Jan 13, 2025

This commit adds various configuration components to TF-PSA-Crypto. These components have been adapted from Mbed TLS to use CMake. Closes #125.

This pull request has a dependency on: #164

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog not required because: testing enhancement.
  • framework PR not required.
  • mbedtls PR not required.
  • tests provided.

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

aditya-deshpande-arm and others added 23 commits April 11, 2023 16:25
Add change log and non-regression test
for CCM* with no tag not supported in
CCM only configuration.

Signed-off-by: Ronald Cron <[email protected]>
Signed-off-by: Dave Rodgman <[email protected]>
Move all changes local to Mbed TLS into config-tfm.h (except for commenting
out a couple of #include's).

Signed-off-by: Dave Rodgman <[email protected]>
Signed-off-by: Dave Rodgman <[email protected]>
Replaces legacy symbols with the PSA equivalents.
This doesn't change the code generated when this config is active

Signed-off-by: Ryan Everett <[email protected]>
Recent commits have changed these reference configurations and they are
not verbatim copies anymore.

Signed-off-by: Janos Follath <[email protected]>
Signed-off-by: Minos Galanakis <[email protected]>
This commit moves the tf-psa-crypto configuration options
for this test to `crypto-config-ccm-aes-sha256.h`.

A blank MbedTLS configuration file is now used for the library
side.

Signed-off-by: Minos Galanakis <[email protected]>
Remove all code guarded by `PSA_WANT_ECC_SECP_K1_224`, which is not and will
not be implemented. (It would be K1_225 anyway, but we don't intend to
implement it anyway.)

Signed-off-by: Gilles Peskine <[email protected]>
@Harry-Ramsey Harry-Ramsey self-assigned this Jan 13, 2025
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration branch 4 times, most recently from 7ebfae7 to 1bd7e54 Compare January 16, 2025 15:26
@Harry-Ramsey Harry-Ramsey added the needs-preceding-pr Requires another PR to be merged first label Jan 21, 2025
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration branch from 1bd7e54 to 815aac7 Compare January 21, 2025 09:33
@Harry-Ramsey Harry-Ramsey added the needs-ci Needs to pass CI tests label Jan 21, 2025
This commit updates configs/README.txt and configs/ext/README.md to
reflect the necessary files which were moved to TF-PSA-Crypto.

Signed-off-by: Harry Ramsey <[email protected]>
This commit adds various configuration components to TF-PSA-Crypto.
These components have been adapted from Mbed TLS to use CMake.

Signed-off-by: Harry Ramsey <[email protected]>
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration branch from 815aac7 to c9df6f6 Compare January 27, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-ci Needs to pass CI tests needs-preceding-pr Requires another PR to be merged first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add components-configurations.sh
9 participants