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

doc: tfm: move overview to a separate page #20874

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

greg-fer
Copy link
Contributor

Moved the Overview section of the TF-M doc to a separate page. NCSDK-32307 and NCSDK-9188.

@greg-fer greg-fer added CI-disable Disable CI for this PR doc only labels Mar 12, 2025
@greg-fer greg-fer added this to the 3.0.0 milestone Mar 12, 2025
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Mar 12, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 12, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 17

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

Note: This message is automatically posted and updated by the CI

@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch from dab0adf to 647ce8f Compare March 12, 2025 10:03
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch 4 times, most recently from d73fb37 to 4b84ead Compare March 24, 2025 13:15
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Mar 24, 2025
@greg-fer greg-fer marked this pull request as ready for review March 24, 2025 13:16
@greg-fer greg-fer requested review from a team as code owners March 24, 2025 13:16
@greg-fer greg-fer changed the title [WIP] doc: tfm: move overview to a separate page doc: tfm: move overview to a separate page Mar 24, 2025
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch 3 times, most recently from 78d01f5 to 3016ef8 Compare March 24, 2025 14:33
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch from 3016ef8 to 0699eed Compare March 26, 2025 12:29
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch 2 times, most recently from b732bed to b32e031 Compare March 26, 2025 14:20
@greg-fer greg-fer requested a review from peknis March 26, 2025 14:21
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch from b32e031 to 0e2ea34 Compare March 26, 2025 14:55
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch from 0e2ea34 to 5cad585 Compare March 27, 2025 07:31
The default configuration of TF-M has all supported features enabled, which results in a significant memory footprint.
For this reason, the |NCS| provides a minimal version of the TF-M secure application, which shows how to configure a reduced version of TF-M.

The secure services supported by the minimal version allow for:
Copy link
Contributor

Choose a reason for hiding this comment

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

In TF-M, functions are either available from the SPE only or from both NSPE+SPE. As a user, it is important to know which functionality I can use from the NSPE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to think where to put this.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that this is solved if we mention that APIs linked to below can be called from the non-secure side, such as is mentioned with Reboot.
For example :
"Using the :ref:platform services <ug_tfm_services_platform> with tfm_platform_api.h from the non-secure side"

@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch 2 times, most recently from c97056d to db8950e Compare March 28, 2025 07:56
Moved the Overview section of the TF-M doc to a separate page.
Moved the PSA Certified API page one level up.
Added information about security services supported by minimal build.
NCSDK-32307, NCSDK-15032, and NCSDK-9188.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer greg-fer force-pushed the doc_tfm_supported_services_limitations branch from db8950e to 3507ec4 Compare March 28, 2025 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-disable Disable CI for this PR doc only doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants