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

Changes in psa_crypto_aead file #9

Draft
wants to merge 3 commits into
base: renesas_master
Choose a base branch
from

Conversation

LavanyaSelvaraju
Copy link
Collaborator

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

Fixed the mbedtls build error when used with keil.

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.

@@ -52,7 +52,7 @@ static psa_status_t psa_aead_setup(

key_bits = attributes->core.bits;

#if defined (MBEDTLS_PSA_CRYPTO_ACCEL_DRV_C)
#if defined (MBEDTLS_PSA_CRYPTO_ACCEL_DRV_C) && (defined(MBEDTLS_GCM_ALT) || defined(MBEDTLS_GCM_ALT))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
#if defined (MBEDTLS_PSA_CRYPTO_ACCEL_DRV_C) && (defined(MBEDTLS_GCM_ALT) || defined(MBEDTLS_GCM_ALT))
#if defined (MBEDTLS_PSA_CRYPTO_ACCEL_DRV_C) && (defined(MBEDTLS_GCM_ALT) || defined(MBEDTLS_CCM_ALT))

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.

2 participants