PSA core 5.12 #982
PSA core 5.12 #982
Conversation
add psa_lifecycle description add psa_platform description add TF-M porting guide extend porting guide with targets and labels description
| @@ -1 +1,3 @@ | |||
| tags | |||
| .DS_Store | |||
There was a problem hiding this comment.
Query: What is this file for? Should it be here?
There was a problem hiding this comment.
these files are annoying artefacts created by MacOS filesystem - https://en.wikipedia.org/wiki/.DS_Store
| @@ -0,0 +1,3 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
Query: What is this file for? Should it be here?
There was a problem hiding this comment.
used this script to generate the nice PNG state machine graph.
please suggest the best place for it.
| @@ -0,0 +1,11 @@ | |||
| digraph { | |||
There was a problem hiding this comment.
Query: What is this file for? Should it be here?
There was a problem hiding this comment.
used this file in DOT language for describing the lifecycle state machine.
please suggest the best place for it.
| | ||
| In mbed-os PSA Lifecycle is implemented as part of [platform service](../platform_servcie.md) | ||
| | ||
| TODO: link to doxygen No newline at end of file |
There was a problem hiding this comment.
Please add an example, in addition to the Doxygen.
There was a problem hiding this comment.
Also, what are the use cases for this API?
There was a problem hiding this comment.
what kind of example required here?
| Platform service introduces system reset and PSA Lifecycle APIs. | ||
| PSA platform service may be extended by target vendors to expose target peripherals only accessible from secure side. | ||
| | ||
| Todo: link to doxygen No newline at end of file |
There was a problem hiding this comment.
Query: What are the use cases for this API?
Also, please add an example, in addition to the Doxygen.
| | ||
| - PSA [RoT](../introduction/glossary.html) internal storage. | ||
| - [PSA Crypto APIs (on GitHub)](https://github.com/ARMmbed/mbed-crypto/tree/development/docs). | ||
| - PSA Attestation - TBD |
There was a problem hiding this comment.
I don't like including "TBD" in our docs. Does anyone oppose removing this line until we have an Attestation link?
| | ||
| <span class="notes">This page gives guidelines for silicon partners adding SPM capabilities.</span> | ||
| | ||
| mbed-os |
There was a problem hiding this comment.
Is this a placeholder for anything, or can we remove this line?
| | ||
| | ||
| #### JSON target definition | ||
| |
There was a problem hiding this comment.
Should we add a link to this page: https://os.mbed.com/docs/mbed-os/development/porting/porting-targets.html?
| | ||
| ### TF-M SPM porting for (ARMv8-M targets) | ||
| | ||
| TF-M HAL functions are defined in `tfm_spm_hal.h` |
There was a problem hiding this comment.
Let's link to this once the Doxygen merges. Also, is there more to add to this section? It feels sparse.
There was a problem hiding this comment.
i'm not sure why it is not generated
according to docs sync meeting
| sorry i opened this PR using our team account - closing and will reopen with my account |
4 participants
CC @GuyWi @avolinski @amiraloosh