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

Trusted firmware m add on patches #1

Open
wants to merge 10 commits into
base: mbedtls-3.2.1
Choose a base branch
from

Conversation

adeaarm
Copy link
Owner

@adeaarm adeaarm commented Sep 20, 2022

Notes:

  • Pull requests cannot be accepted until the PR follows the contributing guidelines. In particular, each commit must have at least one Signed-off-by: line from the committer to certify that the contribution is made under the terms of the Developer Certificate of Origin.
  • This is just a template, so feel free to use/remove the unnecessary things

Description

A few sentences describing the overall goals of the pull request's commits.

Status

READY/IN DEVELOPMENT/HOLD

Requires Backporting

When there is a bug fix, it should be backported to all maintained and supported branches.
Changes do not have to be backported if:

  • This PR is a new feature\enhancement
  • This PR contains changes in the API. If this is true, and there is a need for the fix to be backported, the fix should be handled differently in the legacy branch

Yes | NO
Which branch?

Migrations

If there is any API change, what's the incentive and logic for it.

YES | NO

Additional comments

Any additional information that could be of interest

Todos

  • Tests
  • Documentation
  • Changelog updated
  • Backported

Steps to test or reproduce

Outline the steps to test or reproduce the PR here.

TTornblom and others added 10 commits August 23, 2022 12:52
Applied the same change as in mbed-crypto for using this as a sub
project with the IAR toolchain.

Signed-off-by: TTornblom <[email protected]>
MBEDTLS_CHACHA20_C and MBEDTLS_POLY1305_C are needed when
PSA_WANT_ALG_CHACHA20_POLY1305 is defined.

Signed-off-by: Summer Qin <[email protected]>
Manually hardcode PSA driver entry points for the CC3XX driver
into psa_crypto_driver_wrappers.c (and provide missing entry point
definitions if any). This is a temporary solution until the codegen
framework is available for automatic integration.

Signed-off-by: Summer Qin <[email protected]>
Signed-off-by: Salome Thirot <[email protected]>
Signed-off-by: Abbas Bracken Ziad <[email protected]>
Signed-off-by: Georgios Vasilakis <[email protected]>
Signed-off-by: Antonio de Angelis <[email protected]>
Also an LM-OTS implementation as one is required for LMS.

Signed-off-by: Raef Coles <[email protected]>
…ailable

This wil be enforced by the autogen framework eventually, but for the time
being we need to manually enforce it for the CC3XX driver only to make sure
that multiple drivers can coexist indepedently.

Signed-off-by: Antonio de Angelis <[email protected]>
This patch amends the order of initialisations performed in psa_crypto_init()
to make sure that the driver wrappers based on the PSA driver API are initialised
before anything else.

Signed-off-by: Antonio de Angelis <[email protected]>
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.

6 participants