-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
suit: Align cpuapp partitions to new memory map #19607
base: main
Are you sure you want to change the base?
suit: Align cpuapp partitions to new memory map #19607
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 56551a9abc578827d7ccf8abb46bbccd77fbb1d5 more detailssdk-nrf:
Github labels
List of changed files detected by CI (2)
Outputs:ToolchainVersion: b77d8c1312 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
c02e395
to
b90e973
Compare
@@ -35,11 +38,11 @@ | |||
#size-cells = < 0x1 >; | |||
|
|||
/delete-node/ partition@54000; | |||
cpurad_slot0_partition: partition@66000 { | |||
reg = < 0x66000 DT_SIZE_K(160) >; | |||
cpurad_slot0_partition: partition@54000 { |
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.
I think recovery partition should be located below main partition (to decrease a risk of overwrite)
cpuapp_recovery_partition: partition@a5000 { | ||
reg = <0xa5000 DT_SIZE_K(84)>; | ||
cpuapp_recovery_partition: partition@94000 { | ||
reg = <0x94000 DT_SIZE_K(84)>; | ||
}; |
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.
no cpuapp_recovery_rfu_partition?
Align smp_transfer partitions, as proposed in: nrfconnect#19044 Ref: NCSDK-NONE Signed-off-by: Tomasz Chyrowicz <[email protected]>
b90e973
to
56551a9
Compare
Align smp_transfer partitions, as proposed in:
#19044
Ref: NCSDK-NONE