[nrf noup] boot: zephyr: Update partition macros#624
[nrf noup] boot: zephyr: Update partition macros#624nordicjm wants to merge 1 commit intonrfconnect:mainfrom
Conversation
nordicjm
force-pushed
the
bringinnewmappedbinding
branch
from
bda12be to
a7de5b4
Compare
Updates partition macros to use the new, non-deprecated macros Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
nordicjm
force-pushed
the
bringinnewmappedbinding
branch
from
a7de5b4 to
f2276ed
Compare
|
| #error "mcuboot resume support section not defined in dts" | ||
| #endif | ||
|
|
||
| #define FIXED_PARTITION_ADDR(node_label) DT_REG_ADDR(DT_NODELABEL(node_label)) |
There was a problem hiding this comment.
@tomchy You suggested that for some purpose. Is this removal/replacement OK?
There was a problem hiding this comment.
If you look here: nrfconnect/sdk-zephyr#3880
The devicetree.h file now includes the <zephyr/devicetree/mapped-partition.h> which is the newly introduced mechanism to get partition address/offset/size, without the need to enable FLASH_MAP in the build.
Additionally, the PR sets all partitions on all Nordic platform to use the new compatible (zephyr,mapped-partition).
We still may need to give it a thought if those changes are backward compatible (imagine an out of tree sample that still uses fixed-partitions and fixed-subpartitions). The code there should handle it fine, but since all platforms are migrated, there are no tests that verify the old syntax.
3 participants
Updates partition macros to use the new, non-deprecated macros
manifest-pr-skip