DT improvements #7596
DT improvements #7596
Conversation
|
Fixed stub compile issues |
|
This pull request has been marked as a stale pull request because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment, otherwise this pull request will automatically be closed in 5 days. Note, that you can always re-open a closed issue at any time. |
|
FYI: I will fix the commit message subject: to: |
etienne-lms
left a comment
etienne-lms
left a comment
There was a problem hiding this comment.
@bith3ad, @jenswi-linaro, would be be okay to have the fixup commits squashed?
Yes, please |
bith3ad
force-pushed
the
topic/dt-improvements
branch
from
39a2ca8 to
1058745
Compare
|
etienne-lms
left a comment
etienne-lms
left a comment
There was a problem hiding this comment.
Reviewed-by: Etienne Carriere <[email protected]> with a minor comment.
@etienne-lms The review is for the whole series or only for a particular patch? Also since I'm not to familiar with the OP-TEE dev-flow. I need to add the r-b tag and do a force-push? |
|
Sorry, I was not explicit. My R-b tag stands for the whole series. |
Currently add_res_mem_dt_node() doesn't add a overlay fragment if CFG_EXTERNAL_DTB_OVERLAY=y and the provided DTB already contains a "/reserved-memory" e.g. due to some co-processor reserved-memory descriptions. To fix this add_res_mem_dt_node() must always add a "/reserved-memory" DTB overlay fragment if a DTB overlay shall be created (_CFG_USE_DTB_OVERLAY=y). Reviewed-by: Etienne Carriere <[email protected]> Signed-off-by: Marco Felsch <[email protected]>
Exentend the API to be able to specify the DTB overlay "target-path". Reviewed-by: Etienne Carriere <[email protected]> Signed-off-by: Marco Felsch <[email protected]>
Add a helper to add overlays to an external-dt for a caller provided node. The overlay can be used by the caller to overwrite node properties. The subsequent BL33 can use the overlay to apply the changes to the BL33 DTB and kernel DTB. Reviewed-by: Etienne Carriere <[email protected]> Signed-off-by: Marco Felsch <[email protected]>
Add support to write the "secure-status" property to overlays in addition to the inline DTB changes if the user enabled the overlay support. Most BL33 firmwares don't reuse the DTB provided to OP-TEE. Therefore add an overlay for the requested node to not lose the changes done by OP-TEE. The overlay can be used by the BL33 firmware to apply the changes. Reviewed-by: Etienne Carriere <[email protected]> Signed-off-by: Marco Felsch <[email protected]>
bith3ad
force-pushed
the
topic/dt-improvements
branch
from
1058745 to
1491648
Compare
|
3 participants
Hi,
with the following a few helpers are added to OP-TEE dt lib and some improvements are made:
reserved-memoryhandling in case a of-overlay fragment should be added by OP-TEEsecure-statewhich is required for the BL33 if to collect all changes done by OP-TEE.