-
Notifications
You must be signed in to change notification settings - Fork 1.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
doc: tfm: Add documentation regarding configurable build #19562
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)
Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
426dd6c
to
536d653
Compare
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.
Some comments. Moving to right direction.
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
536d653
to
e15f633
Compare
82f0ec3
to
de44e9e
Compare
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
More comments/suggestions coming later this week. |
dc41adb
to
e06cb9f
Compare
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
853ef7a
to
dc5176a
Compare
0a11c44
to
da30aad
Compare
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.
Almost there :)
doc/nrf/releases_and_maturity/migration/migration_guide_spm_to_tf-m.rst
Outdated
Show resolved
Hide resolved
003a4c3
to
e6cc22b
Compare
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. As discussed with both Seppo and Markus, more improvements can be made, but we can tackle them in separate PRs.
c68aed4
to
a58b96b
Compare
e93f930
to
58d6ba6
Compare
Add documentation regarding CONFIG_TFM_PROFILE_TYPE_NOT_SET and various TF-M partitions that user need to configure. Move SPM migration guide to own page Current user guide does not need this information anymore. It should be moved to a separate migration guide. Restructure sections Move building, configuring and limitations to appear before background information on TF-M user guide. Signed-off-by: Seppo Takalo <[email protected]> Signed-off-by: Markus Lassila <[email protected]>
58d6ba6
to
d52e87f
Compare
Add documentation regarding CONFIG_TFM_PROFILE_TYPE_NOT_SET and various TF-M partitions that user need to configure.
NOTE: This is very early phase. This work is still in progress, but I'm just starting the review process very early.