doc: nsib: remove FEM mention and move some docs #12398
Conversation
greg-fer
requested review from
hakonfam,
oyvindronningstad and
carlescufi
as code owners
github-actions
bot
added
changelog-entry-required
|
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 Publishing GitHub Action. |
| This public key is checked against the provisioned hashes of public keys to determine if the image is valid. | ||
|
|
||
| All public key hashes at lower indices than the matching hash are permanently invalidated at this point. | ||
| You can use this mechanism to decommission broken keys. |
There was a problem hiding this comment.
I wonder if broken is the right word here. Maybe stolen/cracked keys?
There was a problem hiding this comment.
Yes i like the suggestion above
|
|
||
| All peripherals that have been used are reset and the next stage is booted. | ||
|
|
||
| Except for providing your own keys, there is no need to modify the immutable bootloader in any way before you program it. |
There was a problem hiding this comment.
there is no need to modify the immutable bootloader in any way before you program it.
Something seems off with this statement.
I agree that you do not have to do this.
However, in several situations, it is beneficial to configure the immutable bootloader. To mention one: If you want MCUboot as an immutable bootloader with Serial Recovery enabled, you must configure it.
|
For an general overview over my comments: |
greg-fer
force-pushed
the
doc_nsib_sample_structure
branch
from
af955af
to
cffeb59
Compare
|
@sigvartmh , @hellesvik-nordic , @einarthorsrud , kindly please have another look. |
greg-fer
force-pushed
the
doc_nsib_sample_structure
branch
from
cffeb59
to
3f50b85
Compare
Removed mention of FEM from the NSIB readme. Moved generic information about RoT establishment to bootloader docs. Part of NCSIDB-1053. Signed-off-by: Grzegorz Ferenc <[email protected]>
greg-fer
removed
changelog-entry-required
Removed mention of FEM from the NSIB readme.
Moved generic information about RoT establishment to bootloader docs. Part of NCSIDB-1053.