brain-hackers_pepepper-mirror
/
u-boot-brain
mirror of https://github.com/brain-hackers/u-boot-brain
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

lib: optee: migration optee_copy_fdt_nodes for OF_LIVE support 

The optee_copy_fdt_nodes is only used to copy op-tee nodes
of U-Boot device tree (from gd->fdt_blob when OF_LIVE is not activated)
to external device tree but it is not compatible with OF_LIVE.

This patch migrates all used function fdt_ functions to read node on
old_blob to ofnode functions, compatible with OF_LIVE and remove this
parameter "old_blob".

The generated "device tree" is checked on stm32mp platform with OF_LIVE
activated.

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
This commit is contained in:
Patrick Delaunay 2021-02-08 13:54:31 +01:00 committed by Tom Rini
parent 67696abf1f
commit a2535243e0
3 changed files with 21 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
common/image-fdt.c
View File

@ -562,7 +562,7 @@ int image_setup_libfdt(bootm_headers_t *images, void *blob,
goto err; goto err;
} }
fdt_ret = optee_copy_fdt_nodes(gd->fdt_blob, blob); fdt_ret = optee_copy_fdt_nodes(blob);
if (fdt_ret) { if (fdt_ret) {
printf("ERROR: transfer of optee nodes to new fdt failed: %s\n", printf("ERROR: transfer of optee nodes to new fdt failed: %s\n",
fdt_strerror(fdt_ret)); fdt_strerror(fdt_ret));

4
include/tee/optee.h
View File

@ -71,9 +71,9 @@ static inline int optee_verify_bootm_image(unsigned long image_addr,
#endif #endif
#if defined(CONFIG_OPTEE) && defined(CONFIG_OF_LIBFDT) #if defined(CONFIG_OPTEE) && defined(CONFIG_OF_LIBFDT)
int optee_copy_fdt_nodes(const void *old_blob, void *new_blob); int optee_copy_fdt_nodes(void *new_blob);
#else #else
static inline int optee_copy_fdt_nodes(const void *old_blob, void *new_blob) static inline int optee_copy_fdt_nodes(void *new_blob)
{ {
return 0; return 0;
} }

45
lib/optee/optee.c
View File

@ -9,6 +9,8 @@
#include <image.h> #include <image.h>
#include <log.h> #include <log.h>
#include <malloc.h> #include <malloc.h>
#include <dm/ofnode.h>
#include <linux/ioport.h>
#include <linux/libfdt.h> #include <linux/libfdt.h>
#include <tee/optee.h> #include <tee/optee.h>
@ -70,17 +72,11 @@ error:
} }
#if defined(CONFIG_OF_LIBFDT) #if defined(CONFIG_OF_LIBFDT)
static int optee_copy_firmware_node(const void *old_blob, void *fdt_blob) static int optee_copy_firmware_node(ofnode node, void *fdt_blob)
{ {
int old_offs, offs, ret, len; int offs, ret, len;
const void *prop; const void *prop;
old_offs = fdt_path_offset(old_blob, "/firmware/optee");
if (old_offs < 0) {
debug("Original OP-TEE Device Tree node not found");
return old_offs;
}
offs = fdt_path_offset(fdt_blob, "/firmware"); offs = fdt_path_offset(fdt_blob, "/firmware");
if (offs < 0) { if (offs < 0) {
offs = fdt_path_offset(fdt_blob, "/"); offs = fdt_path_offset(fdt_blob, "/");
@ -97,7 +93,7 @@ static int optee_copy_firmware_node(const void *old_blob, void *fdt_blob)
return offs; return offs;
/* copy the compatible property */ /* copy the compatible property */
prop = fdt_getprop(old_blob, old_offs, "compatible", &len); prop = ofnode_get_property(node, "compatible", &len);
if (!prop) { if (!prop) {
debug("missing OP-TEE compatible property"); debug("missing OP-TEE compatible property");
return -EINVAL; return -EINVAL;
@ -108,7 +104,7 @@ static int optee_copy_firmware_node(const void *old_blob, void *fdt_blob)
return ret; return ret;
/* copy the method property */ /* copy the method property */
prop = fdt_getprop(old_blob, old_offs, "method", &len); prop = ofnode_get_property(node, "method", &len);
if (!prop) { if (!prop) {
debug("missing OP-TEE method property"); debug("missing OP-TEE method property");
return -EINVAL; return -EINVAL;
@ -121,19 +117,18 @@ static int optee_copy_firmware_node(const void *old_blob, void *fdt_blob)
return 0; return 0;
} }
int optee_copy_fdt_nodes(const void *old_blob, void *new_blob) int optee_copy_fdt_nodes(void *new_blob)
{ {
int nodeoffset, subnode, ret; ofnode node, subnode;
struct fdt_resource res; int ret;
struct resource res;
if (fdt_check_header(old_blob))
return -EINVAL;
if (fdt_check_header(new_blob)) if (fdt_check_header(new_blob))
return -EINVAL; return -EINVAL;
/* only proceed if there is an /firmware/optee node */ /* only proceed if there is an /firmware/optee node */
if (fdt_path_offset(old_blob, "/firmware/optee") < 0) { node = ofnode_path("/firmware/optee");
if (!ofnode_valid(node)) {
debug("No OP-TEE firmware node in old fdt, nothing to do"); debug("No OP-TEE firmware node in old fdt, nothing to do");
return 0; return 0;
} }
@ -148,20 +143,17 @@ int optee_copy_fdt_nodes(const void *old_blob, void *new_blob)
return 0; return 0;
} }
ret = optee_copy_firmware_node(old_blob, new_blob); ret = optee_copy_firmware_node(node, new_blob);
if (ret < 0) { if (ret < 0) {
printf("Failed to add OP-TEE firmware node\n"); printf("Failed to add OP-TEE firmware node\n");
return ret; return ret;
} }
/* optee inserts its memory regions as reserved-memory nodes */ /* optee inserts its memory regions as reserved-memory nodes */
nodeoffset = fdt_subnode_offset(old_blob, 0, "reserved-memory"); node = ofnode_path("/reserved-memory");
if (nodeoffset >= 0) { if (ofnode_valid(node)) {
for (subnode = fdt_first_subnode(old_blob, nodeoffset); ofnode_for_each_subnode(subnode, node) {
subnode >= 0; const char *name = ofnode_get_name(subnode);
subnode = fdt_next_subnode(old_blob, subnode)) {
const char *name = fdt_get_name(old_blob,
subnode, NULL);
if (!name) if (!name)
return -EINVAL; return -EINVAL;
@ -170,8 +162,7 @@ int optee_copy_fdt_nodes(const void *old_blob, void *new_blob)
continue; continue;
/* check if this subnode has a reg property */ /* check if this subnode has a reg property */
ret = fdt_get_resource(old_blob, subnode, "reg", 0, ret = ofnode_read_resource(subnode, 0, &res);
&res);
if (!ret) { if (!ret) {
struct fdt_memory carveout = { struct fdt_memory carveout = {
.start = res.start, .start = res.start,