-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Crypto platform renaming #9452
Crypto platform renaming #9452
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me except for typos in the commit messages
@@ -24,5 +24,5 @@ To port your CC 310 driver to Mbed OS on your specific target, do the following: | |||
1. In `features/cryptocell/FEATURE_CRYPTOCELL310/TARGET_<target name>`, add your platform-specific libraries for all toolchains in `TOOLCHAIN_ARM`, `TOOLCHAIN_GCC_ARM` and `TOOLCHAIN_IAR` respectively. | |||
1. Add your CC setup code: | |||
* Implement `crypto_platform_setup()` and `crypto_platform_terminate()` to enable CC on your platform, in case you have board-specific setup functionality, required for CC setup. You MUST call 'SaSi_LibInit()` and 'SaSi_LibFini()' respectively in these functions. | |||
* Define `crypto_platform_ctx` in `crypto_platform.h` in a way that suits your implementation. | |||
* Define `crypto_platform_ctx` in `crypto_device_platform.h` in a way that suits your implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any other documentation we need to update for this rename?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Raised ARMmbed/mbed-os-5-docs#929 to change the docc as well
To avoid collisions between the psa `crypto_platform.h` file and the platform specific `crypto_platform.h` file, for the init \ terminate functions, rename the latter to `crypto_device_platform`.
Rename the generic name `ctx` of `mbedtls_platform_context` in `platform_alt.c` to a specific name `plat_ctx`, to avoid conflicts when used as external in crypto modules.
75eab97
to
cad40e1
Compare
@Patater Thank you for your review. |
@RonEld, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
CI started |
Test run: FAILEDSummary: 1 of 8 test jobs failed Failed test jobs:
|
Restarted CI |
Test run: FAILEDSummary: 1 of 12 test jobs failed Failed test jobs:
|
Description
ctx
ofmbedtls_platform_context
in
platform_alt.c
to a specific nameplat_ctx
, to avoid conflictswhen used as external in crypto modules.
crypto_platform.h
file and theplatform specific
crypto_platform.h
file, for the init \ terminatefunctions, rename the latter to
crypto_device_platform
Pull request type
Reviewers
@ARMmbed/mbed-os-crypto